Opened on 10/04/2017 at 03:14:10 PM

Closed on 10/11/2017 at 09:23:02 AM

#5827 closed defect (fixed)

notification.json is empty if lastVersion contains a variant group

Reported by: wspee Assignee:
Priority: Unknown Milestone:
Module: Sitescripts Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29570562/

Description (last modified by wspee)

Environment

Filter server with:

  • Sitecripts: f85775239239
  • Infrastructure: 95ca18258f78

Chrome 61 ABP 1.13.4

How to reproduce

  1. Create and push the following notification to the filter server:
severity = information

[1]
title.en-US = Test
message.en-US = Test
sample = 1
  1. Start the browser and let ABP download the notification.
  2. Disable the previously created notification and add a new notification with the content from above.
  3. Let ABP download the notification again

Observed behaviour

The server delivers the following notification.json:

{
  "notifications": [],
  "version": "201710041503-4/1"
}

Expected behaviour

The server should deliver the following notification.json:

{
  "notifications": [
    {
      "id": "5",
      "message": {
        "en-US": "Test"
      },
      "severity": "information",
      "title": {
        "en-US": "Test"
      }
    }
  ],
  "version": "201710041504-4/1-5/1"
}

Note

To not break the AB machinery the server is supposed to deliver the same notification he delivered last time. But only if the same notification is still active and it shouldn't affect other notifications.

Attachments (0)

Change History (4)

comment:1 Changed on 10/06/2017 at 08:46:17 AM by wspee

  • Description modified (diff)

The old group should be remembered.

comment:2 Changed on 10/09/2017 at 09:12:28 AM by wspee

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:3 Changed on 10/11/2017 at 09:20:43 AM by abpbot

A commit referencing this issue has landed:
Issue 5827 - Assign new groups even if prior groups are present

comment:4 Changed on 10/11/2017 at 09:23:02 AM by wspee

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

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