Changeset 1447 for branches/eraser6/6.0/Eraser/SettingsPanel.cs

Timestamp:

1/4/2010 10:20:21 AM (3 years ago)

Author:

lowjoel

Message:

Do not explicitly allow or disallow a plugin based on whether the plugin is checked in the plugin settings list view. This is because signed plugins load automatically; if the signature changes or if root certificates change the plugin may then be a valid signed assembly. This also fixes the "double error message on startup" where users are greeted with the "invalid unused space and file erasure methods as well as PRNG" will be given the "plugins will only be loaded on restart" message (which makes no sense to the user)

Location:

branches/eraser6/6.0

Files:

• . (modified) (1 prop)
• Eraser/SettingsPanel.cs (modified) (1 diff)

branches/eraser6/6.0/Eraser/SettingsPanel.cs

@@ -353 +353 @@
                 if (!pluginApprovals.ContainsKey(guid))
                 {
-                    pluginApprovals.Add(guid, item.Checked);
-                    pluginApprovalsChanged = true;
+                    if (plugin.Plugin.Loaded != item.Checked)
+                    {
+                        pluginApprovals.Add(guid, item.Checked);
+                        pluginApprovalsChanged = true;
+                    }
                 }
                 else if (pluginApprovals[guid] != item.Checked)