Code review comment for lp:~googol-deactivatedaccount/openlp/bug-900399

Revision history for this message
Tim Bentley (trb143) wrote :

131 is an incorrect removal as when a plugin is disabled the settings dialog will fail as the value ins not defined.
Renaming fields is also iffy in a feature freeze. They are not a bug.
Code simplification is just Ok.

review: Needs Fixing

« Back to merge proposal