xpressengine / xe-core

classes/security/htmlpurifier/library/HTMLPurifier/AttrDef/CSS/AlphaValue.php

Indentation +11 added lines, -11 removed lines

@@ -3,18 +3,18 @@
 class HTMLPurifier_AttrDef_CSS_AlphaValue extends HTMLPurifier_AttrDef_CSS_Number
 {
 
-    public function __construct() {
-        parent::__construct(false); // opacity is non-negative, but we will clamp it
-    }
+	public function __construct() {
+		parent::__construct(false); // opacity is non-negative, but we will clamp it
+	}
 
-    public function validate($number, $config, $context) {
-        $result = parent::validate($number, $config, $context);
-        if ($result === false) return $result;
-        $float = (float) $result;
-        if ($float < 0.0) $result = '0';
-        if ($float > 1.0) $result = '1';
-        return $result;
-    }
+	public function validate($number, $config, $context) {
+		$result = parent::validate($number, $config, $context);
+		if ($result === false) return $result;
+		$float = (float) $result;
+		if ($float < 0.0) $result = '0';
+		if ($float > 1.0) $result = '1';
+		return $result;
+	}
 
 }
 

Braces +9 added lines, -3 removed lines

@@ -9,10 +9,16 @@
 
     public function validate($number, $config, $context) {
         $result = parent::validate($number, $config, $context);
-        if ($result === false) return $result;
+        if ($result === false) {
+        	return $result;
+        }
         $float = (float) $result;
-        if ($float < 0.0) $result = '0';
-        if ($float > 1.0) $result = '1';
+        if ($float < 0.0) {
+        	$result = '0';
+        }
+        if ($float > 1.0) {
+        	$result = '1';
+        }
         return $result;
     }
 

classes/security/htmlpurifier/library/HTMLPurifier/AttrDef/CSS/Background.php

Indentation +74 added lines, -74 removed lines

@@ -7,80 +7,80 @@
 class HTMLPurifier_AttrDef_CSS_Background extends HTMLPurifier_AttrDef
 {
 
-    /**
-     * Local copy of component validators.
-     * @note See HTMLPurifier_AttrDef_Font::$info for a similar impl.
-     */
-    protected $info;
-
-    public function __construct($config) {
-        $def = $config->getCSSDefinition();
-        $this->info['background-color'] = $def->info['background-color'];
-        $this->info['background-image'] = $def->info['background-image'];
-        $this->info['background-repeat'] = $def->info['background-repeat'];
-        $this->info['background-attachment'] = $def->info['background-attachment'];
-        $this->info['background-position'] = $def->info['background-position'];
-    }
-
-    public function validate($string, $config, $context) {
-
-        // regular pre-processing
-        $string = $this->parseCDATA($string);
-        if ($string === '') return false;
-
-        // munge rgb() decl if necessary
-        $string = $this->mungeRgb($string);
-
-        // assumes URI doesn't have spaces in it
-        $bits = explode(' ', strtolower($string)); // bits to process
-
-        $caught = array();
-        $caught['color']    = false;
-        $caught['image']    = false;
-        $caught['repeat']   = false;
-        $caught['attachment'] = false;
-        $caught['position'] = false;
-
-        $i = 0; // number of catches
-        $none = false;
-
-        foreach ($bits as $bit) {
-            if ($bit === '') continue;
-            foreach ($caught as $key => $status) {
-                if ($key != 'position') {
-                    if ($status !== false) continue;
-                    $r = $this->info['background-' . $key]->validate($bit, $config, $context);
-                } else {
-                    $r = $bit;
-                }
-                if ($r === false) continue;
-                if ($key == 'position') {
-                    if ($caught[$key] === false) $caught[$key] = '';
-                    $caught[$key] .= $r . ' ';
-                } else {
-                    $caught[$key] = $r;
-                }
-                $i++;
-                break;
-            }
-        }
-
-        if (!$i) return false;
-        if ($caught['position'] !== false) {
-            $caught['position'] = $this->info['background-position']->
-                validate($caught['position'], $config, $context);
-        }
-
-        $ret = array();
-        foreach ($caught as $value) {
-            if ($value === false) continue;
-            $ret[] = $value;
-        }
-
-        if (empty($ret)) return false;
-        return implode(' ', $ret);
-
-    }
+	/**
+	 * Local copy of component validators.
+	 * @note See HTMLPurifier_AttrDef_Font::$info for a similar impl.
+	 */
+	protected $info;
+
+	public function __construct($config) {
+		$def = $config->getCSSDefinition();
+		$this->info['background-color'] = $def->info['background-color'];
+		$this->info['background-image'] = $def->info['background-image'];
+		$this->info['background-repeat'] = $def->info['background-repeat'];
+		$this->info['background-attachment'] = $def->info['background-attachment'];
+		$this->info['background-position'] = $def->info['background-position'];
+	}
+
+	public function validate($string, $config, $context) {
+
+		// regular pre-processing
+		$string = $this->parseCDATA($string);
+		if ($string === '') return false;
+
+		// munge rgb() decl if necessary
+		$string = $this->mungeRgb($string);
+
+		// assumes URI doesn't have spaces in it
+		$bits = explode(' ', strtolower($string)); // bits to process
+
+		$caught = array();
+		$caught['color']    = false;
+		$caught['image']    = false;
+		$caught['repeat']   = false;
+		$caught['attachment'] = false;
+		$caught['position'] = false;
+
+		$i = 0; // number of catches
+		$none = false;
+
+		foreach ($bits as $bit) {
+			if ($bit === '') continue;
+			foreach ($caught as $key => $status) {
+				if ($key != 'position') {
+					if ($status !== false) continue;
+					$r = $this->info['background-' . $key]->validate($bit, $config, $context);
+				} else {
+					$r = $bit;
+				}
+				if ($r === false) continue;
+				if ($key == 'position') {
+					if ($caught[$key] === false) $caught[$key] = '';
+					$caught[$key] .= $r . ' ';
+				} else {
+					$caught[$key] = $r;
+				}
+				$i++;
+				break;
+			}
+		}
+
+		if (!$i) return false;
+		if ($caught['position'] !== false) {
+			$caught['position'] = $this->info['background-position']->
+				validate($caught['position'], $config, $context);
+		}
+
+		$ret = array();
+		foreach ($caught as $value) {
+			if ($value === false) continue;
+			$ret[] = $value;
+		}
+
+		if (empty($ret)) return false;
+		return implode(' ', $ret);
+
+	}
 
 }
 

