MB will nicely print the banner that says "The data you have submitted does not make any changes to the data already present", which is good, and also give an ISE under it, which is not so good.
Description
MB will nicely print the banner that says "The data you have submitted does not make any changes to the data already present", which is good, and also give an ISE under it, which is not so good.
Not sure how much work I'll be able to do here, so I'll log what I discover to comments:
line 143 of MB::S::Controller calls $opts{post_creation} unconditionally; it should probably only do this if $edit is defined, since Controller::Artist then tries to call ->id on the value it gets in post_creation (which is the actual ISE that happens)
I'm not sure what happens after (yet), it's possible there's other ISEs once that's fixed
Ian McEwen added a comment - 16/Nov/12 12:45 PM Not sure how much work I'll be able to do here, so I'll log what I discover to comments:
line 143 of MB::S::Controller calls $opts{post_creation} unconditionally; it should probably only do this if $edit is defined, since Controller::Artist then tries to call ->id on the value it gets in post_creation (which is the actual ISE that happens)
I'm not sure what happens after (yet), it's possible there's other ISEs once that's fixed
Bug
Closed
Major
ISE when trying to submit an "Edit artist" edit with no changes
Not sure how much work I'll be able to do here, so I'll log what I discover to comments: