Timeline for Edit deleting incorrect code was rejected. Why?
Current License: CC BY-SA 3.0
24 events
when toggle format | what | by | license | comment | |
---|---|---|---|---|---|
May 8, 2014 at 21:39 | comment | added | Andrew Savinykh |
What could have happened is that your edit was ok. It's just the people who rejected it were incompetent in this particular area so they would not know that the code you had deleted is incorrect. Thus, they saw that you removed a "useful" example and rejected the edit. Of course substituting with new DaoSession(/* args */); as noted elsewhere would be even better edit on your part.
|
|
May 8, 2014 at 17:03 | comment | added | Izkata |
If you're not sure of the arguments or don't want to spend time looking them up, something I've occasionally done something like new DaoSession(/* args */); , if I knew args were necessary
|
|
May 8, 2014 at 16:36 | answer | added | Brad Koch | timeline score: 3 | |
May 8, 2014 at 9:19 | comment | added | Richard Le Mesurier | @Manu fair enough - thx | |
May 8, 2014 at 9:14 | comment | added | Manuel Allenspach | @RichardLeMesurier I took it directly from the documentation, not from AlexS. | |
May 8, 2014 at 9:12 | history | edited | AlexS | CC BY-SA 3.0 |
Deleted irrelevant information for the discussion.
|
May 8, 2014 at 8:58 | comment | added | Richard Le Mesurier | @Manu thx for fixing broken code. In all fairness, I believe you should reference AlexS for providing the correct code. | |
May 8, 2014 at 8:40 | comment | added | Manuel Allenspach | @abhitalks OK, will do :) | |
May 8, 2014 at 8:36 | comment | added | Abhitalks | @Manu: Purely as an aside, it would have been better if you could improve your answer after having been pointed out in the comments. As I see, you haven't yet rectified the mistake, but continue arguing in the comments. The answer was accepted, means it solved the Op's problem, but the incorrect code fragment can very easily mislead others and in general is not good. Others editing your code is bad, but you editing your own code is good. | |
May 8, 2014 at 8:32 | comment | added | Manuel Allenspach | @RichardLeMesurier I'm of course trying to always post correct code, but in this case it was a simple NullPointerException which the asker did not found. Showing him where the root of the problem is will be enough. | |
May 8, 2014 at 8:30 | comment | added | Richard Le Mesurier | @Manu Are you advocating posting answers with incorrect code? Surely the task of the answerer is exactly to find a solution, otherwise its not really an answer? Or perhaps I'm doing this all wrong | |
May 8, 2014 at 8:28 | comment | added | Manuel Allenspach | OP of answer here. I knew the constructor may not exist, but IMO it's not the task of the answerer to find a solution to this. I showed where the problem is, but the owner of the question can solve it himself. | |
May 8, 2014 at 7:01 | history | edited | AlexS | CC BY-SA 3.0 |
More information to avoid unneccesary comments and the actual status of the issue itself. (Format)
|
May 7, 2014 at 21:41 | comment | added | Harry Johnston | Generally speaking, an edit by anyone but the original author should not change the meaning of the post, even if the original author was flat-out wrong. | |
May 7, 2014 at 13:47 | vote | accept | AlexS | ||
May 7, 2014 at 13:41 | answer | added | Alexey Malev | timeline score: 22 | |
May 7, 2014 at 13:32 | answer | added | Roy Dictus | timeline score: 16 | |
May 7, 2014 at 13:32 | comment | added | rene | You didn't correct a mistake, you removed the essential part of the answer IMHO. If I'm not familiar with how to initialize a variable with your edit I've no idea. A better edit had been to add a constructor that does exist (as Oded pointed out). And my comment is not only meant for you but also for other visitors of this question. | |
May 7, 2014 at 13:30 | comment | added | Roy Dictus | @AlexS: Because (1) the edited answer is not really an improvement, (2) the change is minor, and (3) the answer was already accepted. Those three add up. | |
May 7, 2014 at 13:21 | comment | added | AlexS | @rene Why should I leave a comment instead of correcting a mistake, if I know 100% I found a mistake? | |
May 7, 2014 at 12:19 | comment | added | AlexS | @Oded I also posted an additional answer where I explained how to initialize, what one has to keep in mind greendao and where to go for further research. I thought this was to much for an edit. But I didn#t want to let wrong code inside of an accepted answer as it will confuse people. | |
May 7, 2014 at 10:43 | comment | added | rene | Be very reluctant when editing code, especially in accepted answers. Better leave a comment to point out what is wrong/confusing. | |
May 7, 2014 at 10:32 | comment | added | Oded StaffMod | Wouldn't a better edit have been to add an existing overload, instead of removing code that shows the concept (even if slightly wrong)? | |
May 7, 2014 at 10:21 | history | asked | AlexS | CC BY-SA 3.0 |