Page MenuHomePhabricator

page-linked notifications seems to be broken
Closed, ResolvedPublic


Steps to reproduce.

  1. Create Foo page with page-linked notifs enabled.
  2. Link to [[Foo]] with another account.

Actual: No page-linked notifs seen.
Expected: It should be shown.

Looks like the event does get inserted into the table but everything seems to be null:

            event_id: 49
          event_type: page-linked
       event_variant: NULL
      event_agent_id: NULL
      event_agent_ip: NULL
event_page_namespace: NULL
    event_page_title: NULL
         event_extra: a:1:{s:17:"link-from-page-id";i:184;}
       event_page_id: 183
1 row in set (0.00 sec)

Event Timeline

Glaisher created this task.Dec 17 2015, 5:43 PM
Glaisher raised the priority of this task from to Needs Triage.
Glaisher updated the task description. (Show Details)
Glaisher added a project: Notifications.
Glaisher added subscribers: Glaisher, MtDu.
Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptDec 17 2015, 5:43 PM
Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald Transcript
Glaisher updated the task description. (Show Details)Dec 17 2015, 5:44 PM
Glaisher set Security to None.
Legoktm claimed this task.Dec 17 2015, 6:21 PM

Change 259774 had a related patch set uploaded (by Legoktm):
RefreshLinksJob: Restore LinksUpdate::setTriggeringUser() call

Change 259774 merged by jenkins-bot:
RefreshLinksJob: Restore LinksUpdate::setTriggeringUser() call

Krinkle closed this task as Resolved.Dec 23 2015, 1:33 AM
Krinkle added a subscriber: Krinkle.