First run page triggering

src/plugin/PluginUserSettings.cpp

Index: src/plugin/PluginUserSettings.cpp
===================================================================
--- a/src/plugin/PluginUserSettings.cpp
+++ b/src/plugin/PluginUserSettings.cpp
@@ -97,16 +97,16 @@
 }
 
 
-CStringW sGetMessage(const CString& key)
+CStringW sGetMessage(const CString& section, const CString& key)
 {
   Dictionary* dictionary = Dictionary::GetInstance();
-  return CString(dictionary->Lookup("settings", std::string(CW2A(key))).c_str());
+  return CString(dictionary->Lookup(std::string(CW2A(section)), std::string(CW2A(key))).c_str());

[Wladimir Palant, 2013/07/22 06:11:44]

Since you didn't address that comment in
http://codereview.adblockplus.org/11013110/ - how about changing CString into
CStringW here to match the return type?

 }
 
-std::wstring sGetMessage(const std::string& key)
+std::wstring sGetMessage(const std::string& section, const std::string& key)
 {
   Dictionary* dictionary = Dictionary::GetInstance();
-  return dictionary->Lookup("settings", key);
+  return dictionary->Lookup(section, key);
 }
 
 
@@ -131,7 +131,7 @@
 
   if (s_GetMessage == method)
   {
-    if (1 != pDispparams->cArgs)
+    if (2 != pDispparams->cArgs)
       return DISP_E_BADPARAMCOUNT;
 
     if (VT_BSTR != pDispparams->rgvarg[0].vt)
@@ -140,7 +140,8 @@
     if (pVarResult)
     {
       CComBSTR key = pDispparams->rgvarg[0].bstrVal;
-      CStringW message = sGetMessage((BSTR)key);
+      CComBSTR section = pDispparams->rgvarg[1].bstrVal;
+      CStringW message = sGetMessage((BSTR)section, (BSTR)key);
 
       pVarResult->vt = VT_BSTR;
       pVarResult->bstrVal = SysAllocString(message);