Comment 1 for bug 697294

Revision history for this message
Curtis Hovey (sinzui) wrote :

The user was trying to set the package to cups to classify the solution picked by the user. This is a legitimate case. The form does not permit the user to change status...that is a separate form with sane controls on it.

This question is open, but I also see a star on a question indicating it was solved. I see from the question history that the asker reopened the question. There are *no* answers after the reopening. Only the asker is providing more information.

Module lp.answers.browser.question, line 747, in change_action
self.updateContextFromData(data)
Module lp.app.browser.launchpadform, line 427, in updateContextFromData
context, context_before_modification, field_names))
...
Module lp.answers.model.questionreopening, line 67, in create_questionreopening
"Open question shouldn't have an answerer.")
AssertionError: Open question shouldn't have an answerer.

The code took an unexpected path after updating the question. create_questionreopening is subscribed to objectEventNotify. The method did work when the question was reopened. It will work again once a user provides an answer. Since the status has not changed, I expect create_questionreopening to return early if there is no status change.

maybe
    if old_question.status == question.status
        return