#69 ✓resolved
Kyle Gordon

Use of short tags

Reported by Kyle Gordon | February 8th, 2010 @ 07:04 PM

Hi,

Several of the PHP files within Concerto use short tags instead of complete tags, ie <? instead of <?php. This is rather non-portable, and causes major breakage on SUSE systems at the very least. It can be worked around by changing short_open_tag = On, but this is not the default shipping setting for SuSE and possibly others, and is also not recommended.

To quote from php.ini on SuSE, Ubuntu and Debian distros - "Using short tags should be avoided when developing applications or libraries that are meant for redistribution, or deployment on PHP servers which are not under your control, because short tags may not be supported on the target server. For portable, redistributable code, be sure not to use short tags."

I've had a quick look, and there's quite a few files that at least have it as the opening line. There are many more with it embedded in the file somewhere

root@flat:/var/www/concerto# find . -iname "*.php" -exec grep -i '<?$' {} \; | wc -l
58
root@flat:/var/www/concerto#

I'm sure there's a way to mass replace it all with sed/grep/find/etc!

Regards

Kyle

Comments and changes to this ticket

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Concerto is an open source digital signage system that makes it easy to engage a large community of people with graphical and text-based announcements, bulletins, and other messages... but enough about us, you came here to get involved.
<br />
If you think you've <b>found a bug in Concerto</b>, please click 'Create new ticket' and let us know about it.
<br />
Looking to peek at our source code? Head to <a href="https://github.com/concerto/concerto_v1">https://github.com/concerto/concerto_v1</a> (Concerto version 1) or <a href="https://github.com/concerto/concerto">https://github.com/concerto/concerto</a> (Concerto version 2).
<br />
Our getting started guide is available here: <a href="https://github.com/concerto/concerto/wiki/Getting-Involved">https://github.com/concerto/concerto/wiki/Getting-Involved</a>.

Pages