Uploaded image for project: 'phpBB'
  1. phpBB
  2. PHPBB-12742

Notifications duplicate code that requires an event

XMLWordPrintable

      		if (empty($users))
      		{
      			return array();
      		}
      		sort($users);
       
      		$auth_read = $this->auth->acl_get_list($users, 'f_read', $post['forum_id']);
       
      		if (empty($auth_read))
      		{
      			return array();
      		}
       
      		$notify_users = $this->check_user_notification_options($auth_read[$post['forum_id']]['f_read'], $options);

      duplicates over almost all the files.
      it should be moved to a sub method in the base class, so we can add an event there instead of having multiple events

            Marc Marc
            nickvergessen Joas Schilling
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: