22

This edit is old: https://stackoverflow.com/review/suggested-edits/8631577.

But I do not know enough about PHP to determine if this correction to the question was welcomed or not. To me, it seems that the edit may have solve the problem that was asked, and as such has invalidated the question.

Can someone knowledgeable about PHP double-check if it should be rolled back or not?

1
  • 2
    You really don't need PHP knowledge for this, such edits should always be rolled back.
    – user247702
    May 3, 2018 at 7:47

2 Answers 2

21

Having a typo in upload_max_filesize could definitely be the source of the issue. PHP does not raise any warning if an extra parameter is added to php.ini, and it has defaults for all (most?) settings, so that would be easy to miss.

I'm not sure how to handle it the best according to Stack Overflow guidelines. I guess reverting the edit, and posting an answer pointing out the typo would do.

8
  • 4
    Typos are non on-Topic on SO, the question is closed accordingly now.
    – Polygnome
    May 2, 2018 at 10:43
  • 3
    i am knowledgeable about PHP. this is the correct answer.
    – hanshenrik
    May 2, 2018 at 10:53
  • 11
    Polygnome, closing the question is good, but pointing out the typo would be best for users consulting the thread, if that ever happen. I will post a comment over there. May 2, 2018 at 11:38
  • @romaricdrigon nicely done.
    – CPHPython
    May 3, 2018 at 15:24
  • 1
    @romaricdrigon, you're absolutely right. Also closing this is probably one reason why new users specifically think SO is unfriendly. On that note we've all had times where one little misspelling screws us over. having a reference for those occasions is pretty handy, and IMO what makes SO so valuable as a resource. May 3, 2018 at 18:10
  • 1
    @Polygnome for completeness' sake: it is on topic! Most of our questions are about debugging, and debugging includes spotting that one silly typo. It's just that these questions are unlikely to help future readers; just read the close banner! If there's a typo that is useful for future readers, there's no need to close it in my opinion. May 3, 2018 at 19:30
  • 1
    @AndrasDeak 99% of the typos that land here of off-topic (so in general, my statement holds). The close reason pretty much covers it.
    – Polygnome
    May 3, 2018 at 19:35
  • @Polygnome The word you're looking for is off-topic. May 4, 2018 at 12:50
18

Maybe. It's unclear whether that was the issue or just a bad copy and paste. It certainly was bad to correct such a technical detail which may have been the issue, so I have rolled it back to the original.

9
  • 16
    Those users accepting this edit should be review banned. This is editing 101: never ever change code in questions! May 2, 2018 at 9:49
  • 3
    @PatrickHofman We shouldn't review-ban for very old reviews. But a warning or reminder may do.
    – Cœur
    May 2, 2018 at 10:10
  • 22
    @PatrickHofman "never ever" is too strong. I frequently change code to fix indentation (except in Python) and to shorten long lines. The Python issue means that you can't just say "never ever change non-whitespace in code". You are left with "never ever make non-cosmetic changes in code" - but "fixing a typo" can sometimes be a cosmetic change, and sometimes significant. Reviewers need to "skip" if they are not sure. May 2, 2018 at 10:26
  • 2
    It's, like, you're encountering a phrase in Chinese and you think it's smart to run a spell-check on it – even tho' you don't read Chinese.
    – Jongware
    May 2, 2018 at 10:32
  • 4
    @usr2564301 when people's code is comparable to chinese for a foreigner, it's safe to assume OP doesn't understand their code either. May 2, 2018 at 13:27
  • 1
    @MartinBonner I would agree with you if this was an answer, but changing question code has the possibility of fixing the issue. Answers are more likely to have innocent typos that can be fixed no problem (if your reviewers accept it...)
    – Clint
    May 3, 2018 at 15:31
  • @Clint Adding line wrap and indentation in C++ code will never fix the issuer. Also, I don't have to worry about my reviewers accepting the change... May 3, 2018 at 15:37
  • @MartinBonner Right, I wasn't trying to suggest that formatting edits are bad on questions. I wasn't clear but I was focusing on the fixing typos bit.
    – Clint
    May 3, 2018 at 15:43
  • 2

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .