mirrors/htmlpurifier
0
0
Fork 0
mirror of https://github.com/ezyang/htmlpurifier.git synced 2025-01-18 11:41:52 +00:00
Code Releases Activity

Correct typehinting of maybeGet* (#240)

Browse Source 
getDefinition can return null, this wasn't properly hinted leaning to false error detections with static analyzers
This commit is contained in:
Anders Jenbo 2019-12-04 16:29:08 +01:00 committed by Edward Z. Yang
parent df923d1f15
commit fe0452d688
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
library/HTMLPurifier/Config.php
View File

@ -408,7 +408,7 @@ class HTMLPurifier_Config
* maybeGetRawHTMLDefinition, which is more explicitly * maybeGetRawHTMLDefinition, which is more explicitly
* named, instead. * named, instead.
* *
* @return HTMLPurifier_HTMLDefinition * @return HTMLPurifier_HTMLDefinition|null
*/ */
public function getHTMLDefinition($raw = false, $optimized = false) public function getHTMLDefinition($raw = false, $optimized = false)
{ {
@ -427,7 +427,7 @@ class HTMLPurifier_Config
* maybeGetRawCSSDefinition, which is more explicitly * maybeGetRawCSSDefinition, which is more explicitly
* named, instead. * named, instead.
* *
* @return HTMLPurifier_CSSDefinition * @return HTMLPurifier_CSSDefinition|null
*/ */
public function getCSSDefinition($raw = false, $optimized = false) public function getCSSDefinition($raw = false, $optimized = false)
{ {
@ -446,7 +446,7 @@ class HTMLPurifier_Config
* maybeGetRawURIDefinition, which is more explicitly * maybeGetRawURIDefinition, which is more explicitly
* named, instead. * named, instead.
* *
* @return HTMLPurifier_URIDefinition * @return HTMLPurifier_URIDefinition|null
*/ */
public function getURIDefinition($raw = false, $optimized = false) public function getURIDefinition($raw = false, $optimized = false)
{ {
@ -468,7 +468,7 @@ class HTMLPurifier_Config
* maybe semantics is the "right thing to do." * maybe semantics is the "right thing to do."
* *
* @throws HTMLPurifier_Exception * @throws HTMLPurifier_Exception
* @return HTMLPurifier_Definition * @return HTMLPurifier_Definition|null
*/ */
public function getDefinition($type, $raw = false, $optimized = false) public function getDefinition($type, $raw = false, $optimized = false)
{ {
@ -647,7 +647,7 @@ class HTMLPurifier_Config
} }
/** /**
* @return HTMLPurifier_HTMLDefinition * @return HTMLPurifier_HTMLDefinition|null
*/ */
public function maybeGetRawHTMLDefinition() public function maybeGetRawHTMLDefinition()
{ {
@ -655,7 +655,7 @@ class HTMLPurifier_Config
} }
/** /**
* @return HTMLPurifier_CSSDefinition * @return HTMLPurifier_CSSDefinition|null
*/ */
public function maybeGetRawCSSDefinition() public function maybeGetRawCSSDefinition()
{ {
@ -663,7 +663,7 @@ class HTMLPurifier_Config
} }
/** /**
* @return HTMLPurifier_URIDefinition * @return HTMLPurifier_URIDefinition|null
*/ */
public function maybeGetRawURIDefinition() public function maybeGetRawURIDefinition()
{ {