Braces +24 added lines, -8 removed lines

@@ -26,7 +26,9 @@ 
 
         // regular pre-processing
         $string = $this->parseCDATA($string);
-        if ($string === '') return false;
+        if ($string === '') {
+        	return false;
+        }
 
         // munge rgb() decl if necessary
         $string = $this->mungeRgb($string);
@@ -45,17 +47,25 @@ 
         $none = false;
 
         foreach ($bits as $bit) {
-            if ($bit === '') continue;
+            if ($bit === '') {
+            	continue;
+            }
             foreach ($caught as $key => $status) {
                 if ($key != 'position') {
-                    if ($status !== false) continue;
+                    if ($status !== false) {
+                    	continue;
+                    }
                     $r = $this->info['background-' . $key]->validate($bit, $config, $context);
                 } else {
                     $r = $bit;
                 }
-                if ($r === false) continue;
+                if ($r === false) {
+                	continue;
+                }
                 if ($key == 'position') {
-                    if ($caught[$key] === false) $caught[$key] = '';
+                    if ($caught[$key] === false) {
+                    	$caught[$key] = '';
+                    }
                     $caught[$key] .= $r . ' ';
                 } else {
                     $caught[$key] = $r;
@@ -65,7 +75,9 @@ 
             }
         }
 
-        if (!$i) return false;
+        if (!$i) {
+        	return false;
+        }
         if ($caught['position'] !== false) {
             $caught['position'] = $this->info['background-position']->
                 validate($caught['position'], $config, $context);
@@ -73,11 +85,15 @@ 
 
         $ret = array();
         foreach ($caught as $value) {
-            if ($value === false) continue;
+            if ($value === false) {
+            	continue;
+            }
             $ret[] = $value;
         }
 
-        if (empty($ret)) return false;
+        if (empty($ret)) {
+        	return false;
+        }
         return implode(' ', $ret);
 
     }

classes/security/htmlpurifier/library/HTMLPurifier/AttrDef/CSS/BackgroundPosition.php

Indentation +83 added lines, -83 removed lines

@@ -44,89 +44,89 @@
 class HTMLPurifier_AttrDef_CSS_BackgroundPosition extends HTMLPurifier_AttrDef
 {
 
-    protected $length;
-    protected $percentage;
-
-    public function __construct() {
-        $this->length     = new HTMLPurifier_AttrDef_CSS_Length();
-        $this->percentage = new HTMLPurifier_AttrDef_CSS_Percentage();
-    }
-
-    public function validate($string, $config, $context) {
-        $string = $this->parseCDATA($string);
-        $bits = explode(' ', $string);
-
-        $keywords = array();
-        $keywords['h'] = false; // left, right
-        $keywords['v'] = false; // top, bottom
-        $keywords['ch'] = false; // center (first word)
-        $keywords['cv'] = false; // center (second word)
-        $measures = array();
-
-        $i = 0;
-
-        $lookup = array(
-            'top' => 'v',
-            'bottom' => 'v',
-            'left' => 'h',
-            'right' => 'h',
-            'center' => 'c'
-        );
-
-        foreach ($bits as $bit) {
-            if ($bit === '') continue;
-
-            // test for keyword
-            $lbit = ctype_lower($bit) ? $bit : strtolower($bit);
-            if (isset($lookup[$lbit])) {
-                $status = $lookup[$lbit];
-                if ($status == 'c') {
-                    if ($i == 0) {
-                        $status = 'ch';
-                    } else {
-                        $status = 'cv';
-                    }
-                }
-                $keywords[$status] = $lbit;
-                $i++;
-            }
-
-            // test for length
-            $r = $this->length->validate($bit, $config, $context);
-            if ($r !== false) {
-                $measures[] = $r;
-                $i++;
-            }
-
-            // test for percentage
-            $r = $this->percentage->validate($bit, $config, $context);
-            if ($r !== false) {
-                $measures[] = $r;
-                $i++;
-            }
-
-        }
-
-        if (!$i) return false; // no valid values were caught
-
-        $ret = array();
-
-        // first keyword
-        if     ($keywords['h'])     $ret[] = $keywords['h'];
-        elseif ($keywords['ch']) {
-            $ret[] = $keywords['ch'];
-            $keywords['cv'] = false; // prevent re-use: center = center center
-        }
-        elseif (count($measures))   $ret[] = array_shift($measures);
-
-        if     ($keywords['v'])     $ret[] = $keywords['v'];
-        elseif ($keywords['cv'])    $ret[] = $keywords['cv'];
-        elseif (count($measures))   $ret[] = array_shift($measures);
-
-        if (empty($ret)) return false;
-        return implode(' ', $ret);
-
-    }
+	protected $length;
+	protected $percentage;
+
+	public function __construct() {
+		$this->length     = new HTMLPurifier_AttrDef_CSS_Length();
+		$this->percentage = new HTMLPurifier_AttrDef_CSS_Percentage();
+	}
+
+	public function validate($string, $config, $context) {
+		$string = $this->parseCDATA($string);
+		$bits = explode(' ', $string);
+
+		$keywords = array();
+		$keywords['h'] = false; // left, right
+		$keywords['v'] = false; // top, bottom
+		$keywords['ch'] = false; // center (first word)
+		$keywords['cv'] = false; // center (second word)
+		$measures = array();
+
+		$i = 0;
+
+		$lookup = array(
+			'top' => 'v',
+			'bottom' => 'v',
+			'left' => 'h',
+			'right' => 'h',
+			'center' => 'c'
+		);
+
+		foreach ($bits as $bit) {
+			if ($bit === '') continue;
+
+			// test for keyword
+			$lbit = ctype_lower($bit) ? $bit : strtolower($bit);
+			if (isset($lookup[$lbit])) {
+				$status = $lookup[$lbit];
+				if ($status == 'c') {
+					if ($i == 0) {
+						$status = 'ch';
+					} else {
+						$status = 'cv';
+					}
+				}
+				$keywords[$status] = $lbit;
+				$i++;
+			}
+
+			// test for length
+			$r = $this->length->validate($bit, $config, $context);
+			if ($r !== false) {
+				$measures[] = $r;
+				$i++;
+			}
+
+			// test for percentage
+			$r = $this->percentage->validate($bit, $config, $context);
+			if ($r !== false) {
+				$measures[] = $r;
+				$i++;
+			}
+
+		}
+
+		if (!$i) return false; // no valid values were caught
+
+		$ret = array();
+
+		// first keyword
+		if     ($keywords['h'])     $ret[] = $keywords['h'];
+		elseif ($keywords['ch']) {
+			$ret[] = $keywords['ch'];
+			$keywords['cv'] = false; // prevent re-use: center = center center
+		}
+		elseif (count($measures))   $ret[] = array_shift($measures);
+
+		if     ($keywords['v'])     $ret[] = $keywords['v'];
+		elseif ($keywords['cv'])    $ret[] = $keywords['cv'];
+		elseif (count($measures))   $ret[] = array_shift($measures);
+
+		if (empty($ret)) return false;
+		return implode(' ', $ret);
+
+	}
 
 }
 

Spacing +2 added lines, -2 removed lines

@@ -112,14 +112,14 @@
         $ret = array();
 
         // first keyword
-        if     ($keywords['h'])     $ret[] = $keywords['h'];
+        if ($keywords['h'])     $ret[] = $keywords['h'];
         elseif ($keywords['ch']) {
             $ret[] = $keywords['ch'];
             $keywords['cv'] = false; // prevent re-use: center = center center
         }
         elseif (count($measures))   $ret[] = array_shift($measures);
 
-        if     ($keywords['v'])     $ret[] = $keywords['v'];
+        if ($keywords['v'])     $ret[] = $keywords['v'];
         elseif ($keywords['cv'])    $ret[] = $keywords['cv'];
         elseif (count($measures))   $ret[] = array_shift($measures);
 

Braces +22 added lines, -9 removed lines

@@ -74,7 +74,9 @@ 
         );
 
         foreach ($bits as $bit) {
-            if ($bit === '') continue;
+            if ($bit === '') {
+            	continue;
+            }
 
             // test for keyword
             $lbit = ctype_lower($bit) ? $bit : strtolower($bit);
@@ -107,23 +109,34 @@ 
 
         }
 
