ghalusa /
PHP-Skeleton-App
This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include, or for example
via PHP's auto-loading mechanism.
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
| 1 | <?php |
||
| 2 | /** |
||
| 3 | * The PHP Skeleton App |
||
| 4 | * |
||
| 5 | * @author Goran Halusa <[email protected]> |
||
| 6 | * @copyright 2015 Goran Halusa |
||
| 7 | * @link https://github.com/ghalusa/PHP-Skeleton-App |
||
| 8 | * @license https://github.com/ghalusa/PHP-Skeleton-App/wiki/License |
||
| 9 | * @version 0.1.1 |
||
| 10 | * @package PHP Skeleton App |
||
| 11 | * |
||
| 12 | * For the full copyright and license information, please view the LICENSE |
||
| 13 | * file that was distributed with this source code. |
||
| 14 | */ |
||
| 15 | |||
| 16 | /** |
||
| 17 | * Group |
||
| 18 | * |
||
| 19 | * Class for the Group module, providing methods for browsing and managing groups. |
||
| 20 | * |
||
| 21 | * @author Goran Halusa <[email protected]> |
||
| 22 | * @since 0.1.0 |
||
| 23 | */ |
||
| 24 | |||
| 25 | namespace PHPSkeleton; |
||
| 26 | use PDO; |
||
| 27 | |||
| 28 | class Group |
||
| 29 | { |
||
| 30 | /** |
||
| 31 | * @var string|bool $session_key The session key |
||
| 32 | */ |
||
| 33 | private $session_key = false; |
||
| 34 | |||
| 35 | /** |
||
| 36 | * @var object $db The database connection object |
||
| 37 | */ |
||
| 38 | public $db; |
||
| 39 | |||
| 40 | /** |
||
| 41 | * Constructor |
||
| 42 | * @param object $db_connection The database connection object |
||
| 43 | * @param string $session_key The session key |
||
| 44 | */ |
||
| 45 | View Code Duplication | public function __construct($db_connection = false, $session_key = false) |
|
| 46 | { |
||
| 47 | if ($db_connection && is_object($db_connection)) { |
||
| 48 | $this->db = $db_connection; |
||
| 49 | } |
||
| 50 | $this->session_key = $session_key; |
||
| 51 | } |
||
| 52 | |||
| 53 | /** |
||
| 54 | * Browse Groups |
||
| 55 | * |
||
| 56 | * Run a query to retreive all groups in the database. |
||
| 57 | * |
||
| 58 | * @param string $sort_field The data value |
||
| 59 | * @param string $sort_order The data value |
||
| 60 | * @param int $start_record The data value |
||
| 61 | * @param int $stop_record The data value |
||
| 62 | * @param string $search The data value |
||
| 63 | * @return array|bool The query result |
||
| 64 | */ |
||
| 65 | public function browse_groups( |
||
| 66 | $sort_field = false, $sort_order = 'DESC', $start_record = 0, $stop_record = 20, $search = false |
||
| 67 | ) { |
||
| 68 | $sort = ""; |
||
| 69 | $search_sql = ""; |
||
| 70 | $pdo_params = array(); |
||
| 71 | $data = array(); |
||
| 72 | |||
| 73 | $limit_sql = " LIMIT {$start_record}, {$stop_record} "; |
||
| 74 | |||
| 75 | View Code Duplication | if ($sort_field) { |
|
|
0 ignored issues
–
show
|
|||
| 76 | switch ($sort_field) { |
||
| 77 | case 'last_modified': |
||
| 78 | $sort = " ORDER BY group.last_modified {$sort_order} "; |
||
| 79 | break; |
||
| 80 | default: |
||
| 81 | $sort = " ORDER BY {$sort_field} {$sort_order} "; |
||
| 82 | } |
||
| 83 | } |
||
| 84 | |||
| 85 | if ($search) { |
||
|
0 ignored issues
–
show
The expression
$search of type false|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.
In PHP, under loose comparison (like For '' == false // true
'' == null // true
'ab' == false // false
'ab' == null // false
// It is often better to use strict comparison
'' === false // false
'' === null // false
Loading history...
|
|||
| 86 | $pdo_params[] = '%'.$search.'%'; |
||
| 87 | $pdo_params[] = '%'.$search.'%'; |
||
| 88 | $pdo_params[] = '%'.$search.'%'; |
||
| 89 | $search_sql = " |
||
| 90 | AND ( |
||
| 91 | `group`.name LIKE ? |
||
| 92 | OR `group`.abbreviation LIKE ? |
||
| 93 | OR `group`.description LIKE ? |
||
| 94 | ) "; |
||
| 95 | } |
||
| 96 | |||
| 97 | $statement = $this->db->prepare(" |
||
| 98 | SELECT SQL_CALC_FOUND_ROWS |
||
| 99 | `group`.group_id AS manage |
||
| 100 | ,`group`.group_id |
||
| 101 | ,`group`.name |
||
| 102 | ,`group`.abbreviation |
||
| 103 | ,`group`.description |
||
| 104 | ,`group`.address_1 AS address |
||
| 105 | ,`group`.city |
||
| 106 | ,DATE_FORMAT(`group`.last_modified,'%m/%d/%Y') AS last_modified |
||
| 107 | ,`group`.group_id AS DT_RowId |
||
| 108 | FROM `group` |
||
| 109 | WHERE `group`.active = 1 |
||
| 110 | {$search_sql} |
||
| 111 | GROUP BY `group`.group_id |
||
| 112 | HAVING 1=1 |
||
| 113 | {$sort} |
||
| 114 | {$limit_sql}"); |
||
| 115 | $statement->execute($pdo_params); |
||
| 116 | $data["aaData"] = $statement->fetchAll(PDO::FETCH_ASSOC); |
||
| 117 | |||
| 118 | $statement = $this->db->prepare("SELECT FOUND_ROWS()"); |
||
| 119 | $statement->execute(); |
||
| 120 | $count = $statement->fetch(PDO::FETCH_ASSOC); |
||
| 121 | $data["iTotalRecords"] = $count["FOUND_ROWS()"]; |
||
| 122 | $data["iTotalDisplayRecords"] = $count["FOUND_ROWS()"]; |
||
| 123 | return $data; |
||
| 124 | } |
||
| 125 | |||
| 126 | /** |
||
| 127 | * Delete Group |
||
| 128 | * |
||
| 129 | * Run a query to remove a group from the database. |
||
| 130 | * |
||
| 131 | * @param int $group_id The data value |
||
| 132 | * @return void |
||
| 133 | */ |
||
| 134 | public function delete_group($group_id) |
||
| 135 | { |
||
| 136 | $statement = $this->db->prepare(" |
||
| 137 | UPDATE `group` |
||
| 138 | SET active = 0 |
||
| 139 | WHERE group_id = :group_id"); |
||
| 140 | $statement->bindValue(":group_id", $group_id, PDO::PARAM_INT); |
||
| 141 | $statement->execute(); |
||
| 142 | } |
||
| 143 | |||
| 144 | /** |
||
| 145 | * Flatten Group Hierarchy |
||
| 146 | * |
||
| 147 | * Flatten the group array, returning a single level array. |
||
| 148 | * |
||
| 149 | * @param object $group_hierarchy The multidimensional array |
||
| 150 | * @return array The single level array |
||
| 151 | */ |
||
| 152 | public function flatten_group_hierarchy($group_hierarchy) |
||
| 153 | { |
||
| 154 | $single_level_array = array(); |
||
| 155 | foreach ($group_hierarchy as $single_node) { |
||
| 156 | $descendants = false; |
||
| 157 | if (isset($single_node["descendants"]) && $single_node["descendants"]) { |
||
| 158 | $descendants = $single_node["descendants"]; |
||
| 159 | unset($single_node["descendants"]); |
||
| 160 | } |
||
| 161 | $single_level_array[] = $single_node; |
||
| 162 | if ($descendants) { |
||
| 163 | $single_level_array = array_merge($single_level_array, $this->flatten_group_hierarchy($descendants)); |
||
| 164 | } |
||
| 165 | } |
||
| 166 | return $single_level_array; |
||
| 167 | } |
||
| 168 | |||
| 169 | /** |
||
| 170 | * Get Descendants |
||
| 171 | * |
||
| 172 | * Query the database for a group's descendants. |
||
| 173 | * |
||
| 174 | * @param object $groups The array |
||
| 175 | * @param int $level The level |
||
| 176 | * @param string $indent_char The character used to indent in the resulting list |
||
| 177 | * @return array|bool The single level array |
||
| 178 | */ |
||
| 179 | public function get_descendants(&$groups, $level = 0, $indent_char = "-") |
||
| 180 | { |
||
| 181 | $level += 1; |
||
| 182 | $indent_string = ""; |
||
| 183 | for ($i=1;$i<=$level;$i++) { |
||
| 184 | $indent_string .= $indent_char; |
||
| 185 | } |
||
| 186 | foreach ($groups as &$single_group) { |
||
| 187 | $statement = $this->db->prepare(" |
||
| 188 | SELECT descendant AS group_id |
||
| 189 | ,name |
||
| 190 | ,abbreviation |
||
| 191 | ,'{$indent_string}' AS indent |
||
| 192 | FROM group_closure_table |
||
| 193 | LEFT JOIN `group` ON `group`.group_id = group_closure_table.descendant |
||
| 194 | WHERE ancestor = :group_id |
||
| 195 | AND ancestor != descendant |
||
| 196 | AND pathlength = 1 |
||
| 197 | GROUP BY descendant |
||
| 198 | ORDER BY name ASC"); |
||
| 199 | $statement->bindValue(":group_id", $single_group["group_id"], PDO::PARAM_INT); |
||
| 200 | $statement->execute(); |
||
| 201 | $descendants = $statement->fetchAll(PDO::FETCH_ASSOC); |
||
| 202 | |||
| 203 | if ($descendants) { |
||
| 204 | $single_group["descendants"] = $descendants; |
||
| 205 | $this->get_descendants($single_group["descendants"], $level, $indent_char); |
||
| 206 | } |
||
| 207 | } |
||
| 208 | } |
||
| 209 | |||
| 210 | /** |
||
| 211 | * Get Group Hierarchy |
||
| 212 | * |
||
| 213 | * Query the database for a group's hierarchy. |
||
| 214 | * |
||
| 215 | * @param string $indent_char The character used to indent in the resulting list |
||
| 216 | * @return array|bool The query result |
||
| 217 | */ |
||
| 218 | public function get_group_hierarchy($indent_char = "-") |
||
| 219 | { |
||
| 220 | // Get the root nodes. |
||
| 221 | $statement = $this->db->prepare(" |
||
| 222 | SELECT descendant AS group_id |
||
| 223 | ,COUNT(ancestor) AS total_parents |
||
| 224 | ,name |
||
| 225 | ,abbreviation |
||
| 226 | FROM group_closure_table |
||
| 227 | LEFT JOIN `group` ON `group`.group_id = group_closure_table.descendant |
||
| 228 | WHERE active = 1 |
||
| 229 | GROUP BY descendant |
||
| 230 | HAVING total_parents = 1 |
||
| 231 | ORDER BY name ASC"); |
||
| 232 | $statement->execute(); |
||
| 233 | $root_nodes = $statement->fetchAll(PDO::FETCH_ASSOC); |
||
| 234 | $this->get_descendants($root_nodes, 0, $indent_char); |
||
| 235 | return $root_nodes; |
||
| 236 | } |
||
| 237 | |||
| 238 | /** |
||
| 239 | * Get Group Record |
||
| 240 | * |
||
| 241 | * Query the database for one group and its parent group |
||
| 242 | * |
||
| 243 | * @param int $group_id The data value |
||
| 244 | * @return array|bool The guery result |
||
| 245 | */ |
||
| 246 | public function get_group_record($group_id) |
||
| 247 | { |
||
| 248 | $statement = $this->db->prepare(" |
||
| 249 | SELECT * |
||
| 250 | FROM `group` |
||
| 251 | WHERE active = 1 |
||
| 252 | AND group_id = :group_id"); |
||
| 253 | $statement->bindValue(":group_id", $group_id, PDO::PARAM_INT); |
||
| 254 | $statement->execute(); |
||
| 255 | $data = $statement->fetch(PDO::FETCH_ASSOC); |
||
| 256 | |||
| 257 | // Get the parent group. |
||
| 258 | $statement = $this->db->prepare(" |
||
| 259 | SELECT ancestor |
||
| 260 | FROM group_closure_table |
||
| 261 | WHERE descendant = :group_id |
||
| 262 | AND pathlength = 1"); |
||
| 263 | $statement->bindValue(":group_id", $group_id, PDO::PARAM_INT); |
||
| 264 | $statement->execute(); |
||
| 265 | $parent_group = $statement->fetch(PDO::FETCH_ASSOC); |
||
| 266 | $data["group_parent"] = $parent_group["ancestor"]; |
||
| 267 | |||
| 268 | return $data; |
||
| 269 | } |
||
| 270 | |||
| 271 | /** |
||
| 272 | * Get Groups |
||
| 273 | * |
||
| 274 | * Query the database for one group and its parent group |
||
| 275 | * |
||
| 276 | * @param array|int $group_ids The data value |
||
| 277 | * @return array|bool The query result |
||
| 278 | */ |
||
| 279 | public function get_groups($group_ids = false) |
||
| 280 | { |
||
| 281 | $pdo_params = array( |
||
| 282 | 1 //active |
||
| 283 | ); |
||
| 284 | $group_sql = ""; |
||
| 285 | if ($group_ids && is_array($group_ids)) { |
||
| 286 | $question_marks = array(); |
||
| 287 | foreach ($group_ids as $single_group_id) { |
||
| 288 | $pdo_params[] = $single_group_id; |
||
| 289 | $question_marks[] = "?"; |
||
| 290 | } |
||
| 291 | $group_sql = " AND group_id IN (" . implode(",", $question_marks) . ") "; |
||
| 292 | } elseif ($group_ids && is_numeric($group_ids)) { |
||
| 293 | $pdo_params[] = $group_ids; |
||
| 294 | $group_sql = " AND group_id = ? "; |
||
| 295 | } |
||
| 296 | |||
| 297 | $statement = $this->db->prepare(" |
||
| 298 | SELECT group_id |
||
| 299 | ,abbreviation |
||
| 300 | ,name |
||
| 301 | FROM `group` |
||
| 302 | WHERE active = ? |
||
| 303 | {$group_sql} |
||
| 304 | ORDER BY name"); |
||
| 305 | $statement->execute($pdo_params); |
||
| 306 | return $statement->fetchAll(PDO::FETCH_ASSOC); |
||
| 307 | } |
||
| 308 | |||
| 309 | /** |
||
| 310 | * Insert/Update Group |
||
| 311 | * |
||
| 312 | * Run a queries to insert and update groups in the database. |
||
| 313 | * |
||
| 314 | * @param array $data The data array |
||
| 315 | * @param array $group_id The data value |
||
| 316 | * @return array|bool The group id |
||
| 317 | */ |
||
| 318 | public function insert_update_group($data, $group_id = false) |
||
| 319 | { |
||
| 320 | $pdo_params = array( |
||
| 321 | $data["name"] |
||
| 322 | ,$data["abbreviation"] |
||
| 323 | ,$data["description"] |
||
| 324 | ,$data["address_1"] |
||
| 325 | ,$data["address_2"] |
||
| 326 | ,$data["city"] |
||
| 327 | ,$data["state"] |
||
| 328 | ,$data["zip"] |
||
| 329 | ,$_SESSION[$this->session_key]["user_account_id"] |
||
| 330 | ,1 |
||
| 331 | ); |
||
| 332 | |||
| 333 | if ($group_id) { |
||
| 334 | $pdo_params[] = $group_id; |
||
| 335 | $statement = $this->db->prepare(" |
||
| 336 | UPDATE `group` |
||
| 337 | SET name = ? |
||
| 338 | ,abbreviation = ? |
||
| 339 | ,description = ? |
||
| 340 | ,address_1 = ? |
||
| 341 | ,address_2 = ? |
||
| 342 | ,city = ? |
||
| 343 | ,state = ? |
||
| 344 | ,zip = ? |
||
| 345 | ,last_modified_user_account_id = ? |
||
| 346 | ,active = ? |
||
| 347 | ,last_modified = NOW() |
||
| 348 | WHERE group_id = ?"); |
||
| 349 | $statement->execute($pdo_params); |
||
| 350 | } else { |
||
| 351 | $pdo_params[] = $_SESSION[$this->session_key]["user_account_id"]; |
||
| 352 | $statement = $this->db->prepare(" |
||
| 353 | INSERT INTO `group` |
||
| 354 | (name |
||
| 355 | ,abbreviation |
||
| 356 | ,description |
||
| 357 | ,address_1 |
||
| 358 | ,address_2 |
||
| 359 | ,city |
||
| 360 | ,state |
||
| 361 | ,zip |
||
| 362 | ,last_modified_user_account_id |
||
| 363 | ,active |
||
| 364 | ,created_by_user_account_id |
||
| 365 | ,last_modified |
||
| 366 | ,date_created) |
||
| 367 | VALUES |
||
| 368 | (?,?,?,?,?,?,?,?,?,?,?,NOW(),NOW())"); |
||
| 369 | $statement->execute($pdo_params); |
||
| 370 | $group_id = $this->db->lastInsertId(); |
||
| 371 | } |
||
| 372 | |||
| 373 | // Update the groups closure table per Bill Karwin's SQL Antipatterns, Chapter 3. |
||
| 374 | // The pathlengh column refers to the jumps in between the ancestor and descendant - |
||
| 375 | // self-reference = 0, first child = 1, and so forth... |
||
| 376 | |||
| 377 | // First, check to see if we need to update or insert records. |
||
| 378 | $group_parent = (isset($data["group_parent"]) && $data["group_parent"]) ? $data["group_parent"] : false; |
||
| 379 | $statement = $this->db->prepare(" |
||
| 380 | SELECT * |
||
| 381 | FROM group_closure_table |
||
| 382 | WHERE descendant = :group_id"); |
||
| 383 | $statement->bindValue(":group_id", $group_id, PDO::PARAM_INT); |
||
| 384 | $statement->execute(); |
||
| 385 | $closure_check = $statement->fetchAll(PDO::FETCH_ASSOC); |
||
| 386 | |||
| 387 | if ($closure_check) { |
||
| 388 | // We need to move everything under it as well. |
||
| 389 | // First, detatch the node subtree... |
||
| 390 | $statement = $this->db->prepare(" |
||
| 391 | DELETE FROM group_closure_table |
||
| 392 | WHERE descendant IN ( |
||
| 393 | SELECT tmpdescendant.d FROM ( |
||
| 394 | SELECT descendant AS d FROM group_closure_table WHERE ancestor = :group_id |
||
| 395 | ) AS tmpdescendant |
||
| 396 | ) |
||
| 397 | AND ancestor IN ( |
||
| 398 | SELECT tmpancestor.a FROM ( |
||
| 399 | SELECT ancestor AS a FROM group_closure_table WHERE descendant = :group_id2 AND ancestor != descendant |
||
| 400 | ) AS tmpancestor |
||
| 401 | )"); |
||
| 402 | $statement->bindValue(":group_id", $group_id, PDO::PARAM_INT); |
||
| 403 | $statement->bindValue(":group_id2", $group_id, PDO::PARAM_INT); |
||
| 404 | $statement->execute(); |
||
| 405 | |||
| 406 | // Now, attach the subtree under the updated group. |
||
| 407 | $statement = $this->db->prepare(" |
||
| 408 | INSERT INTO group_closure_table |
||
| 409 | (ancestor, descendant, pathlength) |
||
| 410 | SELECT supertree.ancestor, subtree.descendant, subtree.pathlength+1 |
||
| 411 | FROM group_closure_table AS supertree |
||
| 412 | CROSS JOIN group_closure_table AS subtree |
||
| 413 | WHERE supertree.descendant = :new_parent |
||
| 414 | AND subtree.ancestor = :group_id"); |
||
| 415 | $statement->bindValue(":new_parent", $group_parent, PDO::PARAM_INT); |
||
| 416 | $statement->bindValue(":group_id", $group_id, PDO::PARAM_INT); |
||
| 417 | $statement->execute(); |
||
| 418 | } else { |
||
| 419 | // Just insert the leaf node. |
||
| 420 | $statement = $this->db->prepare(" |
||
| 421 | INSERT INTO group_closure_table |
||
| 422 | (ancestor, descendant, pathlength) |
||
| 423 | SELECT gct.ancestor, :group_id, pathlength+1 |
||
| 424 | FROM group_closure_table AS gct |
||
| 425 | WHERE gct.descendant = :parent_group |
||
| 426 | UNION ALL |
||
| 427 | SELECT :group_id2, :group_id3,0"); |
||
| 428 | $statement->bindValue(":group_id", $group_id, PDO::PARAM_INT); |
||
| 429 | $statement->bindValue(":parent_group", $group_parent, PDO::PARAM_INT); |
||
| 430 | $statement->bindValue(":group_id2", $group_id, PDO::PARAM_INT); |
||
| 431 | $statement->bindValue(":group_id3", $group_id, PDO::PARAM_INT); |
||
| 432 | $statement->execute(); |
||
| 433 | } |
||
| 434 | |||
| 435 | return $group_id; |
||
| 436 | } |
||
| 437 | |||
| 438 | /** |
||
| 439 | * Get Admin Info From Group List |
||
| 440 | * |
||
| 441 | * Query for all of the admins and editors for the supplied groups, |
||
| 442 | * as well as all of the admin/editors for their parent groups. |
||
| 443 | * |
||
| 444 | * @param array $group_list The data array |
||
| 445 | * @return array|bool The group id |
||
| 446 | */ |
||
| 447 | public function get_admin_info_from_group_list($group_list) |
||
| 448 | { |
||
| 449 | $statement = $this->db->prepare(" |
||
| 450 | SELECT user_account.user_account_email |
||
| 451 | ,user_account.first_name |
||
| 452 | ,user_account.last_name |
||
| 453 | FROM user_account |
||
| 454 | LEFT JOIN user_account_groups ON user_account_groups.user_account_id = user_account.user_account_id |
||
| 455 | LEFT JOIN group_closure_table ON group_closure_table.ancestor = user_account_groups.group_id |
||
| 456 | WHERE user_account_groups.role_id IN (4,1) |
||
| 457 | AND (user_account_groups.group_id IN($group_list) |
||
| 458 | OR group_closure_table.descendant IN (" . $group_list . ")) |
||
| 459 | GROUP BY user_account.user_account_email"); |
||
| 460 | $statement->execute(); |
||
| 461 | return $statement->fetchAll(PDO::FETCH_ASSOC); |
||
| 462 | } |
||
| 463 | } |
||
| 464 |
In PHP, under loose comparison (like
==, or!=, orswitchconditions), values of different types might be equal.For
stringvalues, the empty string''is a special case, in particular the following results might be unexpected: