Uploaded image for project: 'phpBB3'
  1. phpBB3
  2. PHPBB3-12742

Notifications duplicate code that requires an event

    XMLWordPrintable

    Details

      Description

      		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

        Attachments

          Activity

            People

            Assignee:
            Marc Marc
            Reporter:
            nickvergessen Joas Schilling [X] (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: