Completed
Push — feature/code-analysis ( 0f93c5...1158f1 )
by Jonathan
03:21
created
src/Webtrees/Cache.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 	 *
115 115
 	 * @param string $value Value name
116 116
 	 * @param AbstractModule $mod Calling module
117
-	 * @return bool True is cached
117
+	 * @return boolean|null True is cached
118 118
 	 */
119 119
 	public static function isCached($value, AbstractModule $mod = null) {
120 120
 	    self::getInstance()->isCachedI($value, $mod);
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 	 * @param string $value Value name
150 150
 	 * @param mixed $data Value
151 151
 	 * @param AbstractModule $mod Calling module
152
-	 * @return mixed Cached value
152
+	 * @return unknown_type|null Cached value
153 153
 	 */
154 154
 	public function saveI($value, $data, AbstractModule $mod = null){
155 155
 		$this->checkInit();
Please login to merge, or discard this patch.
src/Webtrees/Module/Certificates/CertificateController.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -124,7 +124,9 @@
 block discarded – undo
124 124
         
125 125
         $cid   = Filter::get('cid');
126 126
         $certificate = null;
127
-        if(!empty($cid)) $certificate =  Certificate::getInstance($cid, $WT_TREE, null, $this->provider);
127
+        if(!empty($cid)) {
128
+        	$certificate =  Certificate::getInstance($cid, $WT_TREE, null, $this->provider);
129
+        }
128 130
         
129 131
         $imageBuilder = new ImageBuilder($certificate);
130 132
         
Please login to merge, or discard this patch.