-        if (!$i) return false; // no valid values were caught
+        if (!$i) {
+        	return false;
+        }
+        // no valid values were caught
 
         $ret = array();
 
         // first keyword
-        if     ($keywords['h'])     $ret[] = $keywords['h'];
-        elseif ($keywords['ch']) {
+        if     ($keywords['h']) {
+        	$ret[] = $keywords['h'];
+        } elseif ($keywords['ch']) {
             $ret[] = $keywords['ch'];
             $keywords['cv'] = false; // prevent re-use: center = center center
+        } elseif (count($measures)) {
+        	$ret[] = array_shift($measures);
         }
-        elseif (count($measures))   $ret[] = array_shift($measures);
 
-        if     ($keywords['v'])     $ret[] = $keywords['v'];
-        elseif ($keywords['cv'])    $ret[] = $keywords['cv'];
-        elseif (count($measures))   $ret[] = array_shift($measures);
+        if     ($keywords['v']) {
+        	$ret[] = $keywords['v'];
+        } elseif ($keywords['cv']) {
+        	$ret[] = $keywords['cv'];
+        } elseif (count($measures)) {
+        	$ret[] = array_shift($measures);
+        }
 
-        if (empty($ret)) return false;
+        if (empty($ret)) {
+        	return false;
+        }
         return implode(' ', $ret);
 
     }

classes/security/htmlpurifier/library/HTMLPurifier/AttrDef/CSS/Border.php

Indentation +29 added lines, -29 removed lines

