Filter changed callback

src/FilterEngine.cpp

Index: src/FilterEngine.cpp
===================================================================
--- a/src/FilterEngine.cpp
+++ b/src/FilterEngine.cpp
@@ -277,3 +277,21 @@
   std::string error(params.size() >= 1 && !params[0]->IsNull() ? params[0]->AsString() : "");
   callback(error);
 }
+
+void FilterEngine::SetFilterChangeCallback(FilterEngine::FilterChangeCallback callback)
+{
+  jsEngine->SetEventCallback(eventName, std::tr1::bind(&FilterEngine::FilterChanged,
+      this, "filterChange", callback, std::tr1::placeholders::_1));

[Wladimir Palant, 2013/07/19 09:37:57]

Still not point passing "filterChange" to the callback, this parameter isn't
even used ;)

Also, does this even compile with the eventName variable being undefined? It
should be "filterChange" instead.

+}
+
+void FilterEngine::RemoveFilterChangeCallback()
+{
+  jsEngine->RemoveEventCallback(eventName);

[Wladimir Palant, 2013/07/19 09:37:57]

Does this even compile with the eventName variable being undefined? It should be
"filterChange" instead.

+}
+
+void FilterEngine::FilterChanged(const std::string& eventName, FilterEngine::FilterChangeCallback callback, JsValueList& params)
+{
+  std::string action(params.size() >= 1 && !params[0]->IsNull() ? params[0]->AsString() : "");
+  std::string url(params.size() >= 2 && !params[1]->IsNull() ? params[1]->AsString() : "");

[Wladimir Palant, 2013/07/19 09:37:57]

Please keep the second parameter generic, something like:

JsValuePtr item(params.size() >= 2 ? params[1] : jsEngine->NewValue(false));

The callback can then call new Filter(item) or new Subscription(item) if
necessary.

+  callback(action, url);
+}