Issue 1103 - Migrate Simple Adblock users

src/plugin/PluginClass.cpp

Index: src/plugin/PluginClass.cpp
===================================================================
--- a/src/plugin/PluginClass.cpp
+++ b/src/plugin/PluginClass.cpp
@@ -854,9 +854,14 @@
   {
     return false;
   }
-  
   s_criticalSectionLocal.Lock();
-  int versionCompRes = CPluginClient::GetInstance()->CompareVersions(CPluginClient::GetInstance()->GetPref(L"currentVersion", L"0.0"), L"1.2");
+  std::wstring curVer = CPluginClient::GetInstance()->GetPref(std::wstring(L"currentVersion"), std::wstring(L"0.0"));

[Eric, 2015/05/17 00:29:25]

Don't need explicit conversions to 'std::wstring'

+  // The plugin version can be "simpleadblock", which doesn't have the "."
+  if (curVer.find(L".") == std::wstring::npos)
+  {
+    CPluginClient::GetInstance()->SetPref(L"convertedFrom", curVer);
+  }
+  int versionCompRes = CPluginClient::GetInstance()->CompareVersions(curVer, L"1.2");
 
   bool isFirstRun = CPluginClient::GetInstance()->IsFirstRun();
   CPluginClient::GetInstance()->SetPref(L"currentVersion", std::wstring(IEPLUGIN_VERSION));
@@ -866,7 +871,10 @@
   {   
     if (!isFirstRun)
     {
-      CPluginClient::GetInstance()->SetPref(L"displayUpdatePage", true);
+      if (!CPluginClient::GetInstance()->GetPref(curVer, false))

[Eric, 2015/05/17 00:29:25]

Is the value of 'curVer' the name of a preference?

+      {
+        CPluginClient::GetInstance()->SetPref(L"displayUpdatePage", true);
+      }
     }
 
     // IE6 can't be accessed from another thread, execute in current thread