Parse error with short tags enabled [#829316]

Comment	File	Size	Author
	opensearch_shortag_parse_error.patch	774 bytes	neclimdul

Comment #1

neclimdul

He/Him

CreditAttribution: neclimdul commented 16 June 2010 at 16:52

Status:

Active

» Needs review

Log in or register to post comments

Comment #2

Anonymous (not verified) CreditAttribution: Anonymous commented 19 June 2010 at 20:08

As reported in http://drupal.org/coding-standards#phptags, short tags should never be used with Drupal; I take that means they should never be enabled with Drupal.

Log in or register to post comments

Comment #3

Anonymous (not verified) CreditAttribution: Anonymous commented 19 June 2010 at 20:08

Title:

Parse error with short tags enabled.

» Parse error with short tags enabled

Log in or register to post comments

Comment #4

neclimdul

He/Him

CreditAttribution: neclimdul commented 21 June 2010 at 19:17

I would say they are not mutually exclusive. And its not a big deal to support them.

Log in or register to post comments

Comment #5

neclimdul

He/Him

CreditAttribution: neclimdul commented 23 June 2010 at 18:34

This is sort of a critical but trivial bug fix I'd like to see committed. The patch even includes some additional documentation to match the sort of de facto template documentation.

Just a note, this is similar to the approach taken by core's rss.xml handler, views' RSS template, fivestar's AJAX callback and views bonus' xml template. So its pretty common approach to addressing this sort of problem.