aviat4ion /
Query
Complex classes like QueryBuilder often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use QueryBuilder, and based on these observations, apply Extract Interface, too.
| 1 | <?php declare(strict_types=1); |
||
| 24 | class QueryBuilder implements QueryBuilderInterface { |
||
| 25 | |||
| 26 | /** |
||
| 27 | * Convenience property for connection management |
||
| 28 | * @var string |
||
| 29 | */ |
||
| 30 | public $connName = ''; |
||
| 31 | |||
| 32 | /** |
||
| 33 | * List of queries executed |
||
| 34 | * @var array |
||
| 35 | */ |
||
| 36 | public $queries = [ |
||
| 37 | 'total_time' => 0 |
||
| 38 | ]; |
||
| 39 | |||
| 40 | /** |
||
| 41 | * Whether to do only an explain on the query |
||
| 42 | * @var boolean |
||
| 43 | */ |
||
| 44 | protected $explain = FALSE; |
||
| 45 | |||
| 46 | /** |
||
| 47 | * The current database driver |
||
| 48 | * @var DriverInterface |
||
| 49 | */ |
||
| 50 | public $driver; |
||
| 51 | |||
| 52 | /** |
||
| 53 | * Query parser class instance |
||
| 54 | * @var QueryParser |
||
| 55 | */ |
||
| 56 | protected $parser; |
||
| 57 | |||
| 58 | /** |
||
| 59 | * Query Builder state |
||
| 60 | * @var State |
||
| 61 | */ |
||
| 62 | protected $state; |
||
| 63 | |||
| 64 | // -------------------------------------------------------------------------- |
||
| 65 | // ! Methods |
||
| 66 | // -------------------------------------------------------------------------- |
||
| 67 | |||
| 68 | /** |
||
| 69 | * Constructor |
||
| 70 | * |
||
| 71 | * @param DriverInterface $driver |
||
| 72 | * @param QueryParser $parser |
||
| 73 | */ |
||
| 74 | public function __construct(DriverInterface $driver, QueryParser $parser) |
||
| 83 | |||
| 84 | /** |
||
| 85 | * Destructor |
||
| 86 | * @codeCoverageIgnore |
||
| 87 | */ |
||
| 88 | public function __destruct() |
||
| 92 | |||
| 93 | /** |
||
| 94 | * Calls a function further down the inheritance chain |
||
| 95 | * |
||
| 96 | * @param string $name |
||
| 97 | * @param array $params |
||
| 98 | * @return mixed |
||
| 99 | * @throws BadMethodCallException |
||
| 100 | */ |
||
| 101 | public function __call(string $name, array $params) |
||
| 110 | |||
| 111 | // -------------------------------------------------------------------------- |
||
| 112 | // ! Select Queries |
||
| 113 | // -------------------------------------------------------------------------- |
||
| 114 | |||
| 115 | /** |
||
| 116 | * Specifies rows to select in a query |
||
| 117 | * |
||
| 118 | * @param string $fields |
||
| 119 | * @return QueryBuilderInterface |
||
| 120 | */ |
||
| 121 | public function select(string $fields): QueryBuilderInterface |
||
| 155 | |||
| 156 | /** |
||
| 157 | * Selects the maximum value of a field from a query |
||
| 158 | * |
||
| 159 | * @param string $field |
||
| 160 | * @param string|bool $as |
||
| 161 | * @return QueryBuilderInterface |
||
| 162 | */ |
||
| 163 | public function selectMax(string $field, $as=FALSE): QueryBuilderInterface |
||
| 169 | |||
| 170 | /** |
||
| 171 | * Selects the minimum value of a field from a query |
||
| 172 | * |
||
| 173 | * @param string $field |
||
| 174 | * @param string|bool $as |
||
| 175 | * @return QueryBuilderInterface |
||
| 176 | */ |
||
| 177 | public function selectMin(string $field, $as=FALSE): QueryBuilderInterface |
||
| 183 | |||
| 184 | /** |
||
| 185 | * Selects the average value of a field from a query |
||
| 186 | * |
||
| 187 | * @param string $field |
||
| 188 | * @param string|bool $as |
||
| 189 | * @return QueryBuilderInterface |
||
| 190 | */ |
||
| 191 | public function selectAvg(string $field, $as=FALSE): QueryBuilderInterface |
||
| 197 | |||
| 198 | /** |
||
| 199 | * Selects the sum of a field from a query |
||
| 200 | * |
||
| 201 | * @param string $field |
||
| 202 | * @param string|bool $as |
||
| 203 | * @return QueryBuilderInterface |
||
| 204 | */ |
||
| 205 | public function selectSum(string $field, $as=FALSE): QueryBuilderInterface |
||
| 211 | |||
| 212 | /** |
||
| 213 | * Adds the 'distinct' keyword to a query |
||
| 214 | * |
||
| 215 | * @return QueryBuilderInterface |
||
| 216 | */ |
||
| 217 | public function distinct(): QueryBuilderInterface |
||
| 223 | |||
| 224 | /** |
||
| 225 | * Tell the database to give you the query plan instead of result set |
||
| 226 | * |
||
| 227 | * @return QueryBuilderInterface |
||
| 228 | */ |
||
| 229 | public function explain(): QueryBuilderInterface |
||
| 234 | |||
| 235 | /** |
||
| 236 | * Specify the database table to select from |
||
| 237 | * |
||
| 238 | * @param string $tblname |
||
| 239 | * @return QueryBuilderInterface |
||
| 240 | */ |
||
| 241 | public function from(string $tblname): QueryBuilderInterface |
||
| 256 | |||
| 257 | // -------------------------------------------------------------------------- |
||
| 258 | // ! 'Like' methods |
||
| 259 | // -------------------------------------------------------------------------- |
||
| 260 | |||
| 261 | /** |
||
| 262 | * Creates a Like clause in the sql statement |
||
| 263 | * |
||
| 264 | * @param string $field |
||
| 265 | * @param mixed $val |
||
| 266 | * @param string $pos |
||
| 267 | * @return QueryBuilderInterface |
||
| 268 | */ |
||
| 269 | public function like(string $field, $val, string $pos='both'): QueryBuilderInterface |
||
| 273 | |||
| 274 | /** |
||
| 275 | * Generates an OR Like clause |
||
| 276 | * |
||
| 277 | * @param string $field |
||
| 278 | * @param mixed $val |
||
| 279 | * @param string $pos |
||
| 280 | * @return QueryBuilderInterface |
||
| 281 | */ |
||
| 282 | public function orLike(string $field, $val, string $pos='both'): QueryBuilderInterface |
||
| 286 | |||
| 287 | /** |
||
| 288 | * Generates a NOT LIKE clause |
||
| 289 | * |
||
| 290 | * @param string $field |
||
| 291 | * @param mixed $val |
||
| 292 | * @param string $pos |
||
| 293 | * @return QueryBuilderInterface |
||
| 294 | */ |
||
| 295 | public function notLike(string $field, $val, string $pos='both'): QueryBuilderInterface |
||
| 299 | |||
| 300 | /** |
||
| 301 | * Generates a OR NOT LIKE clause |
||
| 302 | * |
||
| 303 | * @param string $field |
||
| 304 | * @param mixed $val |
||
| 305 | * @param string $pos |
||
| 306 | * @return QueryBuilderInterface |
||
| 307 | */ |
||
| 308 | public function orNotLike(string $field, $val, string $pos='both'): QueryBuilderInterface |
||
| 312 | |||
| 313 | // -------------------------------------------------------------------------- |
||
| 314 | // ! Having methods |
||
| 315 | // -------------------------------------------------------------------------- |
||
| 316 | |||
| 317 | /** |
||
| 318 | * Generates a 'Having' clause |
||
| 319 | * |
||
| 320 | * @param mixed $key |
||
| 321 | * @param mixed $val |
||
| 322 | * @return QueryBuilderInterface |
||
| 323 | */ |
||
| 324 | public function having($key, $val=[]): QueryBuilderInterface |
||
| 328 | |||
| 329 | /** |
||
| 330 | * Generates a 'Having' clause prefixed with 'OR' |
||
| 331 | * |
||
| 332 | * @param mixed $key |
||
| 333 | * @param mixed $val |
||
| 334 | * @return QueryBuilderInterface |
||
| 335 | */ |
||
| 336 | public function orHaving($key, $val=[]): QueryBuilderInterface |
||
| 340 | |||
| 341 | // -------------------------------------------------------------------------- |
||
| 342 | // ! 'Where' methods |
||
| 343 | // -------------------------------------------------------------------------- |
||
| 344 | |||
| 345 | /** |
||
| 346 | * Specify condition(s) in the where clause of a query |
||
| 347 | * Note: this function works with key / value, or a |
||
| 348 | * passed array with key / value pairs |
||
| 349 | * |
||
| 350 | * @param mixed $key |
||
| 351 | * @param mixed $val |
||
| 352 | * @param mixed $escape |
||
| 353 | * @return QueryBuilderInterface |
||
| 354 | */ |
||
| 355 | public function where($key, $val=[], $escape=NULL): QueryBuilderInterface |
||
| 359 | |||
| 360 | /** |
||
| 361 | * Where clause prefixed with "OR" |
||
| 362 | * |
||
| 363 | * @param string $key |
||
| 364 | * @param mixed $val |
||
| 365 | * @return QueryBuilderInterface |
||
| 366 | */ |
||
| 367 | public function orWhere($key, $val=[]): QueryBuilderInterface |
||
| 371 | |||
| 372 | /** |
||
| 373 | * Where clause with 'IN' statement |
||
| 374 | * |
||
| 375 | * @param mixed $field |
||
| 376 | * @param mixed $val |
||
| 377 | * @return QueryBuilderInterface |
||
| 378 | */ |
||
| 379 | public function whereIn($field, $val=[]): QueryBuilderInterface |
||
| 383 | |||
| 384 | /** |
||
| 385 | * Where in statement prefixed with "or" |
||
| 386 | * |
||
| 387 | * @param string $field |
||
| 388 | * @param mixed $val |
||
| 389 | * @return QueryBuilderInterface |
||
| 390 | */ |
||
| 391 | public function orWhereIn($field, $val=[]): QueryBuilderInterface |
||
| 395 | |||
| 396 | /** |
||
| 397 | * WHERE NOT IN (FOO) clause |
||
| 398 | * |
||
| 399 | * @param string $field |
||
| 400 | * @param mixed $val |
||
| 401 | * @return QueryBuilderInterface |
||
| 402 | */ |
||
| 403 | public function whereNotIn($field, $val=[]): QueryBuilderInterface |
||
| 407 | |||
| 408 | /** |
||
| 409 | * OR WHERE NOT IN (FOO) clause |
||
| 410 | * |
||
| 411 | * @param string $field |
||
| 412 | * @param mixed $val |
||
| 413 | * @return QueryBuilderInterface |
||
| 414 | */ |
||
| 415 | public function orWhereNotIn($field, $val=[]): QueryBuilderInterface |
||
| 419 | |||
| 420 | // -------------------------------------------------------------------------- |
||
| 421 | // ! Other Query Modifier methods |
||
| 422 | // -------------------------------------------------------------------------- |
||
| 423 | |||
| 424 | /** |
||
| 425 | * Sets values for inserts / updates / deletes |
||
| 426 | * |
||
| 427 | * @param mixed $key |
||
| 428 | * @param mixed $val |
||
| 429 | * @return QueryBuilderInterface |
||
| 430 | */ |
||
| 431 | public function set($key, $val = NULL): QueryBuilderInterface |
||
| 462 | |||
| 463 | /** |
||
| 464 | * Creates a join phrase in a compiled query |
||
| 465 | * |
||
| 466 | * @param string $table |
||
| 467 | * @param string $condition |
||
| 468 | * @param string $type |
||
| 469 | * @return QueryBuilderInterface |
||
| 470 | */ |
||
| 471 | public function join(string $table, string $condition, string $type=''): QueryBuilderInterface |
||
| 487 | |||
| 488 | /** |
||
| 489 | * Group the results by the selected field(s) |
||
| 490 | * |
||
| 491 | * @param mixed $field |
||
| 492 | * @return QueryBuilderInterface |
||
| 493 | */ |
||
| 494 | public function groupBy($field): QueryBuilderInterface |
||
| 512 | |||
| 513 | /** |
||
| 514 | * Order the results by the selected field(s) |
||
| 515 | * |
||
| 516 | * @param string $field |
||
| 517 | * @param string $type |
||
| 518 | * @return QueryBuilderInterface |
||
| 519 | */ |
||
| 520 | public function orderBy(string $field, string $type=''): QueryBuilderInterface |
||
| 551 | |||
| 552 | /** |
||
| 553 | * Set a limit on the current sql statement |
||
| 554 | * |
||
| 555 | * @param int $limit |
||
| 556 | * @param int|bool $offset |
||
| 557 | * @return QueryBuilderInterface |
||
| 558 | */ |
||
| 559 | public function limit(int $limit, $offset=FALSE): QueryBuilderInterface |
||
| 566 | |||
| 567 | // -------------------------------------------------------------------------- |
||
| 568 | // ! Query Grouping Methods |
||
| 569 | // -------------------------------------------------------------------------- |
||
| 570 | |||
| 571 | /** |
||
| 572 | * Adds a paren to the current query for query grouping |
||
| 573 | * |
||
| 574 | * @return QueryBuilderInterface |
||
| 575 | */ |
||
| 576 | public function groupStart(): QueryBuilderInterface |
||
| 584 | |||
| 585 | /** |
||
| 586 | * Adds a paren to the current query for query grouping, |
||
| 587 | * prefixed with 'NOT' |
||
| 588 | * |
||
| 589 | * @return QueryBuilderInterface |
||
| 590 | */ |
||
| 591 | public function notGroupStart(): QueryBuilderInterface |
||
| 599 | |||
| 600 | /** |
||
| 601 | * Adds a paren to the current query for query grouping, |
||
| 602 | * prefixed with 'OR' |
||
| 603 | * |
||
| 604 | * @return QueryBuilderInterface |
||
| 605 | */ |
||
| 606 | public function orGroupStart(): QueryBuilderInterface |
||
| 612 | |||
| 613 | /** |
||
| 614 | * Adds a paren to the current query for query grouping, |
||
| 615 | * prefixed with 'OR NOT' |
||
| 616 | * |
||
| 617 | * @return QueryBuilderInterface |
||
| 618 | */ |
||
| 619 | public function orNotGroupStart(): QueryBuilderInterface |
||
| 625 | |||
| 626 | /** |
||
| 627 | * Ends a query group |
||
| 628 | * |
||
| 629 | * @return QueryBuilderInterface |
||
| 630 | */ |
||
| 631 | public function groupEnd(): QueryBuilderInterface |
||
| 637 | |||
| 638 | // -------------------------------------------------------------------------- |
||
| 639 | // ! Query execution methods |
||
| 640 | // -------------------------------------------------------------------------- |
||
| 641 | |||
| 642 | /** |
||
| 643 | * Select and retrieve all records from the current table, and/or |
||
| 644 | * execute current compiled query |
||
| 645 | * |
||
| 646 | * @param string $table |
||
| 647 | * @param int|bool $limit |
||
| 648 | * @param int|bool $offset |
||
| 649 | * @return PDOStatement |
||
| 650 | */ |
||
| 651 | public function get(string $table='', $limit=FALSE, $offset=FALSE): ?PDOStatement |
||
| 667 | |||
| 668 | /** |
||
| 669 | * Convenience method for get() with a where clause |
||
| 670 | * |
||
| 671 | * @param string $table |
||
| 672 | * @param mixed $where |
||
| 673 | * @param int|bool $limit |
||
| 674 | * @param int|bool $offset |
||
| 675 | * @return PDOStatement |
||
| 676 | */ |
||
| 677 | public function getWhere(string $table, $where=[], $limit=FALSE, $offset=FALSE): ?PDOStatement |
||
| 685 | |||
| 686 | /** |
||
| 687 | * Retrieve the number of rows in the selected table |
||
| 688 | * |
||
| 689 | * @param string $table |
||
| 690 | * @return int |
||
| 691 | */ |
||
| 692 | public function countAll(string $table): int |
||
| 698 | |||
| 699 | /** |
||
| 700 | * Retrieve the number of results for the generated query - used |
||
| 701 | * in place of the get() method |
||
| 702 | * |
||
| 703 | * @param string $table |
||
| 704 | * @param boolean $reset |
||
| 705 | * @return int |
||
| 706 | */ |
||
| 707 | public function countAllResults(string $table='', bool $reset = TRUE): int |
||
| 720 | |||
| 721 | /** |
||
| 722 | * Creates an insert clause, and executes it |
||
| 723 | * |
||
| 724 | * @param string $table |
||
| 725 | * @param mixed $data |
||
| 726 | * @return PDOStatement |
||
| 727 | */ |
||
| 728 | public function insert(string $table, $data=[]): ?PDOStatement |
||
| 737 | |||
| 738 | /** |
||
| 739 | * Creates and executes a batch insertion query |
||
| 740 | * |
||
| 741 | * @param string $table |
||
| 742 | * @param array $data |
||
| 743 | * @return PDOStatement |
||
| 744 | */ |
||
| 745 | public function insertBatch(string $table, $data=[]): ?PDOStatement |
||
| 754 | |||
| 755 | /** |
||
| 756 | * Creates an update clause, and executes it |
||
| 757 | * |
||
| 758 | * @param string $table |
||
| 759 | * @param mixed $data |
||
| 760 | * @return PDOStatement |
||
| 761 | */ |
||
| 762 | public function update(string $table, $data=[]): PDOStatement |
||
| 771 | |||
| 772 | /** |
||
| 773 | * Creates a batch update, and executes it. |
||
| 774 | * Returns the number of affected rows |
||
| 775 | * |
||
| 776 | * @param string $table |
||
| 777 | * @param array|object $data |
||
| 778 | * @param string $where |
||
| 779 | * @return PDOStatement|null |
||
| 780 | */ |
||
| 781 | public function updateBatch(string $table, $data, $where): ?PDOStatement |
||
| 790 | |||
| 791 | /** |
||
| 792 | * Insertion with automatic overwrite, rather than attempted duplication |
||
| 793 | * |
||
| 794 | * @param string $table |
||
| 795 | * @param array $data |
||
| 796 | * @return \PDOStatement|null |
||
| 797 | */ |
||
| 798 | public function replace(string $table, $data=[]): ?PDOStatement |
||
| 807 | |||
| 808 | /** |
||
| 809 | * Deletes data from a table |
||
| 810 | * |
||
| 811 | * @param string $table |
||
| 812 | * @param mixed $where |
||
| 813 | * @return PDOStatement |
||
| 814 | */ |
||
| 815 | public function delete(string $table, $where=''): ?PDOStatement |
||
| 825 | |||
| 826 | // -------------------------------------------------------------------------- |
||
| 827 | // ! SQL Returning Methods |
||
| 828 | // -------------------------------------------------------------------------- |
||
| 829 | |||
| 830 | /** |
||
| 831 | * Returns the generated 'select' sql query |
||
| 832 | * |
||
| 833 | * @param string $table |
||
| 834 | * @param bool $reset |
||
| 835 | * @return string |
||
| 836 | */ |
||
| 837 | public function getCompiledSelect(string $table='', bool $reset=TRUE): string |
||
| 847 | |||
| 848 | /** |
||
| 849 | * Returns the generated 'insert' sql query |
||
| 850 | * |
||
| 851 | * @param string $table |
||
| 852 | * @param bool $reset |
||
| 853 | * @return string |
||
| 854 | */ |
||
| 855 | public function getCompiledInsert(string $table, bool $reset=TRUE): string |
||
| 859 | |||
| 860 | /** |
||
| 861 | * Returns the generated 'update' sql query |
||
| 862 | * |
||
| 863 | * @param string $table |
||
| 864 | * @param bool $reset |
||
| 865 | * @return string |
||
| 866 | */ |
||
| 867 | public function getCompiledUpdate(string $table='', bool $reset=TRUE): string |
||
| 871 | |||
| 872 | /** |
||
| 873 | * Returns the generated 'delete' sql query |
||
| 874 | * |
||
| 875 | * @param string $table |
||
| 876 | * @param bool $reset |
||
| 877 | * @return string |
||
| 878 | */ |
||
| 879 | public function getCompiledDelete(string $table='', bool $reset=TRUE): string |
||
| 883 | |||
| 884 | // -------------------------------------------------------------------------- |
||
| 885 | // ! Miscellaneous Methods |
||
| 886 | // -------------------------------------------------------------------------- |
||
| 887 | |||
| 888 | /** |
||
| 889 | * Clear out the class variables, so the next query can be run |
||
| 890 | * |
||
| 891 | * @return void |
||
| 892 | */ |
||
| 893 | public function resetQuery(): void |
||
| 898 | |||
| 899 | /** |
||
| 900 | * Method to simplify select_ methods |
||
| 901 | * |
||
| 902 | * @param string $field |
||
| 903 | * @param string|bool $as |
||
| 904 | * @return string |
||
| 905 | */ |
||
| 906 | protected function _select(string $field, $as = FALSE): string |
||
| 919 | |||
| 920 | /** |
||
| 921 | * Helper function for returning sql strings |
||
| 922 | * |
||
| 923 | * @param string $type |
||
| 924 | * @param string $table |
||
| 925 | * @param bool $reset |
||
| 926 | * @return string |
||
| 927 | */ |
||
| 928 | protected function _getCompile(string $type, string $table, bool $reset): string |
||
| 940 | |||
| 941 | /** |
||
| 942 | * Simplify 'like' methods |
||
| 943 | * |
||
| 944 | * @param string $field |
||
| 945 | * @param mixed $val |
||
| 946 | * @param string $pos |
||
| 947 | * @param string $like |
||
| 948 | * @param string $conj |
||
| 949 | * @return self |
||
| 950 | */ |
||
| 951 | protected function _like(string $field, $val, string $pos, string $like='LIKE', string $conj='AND'): self |
||
| 979 | |||
| 980 | /** |
||
| 981 | * Simplify building having clauses |
||
| 982 | * |
||
| 983 | * @param mixed $key |
||
| 984 | * @param mixed $values |
||
| 985 | * @param string $conj |
||
| 986 | * @return self |
||
| 987 | */ |
||
| 988 | protected function _having($key, $values=[], string $conj='AND'): self |
||
| 1015 | |||
| 1016 | /** |
||
| 1017 | * Do all the redundant stuff for where/having type methods |
||
| 1018 | * |
||
| 1019 | * @param mixed $key |
||
| 1020 | * @param mixed $val |
||
| 1021 | * @return array |
||
| 1022 | */ |
||
| 1023 | protected function _where($key, $val=[]): array |
||
| 1045 | |||
| 1046 | /** |
||
| 1047 | * Simplify generating where string |
||
| 1048 | * |
||
| 1049 | * @param mixed $key |
||
| 1050 | * @param mixed $values |
||
| 1051 | * @param string $defaultConj |
||
| 1052 | * @return self |
||
| 1053 | */ |
||
| 1054 | protected function _whereString($key, $values=[], string $defaultConj='AND'): self |
||
| 1091 | |||
| 1092 | /** |
||
| 1093 | * Simplify where_in methods |
||
| 1094 | * |
||
| 1095 | * @param mixed $key |
||
| 1096 | * @param mixed $val |
||
| 1097 | * @param string $in - The (not) in fragment |
||
| 1098 | * @param string $conj - The where in conjunction |
||
| 1099 | * @return self |
||
| 1100 | */ |
||
| 1101 | protected function _whereIn($key, $val=[], string $in='IN', string $conj='AND'): self |
||
| 1114 | |||
| 1115 | /** |
||
| 1116 | * Executes the compiled query |
||
| 1117 | * |
||
| 1118 | * @param string $type |
||
| 1119 | * @param string $table |
||
| 1120 | * @param string $sql |
||
| 1121 | * @param array|null $vals |
||
| 1122 | * @param boolean $reset |
||
| 1123 | * @return PDOStatement |
||
| 1124 | */ |
||
| 1125 | protected function _run(string $type, string $table, string $sql=NULL, array $vals=NULL, bool $reset=TRUE): PDOStatement |
||
| 1157 | |||
| 1158 | /** |
||
| 1159 | * Convert the prepared statement into readable sql |
||
| 1160 | * |
||
| 1161 | * @param array $vals |
||
| 1162 | * @param string $sql |
||
| 1163 | * @param int $totalTime |
||
| 1164 | * @return void |
||
| 1165 | */ |
||
| 1166 | protected function _appendQuery(array $vals = NULL, string $sql, int $totalTime) |
||
| 1194 | |||
| 1195 | /** |
||
| 1196 | * Sub-method for generating sql strings |
||
| 1197 | * |
||
| 1198 | * @param string $type |
||
| 1199 | * @param string $table |
||
| 1200 | * @return string |
||
| 1201 | */ |
||
| 1202 | protected function _compileType(string $type='', string $table=''): string |
||
| 1247 | |||
| 1248 | /** |
||
| 1249 | * String together the sql statements for sending to the db |
||
| 1250 | * |
||
| 1251 | * @param string $type |
||
| 1252 | * @param string $table |
||
| 1253 | * @return string |
||
| 1254 | */ |
||
| 1255 | protected function _compile(string $type='', string $table=''): string |
||
| 1301 | } |
||
| 1302 |
Loading history...
This check looks at variables that are passed out again to other methods.
If the outgoing method call has stricter type requirements than the method itself, an issue is raised.
An additional type check may prevent trouble.