3

I know this question has been asked on Meta a number of times before, but I do not understand why reviewers rejected my edit with the response "this edit was intended to address the author of the post" when that was not the case. I tried to follow the guidelines as closely as possible, and correcting outdated information is definitely what edits are allowed to do.

Did I overlook something and were the reviewers right in rejecting?

20
  • The fact that GetPartsOfType is deprecated and that the indentation is broken. That is a part of the question. by fixing that, you are changing the meaning of the question, no?
    – 10 Rep
    Apr 15, 2021 at 18:33
  • 2
    How is code indentation and GetPartsOfType part of the question? I edited an answer, and the question did not mention GetPartsOfType.
    – asdf101
    Apr 15, 2021 at 18:34
  • 1
    If you left the existing contents and edited in the new version it would have been accepted most likely.
    – Warcupine
    Apr 15, 2021 at 18:35
  • @Warcupine yes that makes sense, was fixing the indentation still right then?
    – asdf101
    Apr 15, 2021 at 18:37
  • Probably okay to fix indention assuming it doesn't change anything in the execution, I don't know if that is the case in C# though.
    – Warcupine
    Apr 15, 2021 at 18:38
  • It doesn't. I'll propose a new edit with correcting the author's whitespace, and add the new implementation below it
    – asdf101
    Apr 15, 2021 at 18:44
  • 1
    Write your own answer with the new syntax instead.
    – Tom
    Apr 15, 2021 at 18:44
  • 2
    @Tom I feel like that would be somewhat unfair, since the author's question is what helped me in the first place. The method is exactly the same, only the way in which ImageParts are retrieved isn't
    – asdf101
    Apr 15, 2021 at 18:46
  • In that case it might be better to just leave the comment I'd already posted, right?
    – asdf101
    Apr 15, 2021 at 18:48
  • @asdf101 - it is likely that your edit will be rejected again (I am sure I would've rejected it if I came across it). Editors who do not yet have a full edit privilege should not change the answers of others (I mean, semantically). The best thing you can do is to ping the author and ask them to make the edit. If they are unresponsive in a reasonable timeframe, consider posting an answer as suggested by yivi with proper credit. Make it a community wiki if you still feel bad about it (but you should not - SO is a collaborative place, not a competition) Apr 15, 2021 at 18:48
  • @OlegValter is that an unwritten rule? Because I did not find that anywhere in the guidelines. The author hasn't posted since september.
    – asdf101
    Apr 15, 2021 at 18:50
  • 1
    @asdf101 - this is a judgement call (code edits) and a common courtesy (asking the author -> posting an answer with proper credit). There is no explicit "do not do ever" - this depends on the change. In your case you made the answer something it was not and this lead to rejection as "intended to address the author of the post" (btw, this is an canned response from an option we select when reviewing suggested edits - so this is one of the official reasons to reject an edit as well. Apr 15, 2021 at 19:09
  • I'd say the comment you left is just fine - despite the OP not being active posted, last time they logged in was 7 hours ago, so pinging them (and preferably addressing them so as they know you want the info to be included in the post) most likely won't go to waste. Apr 15, 2021 at 19:11
  • I personally would not have approved of an edit that simply fixed the indention of the code since whitespace doesn't matter to the C# compiler. I mean you also changed a var to a string without any reason. You also can't change imageParts without also changing sourceSlidePart more than likely. The most important fact is that the syntax used in the answer is still valid. I would have rejected your edit for thos reasons Apr 15, 2021 at 21:17
  • 1
    @Braiam var in C# just means "figure out the type for me at compile time", not "this variable is of a variant type"; the compiler won't let you change the type of variable once declared. So, there is no reason to change var to string. Some of us use var all the time so that finding variable names is easier; my eyes don't have to zig-zag left and right for different types; all of the variable names are four characters in from the indent level. Apr 16, 2021 at 17:09

1 Answer 1

9

Your edit, while well intentioned, removed information form the platform.

It's very well that a new syntax exists, but some other users may still benefit from the old syntax. We all have had to work with not fully updated technology now and then.

Instead of editing the existing answer, post your own, suggesting the new syntax. Credit the old answer if the new one you would post benefits clearly from the other one. Giving credit where it's due is expected in the platform.

As a general rule, editing code in answers is fine if you are correcting an obvious mistake. To provide updated information, I believe posting a new answer (specifying the circumstances under which the new information applies, linking the appropriate documentation if possible, etc) is a better solution.

10
  • Thank you for your answer. I think leaving the comment I already had posted is best, since it already contains all information future OpenXml users should need. The documentation is itself out of date unfortunately, the only way I knew GetPartOfType was outdated was because Visual Studio warned me.
    – asdf101
    Apr 15, 2021 at 19:07
  • "removed information form the platform" [citation needed] Nothing is ever removed from the site. The revision history is literally public on non-deleted post.
    – Braiam
    Apr 15, 2021 at 19:16
  • 2
    Wow. I didn’t know. That sounds like great feature. Next time I read an answer I’ll make sure to read the whole edit history. Maybe a previous version is completely different and more useful. Thanks for the tip.
    – yivi
    Apr 15, 2021 at 19:33
  • 2
    @Braiam It makes that information inaccessible to anyone who doesn't check the revision history - even if you know that's an option, which I suspect most visitors to the site probably don't, you have no way of knowing that there's a different version of the code in it. Or do you expect users to check the full revision history of every single answer they read? Apr 15, 2021 at 19:33
  • @JohnMontgomery and who cares about that? You, me or the one that actually is looking for an up to date solution to their problem? Who is more important of those? Hint: the last ;)
    – Braiam
    Apr 15, 2021 at 21:07
  • @Braiam And I guess the people who need the older solution don't matter at all? Apr 15, 2021 at 21:42
  • @JohnMontgomery and yet, they are 0.000000001%. Should the rest just be kept back just because some don't want to move forward?
    – Braiam
    Apr 16, 2021 at 0:43
  • 2
    @Braiam No, they could read the new answer that yivi suggests the OP create... :/ Apr 16, 2021 at 17:12
  • 1
    @Braiam That's a strawman, nobody is suggesting keeping the rest back. The suggestion is to have both solutions available. Apr 16, 2021 at 18:07
  • @HereticMonkey except that OP will not create it. For OP, the best solution is to make the answer better, not just duplicate it.
    – Braiam
    Apr 17, 2021 at 12:13

You must log in to answer this question.

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