[5.5] Blade @php directive

[MCMatters]

• Laravel Version: 5.5.1
• PHP Version: 7.1.8
• Database Driver & Version: mysql 5.7.19

Description:

Previously there was an ability to create php code with single line of using @php directive. Now I can't use it without @endphp. I didn't see any mentions about it in Upgrade guide or documentation. Is it bug?

Steps To Reproduce:

Try to create view with next code:

@if (isset($team))
    @php ($some = true)
@else
    @php ($some = false)
@endif

@php
    $someElse = true;
@endphp

And you will get

<?php if(isset($team)): ?>
    <?php ($some = true)
@else
    @php ($some = false)
@endif

@php
    $someElse = true;
?>

[themsaid]

@sileence please check, this PR broke the given case: #20065

[sileence]

@themsaid agh, I can see why. I'm working on it.

[themsaid]

@sileence thanks :)

[jmarcher]

Shouldn't it be like this?

@if(isset($team))
    @php($some = true)
@else
    @php($some = false)
@endif

@php
    $someElse = true;
@endphp

[themsaid]

Closing as a no-fix.

You can use regular PHP opening and closing tags.

[MCMatters]

Add to documentation this info and to upgrade guide as well

[ShahidH]

@jmarcher It appear @php($some = true) is longer supported in 5.5

[judgej]

@php(blah) is working in 5.6. Is it back by design, or an anomaly?

[jmarcher]

@judgej It should not be working.

[judgej]

@jmarcher Thanks. I'll put it down to an anomaly, and move to the block format before it stops working on the next update.

[Log1x]

I really wish this wasn't removed...

[Hazzard13]

Hey, can we at least see this added to the upgrade guide? Definitely unhelpful to end up here after nearly an hour of debugging an "Unexpected endforeach".

Don't really understand why the single-line @php was removed, as it makes for much cleaner templates, but it should at least be clear in the guide, as I almost didn't catch this in testing, and followed the upgrade guide to the letter.

[devcircus]

The change was made in 5.5 and mentioned in the upgrade guide. There was a lot of discussion prior to the change due to numerous issues that were not practical to resolve. https://laravel.com/docs/5.5/upgrade

[Hazzard13]

Ah, my apologies. I must've missed while upgrading, because I was upgrading an angular app and probably skimmed the blade section. Sorry to waste your time!

[kapitannwel]

@Hazzard13 - I too like this shortcut format as it seems to be more simple. i usually use this in counting rows @php($counter=0) @foreach() @php($counter++) @Endforeach. Sad to know it's gone. thanks for this thread, as i was informed!

[Log1x]

@Hazzard13 - I too like this shortcut format as it seems to be more simple. i usually use this in counting rows @php($counter=0) @foreach() @php($counter++) @Endforeach. Sad to know it's gone. thanks for this thread, as i was informed!

From my experience, you can still use shorthanded @php() in blade views, you just can't mix it with the long-handed (@php @endphp) version in the same view.

[kapitannwel]

@Hazzard13 - I too like this shortcut format as it seems to be more simple. i usually use this in counting rows @php($counter=0) @foreach() @php($counter++) @Endforeach. Sad to know it's gone. thanks for this thread, as i was informed!

From my experience, you can still use shorthanded @php() in blade views, you just can't mix it with the long-handed (@php @endphp) version in the same view.

i am actually using it on some of my blade files, but it is not working on some blade files, which is weird, that is why i searched on it, thus the reason why i arrived on this thread. knowing this, i think i shouldn't use it anymore to be safe. thank you for your reply. stay safe.

[mirafzal]

regex replace:
@php ((.*))
@php $1 @endphp