Opened on 04/28/2017 at 08:11:56 AM

Last modified on 04/28/2017 at 02:37:56 PM

#5197 new change

Add FilterEngine::SetSubscriptionrChangeCallback

Reported by: sergz Assignee:
Priority: P4 Milestone:
Module: Libadblockplus Keywords:
Cc: asmirnov Blocked By: #5123, #5123, #5196
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):



Currently we receive all kind of notifications in FilterEngine::SetFilterChangeCallback as event (e.g. subscription.disabled) and JsValue&& object, however the user of libadblockplus for the above example should rather receive ("disabled", Subscription&&). Therefore we need to differentiate events from different sources, namely filter, subscription and others and pass a concrete object (Filter&& for filter and Subscription&& for subscription) instead of JsValue. So, since the signature for FilterEngine::SetFilterChangeCallback will be also changed it will force the users to adapt the code (if not now then later after not exposing JsValue from Filter), meanwhile to still provide them with the ability to get other events we need to add some way like FilterEngine::SetGenericEventCallback.

Attachments (0)

Change History (7)

comment:1 Changed on 04/28/2017 at 08:12:33 AM by sergz

  • Blocking 5123, 5196 added

comment:2 Changed on 04/28/2017 at 08:12:49 AM by sergz

  • Blocked By 5123, 5196 added
  • Blocking 5123, 5196 removed

comment:3 Changed on 04/28/2017 at 08:13:56 AM by sergz

  • Blocked By 5123, 5196 removed

comment:4 Changed on 04/28/2017 at 08:14:22 AM by sergz

  • Blocked By 5123, 5196 added

comment:5 Changed on 04/28/2017 at 08:14:48 AM by sergz

  • Blocked By 5123, 5196 removed

comment:6 Changed on 04/28/2017 at 08:17:25 AM by sergz

  • Blocked By 5123, 5196 added

comment:7 Changed on 04/28/2017 at 02:37:56 PM by abpbot

  • Blocked By

Add Comment

Modify Ticket

Change Properties
as new .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from (none).
Next status will be 'reviewing'.
Note: See TracTickets for help on using tickets.