Igor Gariev (gariev) wrote in changelog,
Igor Gariev
gariev
changelog

[livejournal] r19181: LJSUP-8875: No notifications in Inbox an...

Committer: gariev
LJSUP-8875: No notifications in Inbox and LJ Alert
Adding generic support for custom filters in finding subscriptions
U   trunk/cgi-bin/LJ/Event.pm
Modified: trunk/cgi-bin/LJ/Event.pm
===================================================================
--- trunk/cgi-bin/LJ/Event.pm	2011-05-31 09:01:12 UTC (rev 19180)
+++ trunk/cgi-bin/LJ/Event.pm	2011-05-31 09:01:21 UTC (rev 19181)
@@ -598,13 +598,16 @@
         my $journal_match = $allmatch ? "" : "AND journalid=?";
 
         my $limit_sql = ($limit && $limit_remain) ? "LIMIT $limit_remain" : '';
+        my ($extra_condition, @extra_args) = $self->extra_params_for_finding_subscritions();
         my $sql = "SELECT userid, subid, is_dirty, journalid, etypeid, " .
             "arg1, arg2, ntypeid, createtime, expiretime, flags  " .
-            "FROM subs WHERE etypeid=? $journal_match $and_enabled $limit_sql";
+            "FROM subs WHERE etypeid=? $journal_match $and_enabled $extra_condition " .
+            $limit_sql;
 
         my $sth = $udbh->prepare($sql);
         my @args = ($self->etypeid);
         push @args, $self->u->id unless $allmatch;
+        push @args, @extra_args;
         $sth->execute(@args);
         if ($sth->err) {
             warn "SQL: [$sql], args=[@args]\n";
@@ -650,6 +653,10 @@
     return @subs;
 }
 
+sub extra_params_for_finding_subscritions {
+    return '';
+}
+
 # returns a boolean value indicating whether the given subscription matches
 # the event.
 #

Tags: gariev, livejournal, pm
Subscribe
  • Post a new comment

    Error

    Anonymous comments are disabled in this journal

    default userpic

    Your reply will be screened

    Your IP address will be recorded 

  • 0 comments