remove test for page state saved for disabled plugin
authorJoey Hess <joey@kitenet.net>
Sat, 16 Nov 2013 16:43:46 +0000 (12:43 -0400)
committerJoey Hess <joey@kitenet.net>
Sat, 16 Nov 2013 16:43:46 +0000 (12:43 -0400)
My change did cause this state to be retained. I hope this is not a
problem.

Afaik, plugins test if they were disabled before by looking at the toplevel
plugin state, not the per-page plugin state. So the only remaining problem
might be

a) A plugin is disabled but its state keeps being saved. Which is not
   ideal, perhaps, but the large speedup of my optimisation seems worth it.
b) A plugin might have been enabled, be disabled, and get re-enabled, and
   see old state from before. I don't see how this would be different from
   the plugin seeing any other old state, though, so hopefully no breakage.

My optmisation looks a little more risky, but I still hope I can keep it.

t/index.t

index 392a167e9747131dbcd065b022a725ebf35f16d2..3f4b7d9b19a39286ddac71d8024efed8248b9037 100755 (executable)
--- a/t/index.t
+++ b/t/index.t
@@ -38,8 +38,6 @@ $depends{"bar"}={"foo*" => 1};
 $depends{"bar.png"}={};
 $pagestate{"bar"}{meta}{title}="a page about bar";
 $pagestate{"bar"}{meta}{moo}="mooooo";
-# only loaded plugins save state, so this should not be saved out
-$pagestate{"bar"}{nosuchplugin}{moo}="mooooo";
 
 ok(saveindex(), "save index");
 ok(-s "$config{wikistatedir}/indexdb", "index file created");