Check page status if page exists.

[Fisiu]

Signed-off-by: Mariusz Fik mariusz@fidano.pl

[nWidart]

Why is this needed?

[Fisiu]

I page is not found we should return 404, no need to check it's staus. We should check status only if page exists.

[nWidart]

This was already done, you just split it in 2 ifs, so why? :D
A better solution might be to inverse the condition order.

is_null($page) || $page->status == 0

[huydang284]

I agree with @nWidart.

It should be is_null($page) || $page->status == 0

[Fisiu]

It took me a while to get back to work on asgardcms. Here is expected implementation.

[nWidart]

Thanks!