@@ -6,37 +6,37 @@
 class HTMLPurifier_AttrDef_CSS_Border extends HTMLPurifier_AttrDef
 {
 
-    /**
-     * Local copy of properties this property is shorthand for.
-     */
-    protected $info = array();
+	/**
+	 * Local copy of properties this property is shorthand for.
+	 */
+	protected $info = array();
 
-    public function __construct($config) {
-        $def = $config->getCSSDefinition();
-        $this->info['border-width'] = $def->info['border-width'];
-        $this->info['border-style'] = $def->info['border-style'];
-        $this->info['border-top-color'] = $def->info['border-top-color'];
-    }
+	public function __construct($config) {
+		$def = $config->getCSSDefinition();
+		$this->info['border-width'] = $def->info['border-width'];
+		$this->info['border-style'] = $def->info['border-style'];
+		$this->info['border-top-color'] = $def->info['border-top-color'];
+	}
 
-    public function validate($string, $config, $context) {
-        $string = $this->parseCDATA($string);
-        $string = $this->mungeRgb($string);
-        $bits = explode(' ', $string);
-        $done = array(); // segments we've finished
-        $ret = ''; // return value
-        foreach ($bits as $bit) {
-            foreach ($this->info as $propname => $validator) {
-                if (isset($done[$propname])) continue;
-                $r = $validator->validate($bit, $config, $context);
-                if ($r !== false) {
-                    $ret .= $r . ' ';
-                    $done[$propname] = true;
-                    break;
-                }
-            }
-        }
-        return rtrim($ret);
-    }
+	public function validate($string, $config, $context) {
+		$string = $this->parseCDATA($string);
+		$string = $this->mungeRgb($string);
+		$bits = explode(' ', $string);
+		$done = array(); // segments we've finished
+		$ret = ''; // return value
+		foreach ($bits as $bit) {
+			foreach ($this->info as $propname => $validator) {
+				if (isset($done[$propname])) continue;
+				$r = $validator->validate($bit, $config, $context);
+				if ($r !== false) {
+					$ret .= $r . ' ';
+					$done[$propname] = true;
+					break;
+				}
+			}
+		}
+		return rtrim($ret);
+	}
 
 }
 

Braces +3 added lines, -1 removed lines

