mybb / docs.mybb.com

The MyBB documentation.
https://docs.mybb.com
26 stars 75 forks source link

Update standards.md #82

Closed tnadolecki closed 8 years ago

JoshHarmon commented 8 years ago

Good catch! Sad that we blatantly ignored the requirement outlined above it. Thank you for your contribution!

Destroy666x commented 8 years ago

Actually, escaping everything that way is not required, it's even frowned upon if the value is a simple constant string/whatever (no need for escaping in most cases then) or a variable integer/float/whatever ((type) casting is better then). The description is misleading, especially that simple constant strings that don't require escaping are used as examples.

tnadolecki commented 8 years ago

Right, It did not make sense... PS: I will be reading thru more as I'm just beginning to learn... Thanks! T.

2015-11-13 16:02 GMT+00:00 Przemek Pawlas notifications@github.com:

Actually, escaping everything that way is not required, it's even frowned upon if the value is a simple constant string (no need for escaping in most cases then) or a variable integer/float/whatever ((type) casting is better then). The description is misleading.

— Reply to this email directly or view it on GitHub https://github.com/mybb/docs.mybb.com/pull/82#issuecomment-156473566.

JoshHarmon commented 8 years ago

Fair point, @Destroy666x. Would you advise reverting this change?

Destroy666x commented 8 years ago

Yes, and changing the "required" text to something more like "when required", an example with user input ($mybb->get_input('randominput') or whatever) would be nice for that case.

JoshHarmon commented 8 years ago

Alright, I'll revert this and add this to my todo, or @tnadolecki can make the changes and update with an improved PR.