[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Nemo federicol...@tiscali.it changed: What|Removed |Added Flags|Backport_to_Stable? |Backport_to_Stable+ --- Comment #19 from Nemo federicol...@tiscali.it --- Merged -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #16 from Nemo federicol...@tiscali.it --- 1.20 backport was +2'ed by Platonides but needs verified +2 because tests fail. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #17 from Bawolff (Brian Wolff) bawolff...@gmail.com --- Am I allowed to self-verify? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #18 from Nemo federicol...@tiscali.it --- (In reply to comment #17) Am I allowed to self-verify? It's less than a self-merge and IIRC there aren't big objections for self-merge of backports even. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #15 from Mark Clements (HappyDog) w...@kennel17.co.uk --- Great - thanks for the update. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #12 from Mark Clements (HappyDog) w...@kennel17.co.uk --- (In reply to comment #7) Also, this should be back-ported to any previous MW versions that are still supported. Any chance this can be back-ported (or has it already been?) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Platonides platoni...@gmail.com changed: What|Removed |Added CC||platoni...@gmail.com --- Comment #13 from Platonides platoni...@gmail.com --- It was backported to 1.19 and 1.20: https://gerrit.wikimedia.org/r/#/q/Ie1ace158dac1733e6b2b2c1d533004d9bcab8c80,n,z -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Nemo federicol...@tiscali.it changed: What|Removed |Added Flags||Backport_to_Stable? --- Comment #14 from Nemo federicol...@tiscali.it --- Change-Id: Ie1ace158dac1733e6b2b2c1d533004d9bcab8c80 1.19 done, 1.20 pending submit, both not released yet. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Nemo federicol...@tiscali.it changed: What|Removed |Added CC||federicol...@tiscali.it Target Milestone|--- |1.20.x release Severity|normal |major --- Comment #11 from Nemo federicol...@tiscali.it --- - major and +milestone per http://lists.wikimedia.org/pipermail/wikitech-l/2013-February/067005.html , will set flag for backporting when available. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Nemo federicol...@tiscali.it changed: What|Removed |Added Keywords||code-update-regression -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Mark A. Hershberger m...@everybody.org changed: What|Removed |Added CC||m...@everybody.org -- You are receiving this mail because: You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Greg Grossmeier g...@wikimedia.org changed: What|Removed |Added CC||g...@wikimedia.org -- You are receiving this mail because: You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #10 from Bawolff (Brian Wolff) bawolff...@gmail.com --- (In reply to comment #4) Whether it changed or not, why should we hide notices about using the wrong calling conventions in PHP or some other similar type things? I'm just not sold on this, though it's probably not a big deal. I agree 100% that we should not be surpressing warnings about incorrect calling conventions. However there are other E_STRICT warnings that sometimes need to be surpressed (the timezone thing for example). The biggest issue with E_STRICT is that it throws a warning if you make a static call to a function which doesn't have the static keyword. However, in order to be compatible with PHP4, we need to omit it. From a core prespective, given how long ago PHP4 support has been dropped, that's not the most convincing argument. However functions should work as they are documented, and wfSuppressWarnings() is documented as surpressing all warnings. On a more general note, best practice is to disable this kind of error on production machines, however in PHP 5.4, the deliberate error suppression code is actually un-suppressing errors! That seems very broken to me. Didn't even realize that with the original bug, but yeah that's not good. (In reply to comment #7) Also, this should be back-ported to any previous MW versions that are still supported. Yeah probably, especially with it overriding the user's error_reporting settings to enable E_STRICT. /me wonders how to go about doing that/request that it be done in our new git world. Something for me to read up on. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #1 from Mark Clements (HappyDog) w...@kennel17.co.uk --- Note that this is only an issue on PHP 5.4 and above, where E_ALL now includes E_STRICT. In earlier versions of PHP, E_ALL excluded E_STRICT and so this did not cause any issues. In other words, this is a fix for a regression caused by changes in PHP 5.4. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Aaron Schulz aschulz4...@gmail.com changed: What|Removed |Added CC||aschulz4...@gmail.com --- Comment #2 from Aaron Schulz aschulz4...@gmail.com --- Why should it suppress them? Those kind of errors usually are problems with the PHP code and not with some random i/o type errors that you usually want to suppress. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #3 from Mark Clements (HappyDog) w...@kennel17.co.uk --- (In reply to comment #2) Why should it suppress them? A) Because that's what wfSuppressWarnings() is for - to suppress warnings! B) Because that's how the function works in all previous PHP versions. This is a regression. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #4 from Aaron Schulz aschulz4...@gmail.com --- Whether it changed or not, why should we hide notices about using the wrong calling conventions in PHP or some other similar type things? I'm just not sold on this, though it's probably not a big deal. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Jesús Martínez Novo (Ciencia Al Poder) martinezn...@gmail.com changed: What|Removed |Added CC||bawolff...@gmail.com, ||martinezn...@gmail.com --- Comment #5 from Jesús Martínez Novo (Ciencia Al Poder) martinezn...@gmail.com --- This has been already done on bug 43092, where Bawolff did gerrit change 38650 which kills E_STRICT on wfSuppressWarnings. (In reply to comment #4) Whether it changed or not, why should we hide notices about using the wrong calling conventions in PHP or some other similar type things? I'm just not sold on this, though it's probably not a big deal. Because those are for developers, not for production environments, and it has been proved that some E_STRICT warnings arise before LocalSettings.php is actually parsed, so they need to be supressed there for people that can't or don't want to change global configuration about error_reporting. I'm still not sure if I should close this bug as FIXED or as DUPLICATE of bug 43092 (and probably close bug 43092 also) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #6 from Mark Clements (HappyDog) w...@kennel17.co.uk --- Here's one concrete example, which is where I noticed the issue. I'm sure it is not the only situation that it will occur in. I have an extension (WikiDB) that is compatible with older versions of MediaWiki, including MW1.6 which runs on PHP4. I have existing users who still use PHP4 and for whatever reason are unwilling/unable to upgrade, and I still support those users, from a single code-base. The biggest issue with E_STRICT is that it throws a warning if you make a static call to a function which doesn't have the static keyword. However, in order to be compatible with PHP4, we need to omit it. There are no ill-effects by doing this in PHP5, except the annoying warnings. Users of WikiDB are instructed to disable E_STRICT via a call to error_reporting() in their LocalSettings.php, so the code works without these messages being output. However if there are any places in code which suppress warnings, E_STRICT is temporarily re-enabled. This manifests in PHP notices in the search box, for example. Even if the MW code-base is clean, this is something that is likely to occur in extensions, and if warnings are being suppressed, then they should be - well - suppressed. On a more general note, best practice is to disable this kind of error on production machines, however in PHP 5.4, the deliberate error suppression code is actually un-suppressing errors! That seems very broken to me. At the very least it should look at whether E_STRICT is currently being reported or not, and only omit it from the suppression if it is not already suppressed. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #7 from Mark Clements (HappyDog) w...@kennel17.co.uk --- Also, this should be back-ported to any previous MW versions that are still supported. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 --- Comment #8 from Aaron Schulz aschulz4...@gmail.com --- OK, re-enabling it temporarily is definitely bad behavior if it was off before. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Dereckson dereck...@espace-win.org changed: What|Removed |Added Status|NEW |RESOLVED CC||dereck...@espace-win.org Resolution|--- |WORKSFORME --- Comment #9 from Dereckson dereck...@espace-win.org --- Closing this bug as already resolved. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 43594] wfSuppressWarnings() should suppress E_STRICT warnings
https://bugzilla.wikimedia.org/show_bug.cgi?id=43594 Dereckson dereck...@espace-win.org changed: What|Removed |Added Resolution|WORKSFORME |FIXED -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l