@@ -26,7 +26,9 @@
         $ret = ''; // return value
         foreach ($bits as $bit) {
             foreach ($this->info as $propname => $validator) {
-                if (isset($done[$propname])) continue;
+                if (isset($done[$propname])) {
+                	continue;
+                }
                 $r = $validator->validate($bit, $config, $context);
                 if ($r !== false) {
                     $ret .= $r . ' ';

classes/security/htmlpurifier/library/HTMLPurifier/AttrDef/CSS/Color.php

Indentation +60 added lines, -60 removed lines

@@ -6,72 +6,72 @@
 class HTMLPurifier_AttrDef_CSS_Color extends HTMLPurifier_AttrDef
 {
 
-    public function validate($color, $config, $context) {
+	public function validate($color, $config, $context) {
 
-        static $colors = null;
-        if ($colors === null) $colors = $config->get('Core.ColorKeywords');
+		static $colors = null;
+		if ($colors === null) $colors = $config->get('Core.ColorKeywords');
 
-        $color = trim($color);
-        if ($color === '') return false;
+		$color = trim($color);
+		if ($color === '') return false;
 
-        $lower = strtolower($color);
-        if (isset($colors[$lower])) return $colors[$lower];
+		$lower = strtolower($color);
+		if (isset($colors[$lower])) return $colors[$lower];
 
-        if (strpos($color, 'rgb(') !== false) {
-            // rgb literal handling
-            $length = strlen($color);
-            if (strpos($color, ')') !== $length - 1) return false;
-            $triad = substr($color, 4, $length - 4 - 1);
-            $parts = explode(',', $triad);
-            if (count($parts) !== 3) return false;
-            $type = false; // to ensure that they're all the same type
-            $new_parts = array();
-            foreach ($parts as $part) {
-                $part = trim($part);
-                if ($part === '') return false;
-                $length = strlen($part);
-                if ($part[$length - 1] === '%') {
-                    // handle percents
-                    if (!$type) {
-                        $type = 'percentage';
-                    } elseif ($type !== 'percentage') {
-                        return false;
-                    }
-                    $num = (float) substr($part, 0, $length - 1);
-                    if ($num < 0) $num = 0;
-                    if ($num > 100) $num = 100;
-                    $new_parts[] = "$num%";
-                } else {
-                    // handle integers
-                    if (!$type) {
-                        $type = 'integer';
-                    } elseif ($type !== 'integer') {
-                        return false;
-                    }
-                    $num = (int) $part;
-                    if ($num < 0) $num = 0;
-                    if ($num > 255) $num = 255;
-                    $new_parts[] = (string) $num;
-                }
-            }
-            $new_triad = implode(',', $new_parts);
-            $color = "rgb($new_triad)";
-        } else {
-            // hexadecimal handling
-            if ($color[0] === '#') {
-                $hex = substr($color, 1);
-            } else {
-                $hex = $color;
-                $color = '#' . $color;
-            }
-            $length = strlen($hex);
-            if ($length !== 3 && $length !== 6) return false;
-            if (!ctype_xdigit($hex)) return false;
-        }
+		if (strpos($color, 'rgb(') !== false) {
+			// rgb literal handling
+			$length = strlen($color);
+			if (strpos($color, ')') !== $length - 1) return false;
+			$triad = substr($color, 4, $length - 4 - 1);
+			$parts = explode(',', $triad);
+			if (count($parts) !== 3) return false;
+			$type = false; // to ensure that they're all the same type
+			$new_parts = array();
+			foreach ($parts as $part) {
+				$part = trim($part);
+				if ($part === '') return false;
+				$length = strlen($part);
+				if ($part[$length - 1] === '%') {
+					// handle percents
+					if (!$type) {
+						$type = 'percentage';
+					} elseif ($type !== 'percentage') {
+						return false;
+					}
+					$num = (float) substr($part, 0, $length - 1);
+					if ($num < 0) $num = 0;
+					if ($num > 100) $num = 100;
+					$new_parts[] = "$num%";
+				} else {
+					// handle integers
+					if (!$type) {
+						$type = 'integer';
+					} elseif ($type !== 'integer') {
+						return false;
+					}
+					$num = (int) $part;
+					if ($num < 0) $num = 0;
+					if ($num > 255) $num = 255;
+					$new_parts[] = (string) $num;
+				}
+			}
+			$new_triad = implode(',', $new_parts);
+			$color = "rgb($new_triad)";
+		} else {
+			// hexadecimal handling
+			if ($color[0] === '#') {
+				$hex = substr($color, 1);
+			} else {
+				$hex = $color;
+				$color = '#' . $color;
+			}
+			$length = strlen($hex);
+			if ($length !== 3 && $length !== 6) return false;
+			if (!ctype_xdigit($hex)) return false;
+		}
 
-        return $color;
+		return $color;
 
-    }
+	}
 
 }
 

Braces +36 added lines, -12 removed lines

@@ -9,26 +9,38 @@ 
     public function validate($color, $config, $context) {
 
         static $colors = null;
-        if ($colors === null) $colors = $config->get('Core.ColorKeywords');
+        if ($colors === null) {
+        	$colors = $config->get('Core.ColorKeywords');
+        }
 
         $color = trim($color);
-        if ($color === '') return false;
+        if ($color === '') {
+        	return false;
+        }
 
         $lower = strtolower($color);
-        if (isset($colors[$lower])) return $colors[$lower];
+        if (isset($colors[$lower])) {
+        	return $colors[$lower];
+        }
 
         if (strpos($color, 'rgb(') !== false) {
             // rgb literal handling
             $length = strlen($color);
-            if (strpos($color, ')') !== $length - 1) return false;
+            if (strpos($color, ')') !== $length - 1) {
+            	return false;
+            }
             $triad = substr($color, 4, $length - 4 - 1);
             $parts = explode(',', $triad);
-            if (count($parts) !== 3) return false;
+            if (count($parts) !== 3) {
+            	return false;
+            }
             $type = false; // to ensure that they're all the same type
             $new_parts = array();
             foreach ($parts as $part) {
                 $part = trim($part);
-                if ($part === '') return false;
+                if ($part === '') {
+                	return false;
+                }
                 $length = strlen($part);
                 if ($part[$length - 1] === '%') {
                     // handle percents
@@ -38,8 +50,12 @@ 
                         return false;
                     }
                     $num = (float) substr($part, 0, $length - 1);
-                    if ($num < 0) $num = 0;
-                    if ($num > 100) $num = 100;
+                    if ($num < 0) {
+                    	$num = 0;
+                    }
+                    if ($num > 100) {
+                    	$num = 100;
+                    }
                     $new_parts[] = "$num%";
                 } else {
                     // handle integers
@@ -49,8 +65,12 @@ 
                         return false;
                     }
                     $num = (int) $part;
-                    if ($num < 0) $num = 0;
-                    if ($num > 255) $num = 255;
+                    if ($num < 0) {
+                    	$num = 0;
+                    }
+                    if ($num > 255) {
+                    	$num = 255;
+                    }
                     $new_parts[] = (string) $num;
                 }
             }
@@ -65,8 +85,12 @@ 
                 $color = '#' . $color;
             }
             $length = strlen($hex);
-            if ($length !== 3 && $length !== 6) return false;
-            if (!ctype_xdigit($hex)) return false;
+            if ($length !== 3 && $length !== 6) {
+            	return false;
+            }
+            if (!ctype_xdigit($hex)) {
+            	return false;
+            }
         }
 
         return $color;

classes/security/htmlpurifier/library/HTMLPurifier/AttrDef/CSS/Composite.php

Indentation +18 added lines, -18 removed lines

@@ -12,26 +12,26 @@
 class HTMLPurifier_AttrDef_CSS_Composite extends HTMLPurifier_AttrDef
 {
 
-    /**
-     * List of HTMLPurifier_AttrDef objects that may process strings
-     * @todo Make protected
-     */
-    public $defs;
+	/**
+	 * List of HTMLPurifier_AttrDef objects that may process strings
+	 * @todo Make protected
+	 */
+	public $defs;
 
-    /**
-     * @param $defs List of HTMLPurifier_AttrDef objects
-     */
-    public function __construct($defs) {
-        $this->defs = $defs;
-    }
+	/**
+	 * @param $defs List of HTMLPurifier_AttrDef objects
+	 */
+	public function __construct($defs) {
+		$this->defs = $defs;
+	}
 
-    public function validate($string, $config, $context) {
-        foreach ($this->defs as $i => $def) {
-            $result = $this->defs[$i]->validate($string, $config, $context);
-            if ($result !== false) return $result;
-        }
-        return false;
-    }
+	public function validate($string, $config, $context) {
+		foreach ($this->defs as $i => $def) {
+			$result = $this->defs[$i]->validate($string, $config, $context);
+			if ($result !== false) return $result;
+		}
+		return false;
+	}
 
 }
 

Braces +3 added lines, -1 removed lines

@@ -28,7 +28,9 @@
     public function validate($string, $config, $context) {
         foreach ($this->defs as $i => $def) {
             $result = $this->defs[$i]->validate($string, $config, $context);
-            if ($result !== false) return $result;
+            if ($result !== false) {
+            	return $result;
+            }
         }
         return false;
     }

classes/security/htmlpurifier/library/HTMLPurifier/AttrDef/CSS/DenyElementDecorator.php

Indentation +17 added lines, -17 removed lines

@@ -5,24 +5,24 @@
  */
 class HTMLPurifier_AttrDef_CSS_DenyElementDecorator extends HTMLPurifier_AttrDef
 {
-    public $def, $element;
+	public $def, $element;
 
-    /**
-     * @param $def Definition to wrap
-     * @param $element Element to deny
-     */
-    public function __construct($def, $element) {
-        $this->def = $def;
-        $this->element = $element;
-    }
-    /**
-     * Checks if CurrentToken is set and equal to $this->element
-     */
-    public function validate($string, $config, $context) {
-        $token = $context->get('CurrentToken', true);
-        if ($token && $token->name == $this->element) return false;
-        return $this->def->validate($string, $config, $context);
-    }
+	/**
+	 * @param $def Definition to wrap
+	 * @param $element Element to deny
+	 */
+	public function __construct($def, $element) {
+		$this->def = $def;
+		$this->element = $element;
+	}
+	/**
+	 * Checks if CurrentToken is set and equal to $this->element
+	 */
+	public function validate($string, $config, $context) {
+		$token = $context->get('CurrentToken', true);
+		if ($token && $token->name == $this->element) return false;
+		return $this->def->validate($string, $config, $context);
+	}
 }
 
 // vim: et sw=4 sts=4

Braces +3 added lines, -1 removed lines

@@ -20,7 +20,9 @@
      */
     public function validate($string, $config, $context) {
         $token = $context->get('CurrentToken', true);
-        if ($token && $token->name == $this->element) return false;
+        if ($token && $token->name == $this->element) {
+        	return false;
+        }
         return $this->def->validate($string, $config, $context);
     }
 }

