Skip to main content

This site requires you to update your browser. Your browsing experience maybe affected by not having the most up to date version.

Archive

Our old forums are still available as a read-only archive.

Moderators: martimiz, Sean, biapar, Willr, Ingo, simon_w

Question on Coding Conventions


Reply

5 Posts   1120 Views

Avatar
Double-A-Ron

23 September 2008 at 1:25pm (Last edited: 23 September 2008 1:25pm), Community Member, 604 Posts

[url]http://doc.silverstripe.com/doku.php?id=coding-conventions[/url]

Quote:

Don’t end them with ?> - if you leave it out, it’s implied and it prevents blank lines from accidentally being outputted when you have newlines after the ?>

I understand this. I use this method all the time with standard PHP includes. But I am unsure when it comes to how SS processes php files.

I mean, should this rule apply to Page Types or DataObject files I create in /mysite/?

Does it apply everywhere an extender or module developer uses PHP? I've always been under the impression, that in roll-your-own php apps, this method should only be used on included php files, not primary pages that a user might view directly. (I may have misunderstood this)

Cheers
Aaron Cooper

Avatar
Fuzz10

23 September 2008 at 10:56pm Community Member, 787 Posts

Hmmm.. good question! ;-)

Subscribe....

Avatar
ajshort

23 September 2008 at 11:53pm (Last edited: 23 September 2008 11:54pm), Community Member, 244 Posts

Im fairly sure that this rule is just a good practice to use on every file that is purely PHP. It helps to prevent the "header has already been sent" messages and stuff.

Avatar
Ingo

25 September 2008 at 10:46pm Forum Moderator, 801 Posts

I don't think we should have coding conventions with "exceptions", so yeah, if its written there it should apply consistently for all PHP files you create. I'm more in favor of actually checking your code for trailing blanks and close your tags properly btw, but thats not what the conventions state ;)

Avatar
Double-A-Ron

25 September 2008 at 11:14pm Community Member, 604 Posts

Thanks for the clarification guys.

Aaron