Display specific link if page is published only

[mjut]

Hello,
I am having trouble achieving this: I am trying to place a link to a specific page in my template.
But I want this link displayed only, if the page is published.

With this code, its not working:

<?php if ($pages->find("id=1241, status=published")) { ?>
	<a href="<?php echo $pages->get(1241)->url() ?>"><?php echo $pages->get(1241)->title() ?></a>
<?php } ?>

(The link is being displayed no matter the status of the page "1241")

How can I fix this?

Thanks for your help!!!

[Sergio]

<?php 
	$p = $pages->get(1241);
	if (!$p->is(Page::statusUnpublished)) : 
?>
	<a href="<?php echo $p->url() ?>"><?php echo $p->title() ?></a>
<?php endif; ?>

https://processwire.com/api/ref/page/status/

Edited September 24, 2019 by Sergio

[Sergio]

There's also a shortcut:

if (!$p->isUnpublished()) :

 

[mjut]

@Sergio Wow.
This is working. I am no php expert. I am always having trouble on this... But I am on my way to get more comfortable on working with variables....
Thanks a lot!

<?php
  $p = $pages->get(1241);
  if (!$p->isUnpublished()) :
?>
  <a href="<?php echo $p->url() ?>"><?php echo $p->title() ?></a>
<?php endif; ?>

yup - tested! The short version works 100%!

[adrian]

How about this:

<?php
  $p = $pages->findOne(1241);
  if ($p->id) :
?>
  <a href="<?php echo $p->url() ?>"><?php echo $p->title() ?></a>
<?php endif; ?>

Perhaps it's only a minor optimization, but it saves it returning any page object if the page isn't published. 

Be sure to read the docs on findOne() vs get() to understand fully, but basically find() and findOne() won't return unpublished pages unless to specifically request them to, but get() will ignore unpublished and hidden statuses and always return a page if everything else matches.