Don't run filename patterns through preg_quote twice

This would result in extra slashes, e.g.
    '/\/([0-9]+?)\s\\\-\s(.+?)\..+$/' instead of
    '/\/([0-9]+?)\s\-\s(.+?)\..+$/'

There are some unrelated cosmetic changes as well.

docs/CHANGELOG

@@ -4,6 +4,8 @@
 
 --------------------------------------------------------------------------
   v.3.6-Alpha2
+        - Fixed an issue with filename pattern matching when patterns contained
+                characters that are part of regex syntax (such as -)
         - Fixed display of logic operator in rules (reported by Twister)
         - Fixed newsearch issue preventing use of more than 9 rules 
                 (reported by Twister)

lib/class/vainfo.class.php

@@ -941,26 +941,33 @@ class vainfo {
 			$slash_type = '\\';
 		}
 
+		// Combine the patterns
 		$pattern = preg_quote($this->_dir_pattern) . $slash_type . preg_quote($this->_file_pattern);
-		preg_match_all("/\%\w/",$pattern,$elements);
 		
-		$preg_pattern = preg_quote($pattern);
-		$preg_pattern = preg_replace("/\%[Ty]/","([0-9]+?)",$preg_pattern);
-		$preg_pattern = preg_replace("/\%\w/","(.+?)",$preg_pattern);
-		$preg_pattern = str_replace("/","\/",$preg_pattern);
-		$preg_pattern = str_replace(" ","\s",$preg_pattern);
-		$preg_pattern = "/" . $preg_pattern . "\..+$/";
-		preg_match($preg_pattern,$filename,$matches);
-		/* Cut out the Full line, we don't need that */
-		array_shift($matches);
+		// Pull out the pattern codes into an array
+		preg_match_all('/\%\w/', $pattern, $elements);
 
-		/* Foreach through what we've found */
-		foreach ($matches as $key=>$value) {
+		// Mangle the pattern by turning the codes into regex captures
+		$pattern = preg_replace('/\%[Ty]/', '([0-9]+?)', $pattern);
+		$pattern = preg_replace('/\%\w/', '(.+?)', $pattern);
+		$pattern = str_replace('/', '\/', $pattern);
+		$pattern = str_replace(' ', '\s', $pattern);
+		$pattern = '/' . $pattern . '\..+$/';
+
+		// Pull out our actual matches
+		preg_match($pattern, $filename, $matches);
+
+		// The first element is the full match text
+		$matched = array_shift($matches);
+		debug_event('vainfo', $pattern . ' matched ' . $matched . ' on ' . $filename, 5);
+
+		// Iterate over what we found
+		foreach ($matches as $key => $value) {
 			$new_key = translate_pattern_code($elements['0'][$key]);
 			if ($new_key) {
 				$results[$new_key] = $value;
 			}
-		} // end foreach matches
+		}
 
 		$results['size'] = filesize($filename);