Comment 2 for bug 522599

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

This fix is bad. We need to land a correct fix very quickly to prevent the last attempted fix from introducing bad data. The slave was deleted in my QA on edge, but we wanted to delete the master. ie. https://bugs.edge.launchpad.net/gdp/trunk/+bug/420173 shows a master bug without a slave. The master is targeted to a series, which make no sense since there is no bug in the project. This may require SQL to fix.

This bad chunk:
            if bugtask.conjoined_master is not None:
                Store.of(bugtask).remove(bugtask)
 should be:
            if bugtask.conjoined_master is not None:
                Store.of(bugtask.conjoined_master).remove(bugtask)