classes/security/htmlpurifier/library/HTMLPurifier/AttrDef/CSS/Filter.php

Indentation +38 added lines, -38 removed lines

@@ -8,46 +8,46 @@
 class HTMLPurifier_AttrDef_CSS_Filter extends HTMLPurifier_AttrDef
 {
 
-    protected $intValidator;
+	protected $intValidator;
 
-    public function __construct() {
-        $this->intValidator = new HTMLPurifier_AttrDef_Integer();
-    }
+	public function __construct() {
+		$this->intValidator = new HTMLPurifier_AttrDef_Integer();
+	}
 
-    public function validate($value, $config, $context) {
-        $value = $this->parseCDATA($value);
-        if ($value === 'none') return $value;
-        // if we looped this we could support multiple filters
-        $function_length = strcspn($value, '(');
-        $function = trim(substr($value, 0, $function_length));
-        if ($function !== 'alpha' &&
-            $function !== 'Alpha' &&
-            $function !== 'progid:DXImageTransform.Microsoft.Alpha'
-            ) return false;
-        $cursor = $function_length + 1;
-        $parameters_length = strcspn($value, ')', $cursor);
-        $parameters = substr($value, $cursor, $parameters_length);
-        $params = explode(',', $parameters);
-        $ret_params = array();
-        $lookup = array();
-        foreach ($params as $param) {
-            list($key, $value) = explode('=', $param);
-            $key   = trim($key);
-            $value = trim($value);
-            if (isset($lookup[$key])) continue;
-            if ($key !== 'opacity') continue;
-            $value = $this->intValidator->validate($value, $config, $context);
-            if ($value === false) continue;
-            $int = (int) $value;
-            if ($int > 100) $value = '100';
-            if ($int < 0) $value = '0';
-            $ret_params[] = "$key=$value";
-            $lookup[$key] = true;
-        }
-        $ret_parameters = implode(',', $ret_params);
-        $ret_function = "$function($ret_parameters)";
-        return $ret_function;
-    }
+	public function validate($value, $config, $context) {
+		$value = $this->parseCDATA($value);
+		if ($value === 'none') return $value;
+		// if we looped this we could support multiple filters
+		$function_length = strcspn($value, '(');
+		$function = trim(substr($value, 0, $function_length));
+		if ($function !== 'alpha' &&
+			$function !== 'Alpha' &&
+			$function !== 'progid:DXImageTransform.Microsoft.Alpha'
+			) return false;
+		$cursor = $function_length + 1;
+		$parameters_length = strcspn($value, ')', $cursor);
+		$parameters = substr($value, $cursor, $parameters_length);
+		$params = explode(',', $parameters);
+		$ret_params = array();
+		$lookup = array();
+		foreach ($params as $param) {
+			list($key, $value) = explode('=', $param);
+			$key   = trim($key);
+			$value = trim($value);
+			if (isset($lookup[$key])) continue;
+			if ($key !== 'opacity') continue;
+			$value = $this->intValidator->validate($value, $config, $context);
+			if ($value === false) continue;
+			$int = (int) $value;
+			if ($int > 100) $value = '100';
+			if ($int < 0) $value = '0';
+			$ret_params[] = "$key=$value";
+			$lookup[$key] = true;
+		}
+		$ret_parameters = implode(',', $ret_params);
+		$ret_function = "$function($ret_parameters)";
+		return $ret_function;
+	}
 
 }
 

Braces +21 added lines, -7 removed lines

@@ -16,14 +16,18 @@ 
 
     public function validate($value, $config, $context) {
         $value = $this->parseCDATA($value);
-        if ($value === 'none') return $value;
+        if ($value === 'none') {
+        	return $value;
+        }
         // if we looped this we could support multiple filters
         $function_length = strcspn($value, '(');
         $function = trim(substr($value, 0, $function_length));
         if ($function !== 'alpha' &&
             $function !== 'Alpha' &&
             $function !== 'progid:DXImageTransform.Microsoft.Alpha'
-            ) return false;
+            ) {
+        	return false;
+        }
         $cursor = $function_length + 1;
         $parameters_length = strcspn($value, ')', $cursor);
         $parameters = substr($value, $cursor, $parameters_length);
@@ -34,13 +38,23 @@ 
             list($key, $value) = explode('=', $param);
             $key   = trim($key);
             $value = trim($value);
