Quellcode durchsuchen

Why did we have two eregi() functions in a row? I changed it so one regexp
should do it.

Tyler Akins vor 24 Jahren
Ursprung
Commit
1c6da80f2f
1 geänderte Dateien mit 8 neuen und 6 gelöschten Zeilen
  1. 8 6
      functions/imap_mailbox.php

+ 8 - 6
functions/imap_mailbox.php

@@ -258,15 +258,17 @@
 
 
          // Then list special folders and their subfolders
          // Then list special folders and their subfolders
          for ($i = 0 ; $i <= count($boxes) ; $i++) {
          for ($i = 0 ; $i <= count($boxes) ; $i++) {
-            if((eregi("^".quotemeta($trash_folder).'$', $boxes[$i]["unformatted"]) ||
-                eregi("^".quotemeta($trash_folder).quotemeta($dm), $boxes[$i]["unformatted"]) )  &&
-               ($move_to_trash)) {        
+	    if ($move_to_trash && 
+	        eregi("^" . quotemeta($trash_folder) . "(" . 
+		    quotemeta($dm) . ")?$", $boxes[$i]["unformatted"]))
+	    {
                $boxesnew[] = $boxes[$i];
                $boxesnew[] = $boxes[$i];
                $boxes[$i]["used"] = true;
                $boxes[$i]["used"] = true;
             }
             }
-            else if((eregi("^".quotemeta($sent_folder).'$', $boxes[$i]["unformatted"]) ||
-                     eregi("^".quotemeta($sent_folder).quotemeta($dm), $boxes[$i]["unformatted"]) )  &&
-                    ($move_to_sent)) {        
+            elseif ($move_to_sent &&
+	        eregi("^" . quotemeta($sent_folder) . "(" .
+		    quotemeta($dm) . ")?$", $boxes[$i]["unformatted"]))
+	    {
                $boxesnew[] = $boxes[$i];
                $boxesnew[] = $boxes[$i];
                $boxes[$i]["used"] = true;
                $boxes[$i]["used"] = true;
             }
             }