eventum-auth-repliers.patch

   1 [16:07:39] < glen> bug
   2 [16:07:50] < glen> i can't add some low level user to authorized repliers list
   3 [16:08:05] < glen> but when it replies, i can convert note and choose add to recipient list... no error produced
   4 [16:08:12] < glen> altho it's not included into list eighter
   5 [16:08:57] < lordrashmi> low level = reporter?
   6 [16:09:03] < glen> yeah sth like that
   7 [16:09:23] < glen> but funny is, if he has no account in eventum, i would be allowed to add his email address
   8 [16:09:44] < glen> ie, i remember one issue where it worked earlier, that outside person was able to reply emails
   9 [16:12:02] < lordrashmi> glen: include/class.authorized_replier.php, line 196
  10 [16:12:21] < lordrashmi> change <= to ==
  11 [16:12:42] < glen> will this be in next eventum too?
  12 [16:12:49] < glen> or it will be my local fix only?
  13 [16:13:18] < lordrashmi> committing now
  14 [16:13:23] < glen> nice
  15 [16:13:29] < lordrashmi> be glad being in prague makes me in a good mood
  16
  17 --- ./include/class.authorized_replier.php~     2005-04-22 18:34:16.000000000 +0300
  18 +++ ./include/class.authorized_replier.php      2005-05-10 16:13:06.000000000 +0300
  19 @@ -193,7 +193,7 @@
  20      function addUser($issue_id, $usr_id, $add_history = true)
  21      {
  22          // only users above customer role can be authorized repliers. All customers should already be able to reply.
  23 -        if (User::getRoleByUser($usr_id, Issue::getProjectID($issue_id)) <= User::getRoleID("Customer")) {
  24 +        if (User::getRoleByUser($usr_id, Issue::getProjectID($issue_id)) == User::getRoleID("Customer")) {
  25              return -2;
  26          }
  27