-            if (isset($lookup[$key])) continue;
-            if ($key !== 'opacity') continue;
+            if (isset($lookup[$key])) {
+            	continue;
+            }
+            if ($key !== 'opacity') {
+            	continue;
+            }
             $value = $this->intValidator->validate($value, $config, $context);
-            if ($value === false) continue;
+            if ($value === false) {
+            	continue;
+            }
             $int = (int) $value;
-            if ($int > 100) $value = '100';
-            if ($int < 0) $value = '0';
+            if ($int > 100) {
+            	$value = '100';
+            }
+            if ($int < 0) {
+            	$value = '0';
+            }
             $ret_params[] = "$key=$value";
             $lookup[$key] = true;
         }

classes/security/htmlpurifier/library/HTMLPurifier/AttrDef/CSS/Font.php

Indentation +127 added lines, -127 removed lines

@@ -6,143 +6,143 @@
 class HTMLPurifier_AttrDef_CSS_Font extends HTMLPurifier_AttrDef
 {
 
-    /**
-     * Local copy of component validators.
-     *
-     * @note If we moved specific CSS property definitions to their own
-     *       classes instead of having them be assembled at run time by
-     *       CSSDefinition, this wouldn't be necessary.  We'd instantiate
-     *       our own copies.
-     */
-    protected $info = array();
+	/**
+	 * Local copy of component validators.
+	 *
+	 * @note If we moved specific CSS property definitions to their own
+	 *       classes instead of having them be assembled at run time by
+	 *       CSSDefinition, this wouldn't be necessary.  We'd instantiate
+	 *       our own copies.
+	 */
+	protected $info = array();
 
-    public function __construct($config) {
-        $def = $config->getCSSDefinition();
-        $this->info['font-style']   = $def->info['font-style'];
-        $this->info['font-variant'] = $def->info['font-variant'];
-        $this->info['font-weight']  = $def->info['font-weight'];
-        $this->info['font-size']    = $def->info['font-size'];
-        $this->info['line-height']  = $def->info['line-height'];
-        $this->info['font-family']  = $def->info['font-family'];
-    }
+	public function __construct($config) {
+		$def = $config->getCSSDefinition();
+		$this->info['font-style']   = $def->info['font-style'];
+		$this->info['font-variant'] = $def->info['font-variant'];
+		$this->info['font-weight']  = $def->info['font-weight'];
+		$this->info['font-size']    = $def->info['font-size'];
+		$this->info['line-height']  = $def->info['line-height'];
+		$this->info['font-family']  = $def->info['font-family'];
+	}
 
-    public function validate($string, $config, $context) {
+	public function validate($string, $config, $context) {
 
-        static $system_fonts = array(
-            'caption' => true,
-            'icon' => true,
-            'menu' => true,
-            'message-box' => true,
-            'small-caption' => true,
-            'status-bar' => true
-        );
+		static $system_fonts = array(
+			'caption' => true,
+			'icon' => true,
+			'menu' => true,
+			'message-box' => true,
+			'small-caption' => true,
+			'status-bar' => true
+		);
 
-        // regular pre-processing
-        $string = $this->parseCDATA($string);
-        if ($string === '') return false;
+		// regular pre-processing
+		$string = $this->parseCDATA($string);
+		if ($string === '') return false;
 
-        // check if it's one of the keywords
-        $lowercase_string = strtolower($string);
-        if (isset($system_fonts[$lowercase_string])) {
-            return $lowercase_string;
-        }
+		// check if it's one of the keywords
+		$lowercase_string = strtolower($string);
+		if (isset($system_fonts[$lowercase_string])) {
+			return $lowercase_string;
+		}
 
-        $bits = explode(' ', $string); // bits to process
-        $stage = 0; // this indicates what we're looking for
-        $caught = array(); // which stage 0 properties have we caught?
-        $stage_1 = array('font-style', 'font-variant', 'font-weight');
-        $final = ''; // output
+		$bits = explode(' ', $string); // bits to process
+		$stage = 0; // this indicates what we're looking for
+		$caught = array(); // which stage 0 properties have we caught?
+		$stage_1 = array('font-style', 'font-variant', 'font-weight');
+		$final = ''; // output
 
-        for ($i = 0, $size = count($bits); $i < $size; $i++) {
-            if ($bits[$i] === '') continue;
-            switch ($stage) {
+		for ($i = 0, $size = count($bits); $i < $size; $i++) {
+			if ($bits[$i] === '') continue;
+			switch ($stage) {
 
-                // attempting to catch font-style, font-variant or font-weight
-                case 0:
-                    foreach ($stage_1 as $validator_name) {
-                        if (isset($caught[$validator_name])) continue;
-                        $r = $this->info[$validator_name]->validate(
-                                                $bits[$i], $config, $context);
-                        if ($r !== false) {
-                            $final .= $r . ' ';
-                            $caught[$validator_name] = true;
-                            break;
-                        }
-                    }
-                    // all three caught, continue on
-                    if (count($caught) >= 3) $stage = 1;
-                    if ($r !== false) break;
+				// attempting to catch font-style, font-variant or font-weight
+				case 0:
+					foreach ($stage_1 as $validator_name) {
+						if (isset($caught[$validator_name])) continue;
+						$r = $this->info[$validator_name]->validate(
+												$bits[$i], $config, $context);
+						if ($r !== false) {
+							$final .= $r . ' ';
+							$caught[$validator_name] = true;
+							break;
+						}
+					}
+					// all three caught, continue on
+					if (count($caught) >= 3) $stage = 1;
+					if ($r !== false) break;
 
-                // attempting to catch font-size and perhaps line-height
-                case 1:
-                    $found_slash = false;
-                    if (strpos($bits[$i], '/') !== false) {
-                        list($font_size, $line_height) =
-                                                    explode('/', $bits[$i]);
-                        if ($line_height === '') {
-                            // ooh, there's a space after the slash!
-                            $line_height = false;
-                            $found_slash = true;
-                        }
-                    } else {
-                        $font_size = $bits[$i];
-                        $line_height = false;
-                    }
-                    $r = $this->info['font-size']->validate(
-                                              $font_size, $config, $context);
-                    if ($r !== false) {
-                        $final .= $r;
-                        // attempt to catch line-height
-                        if ($line_height === false) {
-                            // we need to scroll forward
-                            for ($j = $i + 1; $j < $size; $j++) {
-                                if ($bits[$j] === '') continue;
-                                if ($bits[$j] === '/') {
-                                    if ($found_slash) {
-                                        return false;
-                                    } else {
-                                        $found_slash = true;
-                                        continue;
-                                    }
-                                }
-                                $line_height = $bits[$j];
-                                break;
-                            }
-                        } else {
-                            // slash already found
-                            $found_slash = true;
-                            $j = $i;
-                        }
-                        if ($found_slash) {
-                            $i = $j;
-                            $r = $this->info['line-height']->validate(
-                                              $line_height, $config, $context);
-                            if ($r !== false) {
-                                $final .= '/' . $r;
-                            }
-                        }
-                        $final .= ' ';
-                        $stage = 2;
-                        break;
-                    }
-                    return false;
+				// attempting to catch font-size and perhaps line-height
+				case 1:
+					$found_slash = false;
+					if (strpos($bits[$i], '/') !== false) {
+						list($font_size, $line_height) =
+													explode('/', $bits[$i]);
+						if ($line_height === '') {
+							// ooh, there's a space after the slash!
+							$line_height = false;
+							$found_slash = true;
+						}
+					} else {
+						$font_size = $bits[$i];
+						$line_height = false;
+					}
+					$r = $this->info['font-size']->validate(
+											  $font_size, $config, $context);
+					if ($r !== false) {
+						$final .= $r;
+						// attempt to catch line-height
+						if ($line_height === false) {
+							// we need to scroll forward
+							for ($j = $i + 1; $j < $size; $j++) {
+								if ($bits[$j] === '') continue;
+								if ($bits[$j] === '/') {
+									if ($found_slash) {
+										return false;
+									} else {
+										$found_slash = true;
+										continue;
+									}
+								}
+								$line_height = $bits[$j];
+								break;
+							}
+						} else {
+							// slash already found
+							$found_slash = true;
+							$j = $i;
+						}
+						if ($found_slash) {
+							$i = $j;
+							$r = $this->info['line-height']->validate(
+											  $line_height, $config, $context);
+							if ($r !== false) {
+								$final .= '/' . $r;
+							}
+						}
+						$final .= ' ';
+						$stage = 2;
+						break;
+					}
+					return false;
 
-                // attempting to catch font-family
-                case 2:
-                    $font_family =
-                        implode(' ', array_slice($bits, $i, $size - $i));
-                    $r = $this->info['font-family']->validate(
-                                              $font_family, $config, $context);
-                    if ($r !== false) {
-                        $final .= $r . ' ';
-                        // processing completed successfully
-                        return rtrim($final);
-                    }
-                    return false;
-            }
-        }
-        return false;
-    }
+				// attempting to catch font-family
+				case 2:
+					$font_family =
+						implode(' ', array_slice($bits, $i, $size - $i));
+					$r = $this->info['font-family']->validate(
+											  $font_family, $config, $context);
+					if ($r !== false) {
+						$final .= $r . ' ';
+						// processing completed successfully
+						return rtrim($final);
+					}
+					return false;
+			}
+		}
+		return false;
+	}
 
 }
 

