DakuTree / manga-tracker

application/models/User_Options_Model.php

Doc Comments +4 added lines, -1 removed lines

@@ -73,7 +73,7 @@ 
 	/**
 	 * Get user option, or default option if it does not exist.
 	 * @param string $option
-	 * @return mixed Returns option value as STRING, or FALSE if option does not exist.
+	 * @return string Returns option value as STRING, or FALSE if option does not exist.
 	 */
 	public function get(string $option) {
 		return $this->get_by_userid($option, (int) $this->User->id);
@@ -135,6 +135,9 @@ 
 		return $success;
 	}
 
+	/**
+	 * @return string
+	 */
 	private function get_db(string $option, int $userID) {
 		//This function assumes we've already done some basic validation.
 		$query = $this->db->select('value_str, value_int')

Braces +10 added lines, -3 removed lines

@@ -137,7 +137,9 @@ 
 				}
 
 				$success = $this->set_db($idData, $valueData);
-				if($success) $this->session->unset_tempdata("option_{$option}");
+				if($success) {
+					$this->session->unset_tempdata("option_{$option}");
+				}
 			} else {
 				$success = FALSE;
 			}
@@ -164,7 +166,9 @@ 
 			}
 
 			//Overall fallback method.
-			if(!isset($value)) $value = $this->options[$option]['default'];
+			if(!isset($value)) {
+				$value = $this->options[$option]['default'];
+			}
 		} else {
 			$value = FALSE;
 		}
@@ -215,7 +219,10 @@ 
 				//This should never happen.
 				break;
 		}
-		if(!isset($value)) $value = FALSE; //FIXME: This won't play nice with BOOL type false?
+		if(!isset($value)) {
+			$value = FALSE;
+		}
+		//FIXME: This won't play nice with BOOL type false?
 
 		return $value;
 	}