14

I am surprised to find the following bug in edit system:

Follow the steps bellow:

  1. A user with reputation less than 2000 points edits a post.
  2. Instead of a edit summary message he insert this: " " (one space) and he receive a message that he has to enter a 10 chars longer message. He insert more than ten spaces inside of Edit Summary textbox, and click Save Edits button.
  3. When a user with reputation higher than 2000 reviews this edit the result is this:

enter image description here

Do you see? No edit summary at above screen shot.

For another example of the issue, take a look at rev 4 in the history of this very question.

Instead of using if (... .length > 10), shouldn't be used if (... .trim().length > 10)?

Improve this question
9
  • SE is build on top of .NET/C#...so those function and property names should be UpperCamelCase (.Trim())...just saying...
    – Time Traveling Bobby
    Feb 15, 2013 at 13:38
  • 1
    @SulfurizedDemonbobby, the important thing is the idea. I wrote in Javascript because I think that there is a client side.
    – Ionică Bizău
    Feb 15, 2013 at 13:40
  • @John good idea just did it with 10 spaces, take a look at update in rev 4 :)
    – gnat
    Feb 15, 2013 at 13:48
  • @gnat, You've just edited my question with no edit summary message. :-)
    – Ionică Bizău
    Feb 15, 2013 at 13:48
  • @gnat since you've got the rep to edit, you're not required to enter a summary anyway.
    – yoozer8
    Feb 15, 2013 at 14:01
  • 1
    Although if you leave it blank, it autofills something like "added X charcters in body" or "edited tags" or similar. So a whitespace summary should act the same
    – yoozer8
    Feb 15, 2013 at 14:02
  • 1
    @Jim, if you don't insert any Edit Summary message the default message looks like this "deleted 20 chars in body" or "edited the title" or similar.
    – Ionică Bizău
    Feb 15, 2013 at 14:02
  • This is my first question on Meta. When do I know that bug was fixed? I wait for an official answer.
    – Ionică Bizău
    Feb 26, 2013 at 13:31
  • @SulfurizedDemonbobby, And YES. The code is written in Javascript. Search "Your edit" here.
    – Ionică Bizău
    Mar 7, 2013 at 13:06

2 Answers 2

Reset to default
10

If you don't leave an edit summary, you're just disrespecting the folks who'll be reviewing that edit - this amounts to shooting yourself in the foot, since if they don't understand why you're editing, they're more likely to reject the edit.

As Linuxios notes, it's not like we can force you to leave a meaningful edit summary; the length restriction is just encouragement.

Improve this answer
0
4

I don't think this is really a bug. Considering that I could just type ssadfafasdfsedf for my edit summary, anyone who doesn't want to give one will find a way to be able to. The length thing is mostly just to encourage giving an edit summary. Anyway, some edits are minor enough that they don't even need a summary anyway.

Improve this answer
2
  • @Linuxious Your answer is correct seeing from this side. But then why did they put a chars limit? If they did this, I think that this would be nice to be fixed.
    – Ionică Bizău
    Feb 15, 2013 at 14:34
  • @john: I'd say as more of a suggestion to try to make people provide a description. But if someone is determined to do something enough, they can usually bypass the restriction.
    – Linuxios
    Feb 15, 2013 at 14:37

You must log in to answer this question.

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