Braces +18 added lines, -6 removed lines

@@ -39,7 +39,9 @@ 
 
         // regular pre-processing
         $string = $this->parseCDATA($string);
-        if ($string === '') return false;
+        if ($string === '') {
+        	return false;
+        }
 
         // check if it's one of the keywords
         $lowercase_string = strtolower($string);
@@ -54,13 +56,17 @@ 
         $final = ''; // output
 
         for ($i = 0, $size = count($bits); $i < $size; $i++) {
-            if ($bits[$i] === '') continue;
+            if ($bits[$i] === '') {
+            	continue;
+            }
             switch ($stage) {
 
                 // attempting to catch font-style, font-variant or font-weight
                 case 0:
                     foreach ($stage_1 as $validator_name) {
-                        if (isset($caught[$validator_name])) continue;
+                        if (isset($caught[$validator_name])) {
+                        	continue;
+                        }
                         $r = $this->info[$validator_name]->validate(
                                                 $bits[$i], $config, $context);
                         if ($r !== false) {
@@ -70,8 +76,12 @@ 
                         }
                     }
                     // all three caught, continue on
-                    if (count($caught) >= 3) $stage = 1;
-                    if ($r !== false) break;
+                    if (count($caught) >= 3) {
+                    	$stage = 1;
+                    }
+                    if ($r !== false) {
+                    	break;
+                    }
 
                 // attempting to catch font-size and perhaps line-height
                 case 1:
@@ -96,7 +106,9 @@ 
                         if ($line_height === false) {
                             // we need to scroll forward
                             for ($j = $i + 1; $j < $size; $j++) {
-                                if ($bits[$j] === '') continue;
+                                if ($bits[$j] === '') {
+                                	continue;
+                                }
                                 if ($bits[$j] === '/') {
                                     if ($found_slash) {
                                         return false;