@@ -63,11 +63,23 @@ discard block |
||
| 63 | 63 | $rs = null; |
| 64 | 64 | |
| 65 | 65 | // Check parameter ranges |
| 66 | - if($symsize < 0 || $symsize > 8) return $rs; |
|
| 67 | - if($fcr < 0 || $fcr >= (1<<$symsize)) return $rs; |
|
| 68 | - if($prim <= 0 || $prim >= (1<<$symsize)) return $rs; |
|
| 69 | - if($nroots < 0 || $nroots >= (1<<$symsize)) return $rs; // Can't have more roots than symbol values! |
|
| 70 | - if($pad < 0 || $pad >= ((1<<$symsize) -1 - $nroots)) return $rs; // Too much padding |
|
| 66 | + if($symsize < 0 || $symsize > 8) { |
|
| 67 | + return $rs; |
|
| 68 | + } |
|
| 69 | + if($fcr < 0 || $fcr >= (1<<$symsize)) { |
|
| 70 | + return $rs; |
|
| 71 | + } |
|
| 72 | + if($prim <= 0 || $prim >= (1<<$symsize)) { |
|
| 73 | + return $rs; |
|
| 74 | + } |
|
| 75 | + if($nroots < 0 || $nroots >= (1<<$symsize)) { |
|
| 76 | + return $rs; |
|
| 77 | + } |
|
| 78 | + // Can't have more roots than symbol values! |
|
| 79 | + if($pad < 0 || $pad >= ((1<<$symsize) -1 - $nroots)) { |
|
| 80 | + return $rs; |
|
| 81 | + } |
|
| 82 | + // Too much padding |
|
| 71 | 83 | |
| 72 | 84 | $rs = new QRrsItem(); |
| 73 | 85 | $rs->mm = $symsize; |
@@ -133,8 +145,9 @@ discard block |
||
| 133 | 145 | } |
| 134 | 146 | |
| 135 | 147 | // convert rs->genpoly[] to index form for quicker encoding |
| 136 | - for ($i = 0; $i <= $nroots; $i++) |
|
| 137 | - $rs->genpoly[$i] = $rs->index_of[$rs->genpoly[$i]]; |
|
| 148 | + for ($i = 0; $i <= $nroots; $i++) { |
|
| 149 | + $rs->genpoly[$i] = $rs->index_of[$rs->genpoly[$i]]; |
|
| 150 | + } |
|
| 138 | 151 | |
| 139 | 152 | return $rs; |
| 140 | 153 | } |
@@ -192,12 +205,24 @@ discard block |
||
| 192 | 205 | public static function init_rs($symsize, $gfpoly, $fcr, $prim, $nroots, $pad) |
| 193 | 206 | {
|
| 194 | 207 | foreach(self::$items as $rs) {
|
| 195 | - if($rs->pad != $pad) continue; |
|
| 196 | - if($rs->nroots != $nroots) continue; |
|
| 197 | - if($rs->mm != $symsize) continue; |
|
| 198 | - if($rs->gfpoly != $gfpoly) continue; |
|
| 199 | - if($rs->fcr != $fcr) continue; |
|
| 200 | - if($rs->prim != $prim) continue; |
|
| 208 | + if($rs->pad != $pad) { |
|
| 209 | + continue; |
|
| 210 | + } |
|
| 211 | + if($rs->nroots != $nroots) { |
|
| 212 | + continue; |
|
| 213 | + } |
|
| 214 | + if($rs->mm != $symsize) { |
|
| 215 | + continue; |
|
| 216 | + } |
|
| 217 | + if($rs->gfpoly != $gfpoly) { |
|
| 218 | + continue; |
|
| 219 | + } |
|
| 220 | + if($rs->fcr != $fcr) { |
|
| 221 | + continue; |
|
| 222 | + } |
|
| 223 | + if($rs->prim != $prim) { |
|
| 224 | + continue; |
|
| 225 | + } |
|
| 201 | 226 | |
| 202 | 227 | return $rs; |
| 203 | 228 | } |
@@ -108,13 +108,15 @@ discard block |
||
| 108 | 108 | //---------------------------------------------------------------------- |
| 109 | 109 | public function appendNum($bits, $num) |
| 110 | 110 | {
|
| 111 | - if ($bits == 0) |
|
| 112 | - return 0; |
|
| 111 | + if ($bits == 0) { |
|
| 112 | + return 0; |
|
| 113 | + } |
|
| 113 | 114 | |
| 114 | 115 | $b = QRbitstream::newFromNum($bits, $num); |
| 115 | 116 | |
| 116 | - if(is_null($b)) |
|
| 117 | - return -1; |
|
| 117 | + if(is_null($b)) { |
|
| 118 | + return -1; |
|
| 119 | + } |
|
| 118 | 120 | |
| 119 | 121 | $ret = $this->append($b); |
| 120 | 122 | unset($b); |
@@ -125,13 +127,15 @@ discard block |
||
| 125 | 127 | //---------------------------------------------------------------------- |
| 126 | 128 | public function appendBytes($size, $data) |
| 127 | 129 | {
|
| 128 | - if ($size == 0) |
|
| 129 | - return 0; |
|
| 130 | + if ($size == 0) { |
|
| 131 | + return 0; |
|
| 132 | + } |
|
| 130 | 133 | |
| 131 | 134 | $b = QRbitstream::newFromBytes($size, $data); |
| 132 | 135 | |
| 133 | - if(is_null($b)) |
|
| 134 | - return -1; |
|
| 136 | + if(is_null($b)) { |
|
| 137 | + return -1; |
|
| 138 | + } |
|
| 135 | 139 | |
| 136 | 140 | $ret = $this->append($b); |
| 137 | 141 | unset($b); |
@@ -176,8 +176,9 @@ discard block |
||
| 176 | 176 | {
|
| 177 | 177 | $barcode_array = array(); |
| 178 | 178 | |
| 179 | - if (!is_array($mode)) |
|
| 180 | - $mode = explode(',', $mode);
|
|
| 179 | + if (!is_array($mode)) { |
|
| 180 | + $mode = explode(',', $mode); |
|
| 181 | + } |
|
| 181 | 182 | |
| 182 | 183 | $eccLevel = 'L'; |
| 183 | 184 | |
@@ -194,8 +195,9 @@ discard block |
||
| 194 | 195 | |
| 195 | 196 | foreach ($qrTab as $line) {
|
| 196 | 197 | $arrAdd = array(); |
| 197 | - foreach(str_split($line) as $char) |
|
| 198 | - $arrAdd[] = ($char=='1')?1:0; |
|
| 198 | + foreach(str_split($line) as $char) { |
|
| 199 | + $arrAdd[] = ($char=='1')?1:0; |
|
| 200 | + } |
|
| 199 | 201 | $barcode_array['bcode'][] = $arrAdd; |
| 200 | 202 | } |
| 201 | 203 | |
@@ -223,8 +225,9 @@ discard block |
||
| 223 | 225 | |
| 224 | 226 | $width = count($frame); |
| 225 | 227 | $bitMask = array_fill(0, $width, array_fill(0, $width, 0)); |
| 226 | - for ($maskNo = 0; $maskNo < 8; $maskNo++) |
|
| 227 | - $mask->makeMaskNo($maskNo, $width, $frame, $bitMask, true); |
|
| 228 | + for ($maskNo = 0; $maskNo < 8; $maskNo++) { |
|
| 229 | + $mask->makeMaskNo($maskNo, $width, $frame, $bitMask, true); |
|
| 230 | + } |
|
| 228 | 231 | } |
| 229 | 232 | |
| 230 | 233 | QRtools::markTime('after_build_cache');
|
@@ -261,8 +264,9 @@ discard block |
||
| 261 | 264 | list($usec, $sec) = explode(" ", microtime());
|
| 262 | 265 | $time = ((float)$usec + (float)$sec); |
| 263 | 266 | |
| 264 | - if (!isset($GLOBALS['qr_time_bench'])) |
|
| 265 | - $GLOBALS['qr_time_bench'] = array(); |
|
| 267 | + if (!isset($GLOBALS['qr_time_bench'])) { |
|
| 268 | + $GLOBALS['qr_time_bench'] = array(); |
|
| 269 | + } |
|
| 266 | 270 | |
| 267 | 271 | $GLOBALS['qr_time_bench'][$markerId] = $time; |
| 268 | 272 | } |
@@ -427,8 +431,9 @@ discard block |
||
| 427 | 431 | |
| 428 | 432 | for($i=1; $i<= QRSPEC_VERSION_MAX; $i++) {
|
| 429 | 433 | $words = self::$capacity[$i][QRCAP_WORDS] - self::$capacity[$i][QRCAP_EC][$level]; |
| 430 | - if($words >= $size) |
|
| 431 | - return $i; |
|
| 434 | + if($words >= $size) { |
|
| 435 | + return $i; |
|
| 436 | + } |
|
| 432 | 437 | } |
| 433 | 438 | |
| 434 | 439 | return -1; |
@@ -446,8 +451,9 @@ discard block |
||
| 446 | 451 | //---------------------------------------------------------------------- |
| 447 | 452 | public static function lengthIndicator($mode, $version) |
| 448 | 453 | {
|
| 449 | - if ($mode == QR_MODE_STRUCTURE) |
|
| 450 | - return 0; |
|
| 454 | + if ($mode == QR_MODE_STRUCTURE) { |
|
| 455 | + return 0; |
|
| 456 | + } |
|
| 451 | 457 | |
| 452 | 458 | if ($version <= 9) {
|
| 453 | 459 | $l = 0; |
@@ -463,8 +469,9 @@ discard block |
||
| 463 | 469 | //---------------------------------------------------------------------- |
| 464 | 470 | public static function maximumWords($mode, $version) |
| 465 | 471 | {
|
| 466 | - if($mode == QR_MODE_STRUCTURE) |
|
| 467 | - return 3; |
|
| 472 | + if($mode == QR_MODE_STRUCTURE) { |
|
| 473 | + return 3; |
|
| 474 | + } |
|
| 468 | 475 | |
| 469 | 476 | if($version <= 9) {
|
| 470 | 477 | $l = 0; |
@@ -610,8 +617,9 @@ discard block |
||
| 610 | 617 | //---------------------------------------------------------------------- |
| 611 | 618 | public static function putAlignmentPattern($version, &$frame, $width) |
| 612 | 619 | {
|
| 613 | - if($version < 2) |
|
| 614 | - return; |
|
| 620 | + if($version < 2) { |
|
| 621 | + return; |
|
| 622 | + } |
|
| 615 | 623 | |
| 616 | 624 | $d = self::$alignmentPattern[$version][1] - self::$alignmentPattern[$version][0]; |
| 617 | 625 | if($d < 0) {
|
@@ -663,8 +671,9 @@ discard block |
||
| 663 | 671 | //---------------------------------------------------------------------- |
| 664 | 672 | public static function getVersionPattern($version) |
| 665 | 673 | {
|
| 666 | - if($version < 7 || $version > QRSPEC_VERSION_MAX) |
|
| 667 | - return 0; |
|
| 674 | + if($version < 7 || $version > QRSPEC_VERSION_MAX) { |
|
| 675 | + return 0; |
|
| 676 | + } |
|
| 668 | 677 | |
| 669 | 678 | return self::$versionPattern[$version -7]; |
| 670 | 679 | } |
@@ -681,11 +690,13 @@ discard block |
||
| 681 | 690 | |
| 682 | 691 | public static function getFormatInfo($mask, $level) |
| 683 | 692 | {
|
| 684 | - if($mask < 0 || $mask > 7) |
|
| 685 | - return 0; |
|
| 693 | + if($mask < 0 || $mask > 7) { |
|
| 694 | + return 0; |
|
| 695 | + } |
|
| 686 | 696 | |
| 687 | - if($level < 0 || $level > 3) |
|
| 688 | - return 0; |
|
| 697 | + if($level < 0 || $level > 3) { |
|
| 698 | + return 0; |
|
| 699 | + } |
|
| 689 | 700 | |
| 690 | 701 | return self::$formatInfo[$level][$mask]; |
| 691 | 702 | } |
@@ -865,8 +876,9 @@ discard block |
||
| 865 | 876 | //---------------------------------------------------------------------- |
| 866 | 877 | public static function newFrame($version) |
| 867 | 878 | {
|
| 868 | - if($version < 1 || $version > QRSPEC_VERSION_MAX) |
|
| 869 | - return null; |
|
| 879 | + if($version < 1 || $version > QRSPEC_VERSION_MAX) { |
|
| 880 | + return null; |
|
| 881 | + } |
|
| 870 | 882 | |
| 871 | 883 | if(!isset(self::$frames[$version])) {
|
| 872 | 884 | |
@@ -884,8 +896,9 @@ discard block |
||
| 884 | 896 | } |
| 885 | 897 | } |
| 886 | 898 | |
| 887 | - if(is_null(self::$frames[$version])) |
|
| 888 | - return null; |
|
| 899 | + if(is_null(self::$frames[$version])) { |
|
| 900 | + return null; |
|
| 901 | + } |
|
| 889 | 902 | |
| 890 | 903 | return self::$frames[$version]; |
| 891 | 904 | } |
@@ -950,7 +963,7 @@ discard block |
||
| 950 | 963 | ImagePng($image, $filename); |
| 951 | 964 | header("Content-type: image/png");
|
| 952 | 965 | ImagePng($image); |
| 953 | - }else{
|
|
| 966 | + } else{
|
|
| 954 | 967 | ImagePng($image, $filename); |
| 955 | 968 | } |
| 956 | 969 | } |
@@ -1210,8 +1223,9 @@ discard block |
||
| 1210 | 1223 | {
|
| 1211 | 1224 | $bits = 0; |
| 1212 | 1225 | |
| 1213 | - if($version == 0) |
|
| 1214 | - $version = 1; |
|
| 1226 | + if($version == 0) { |
|
| 1227 | + $version = 1; |
|
| 1228 | + } |
|
| 1215 | 1229 | |
| 1216 | 1230 | switch($this->mode) {
|
| 1217 | 1231 | case QR_MODE_NUM: $bits = QRinput::estimateBitsModeNum($this->size); break; |
@@ -1270,8 +1284,9 @@ discard block |
||
| 1270 | 1284 | break; |
| 1271 | 1285 | } |
| 1272 | 1286 | |
| 1273 | - if($ret < 0) |
|
| 1274 | - return -1; |
|
| 1287 | + if($ret < 0) { |
|
| 1288 | + return -1; |
|
| 1289 | + } |
|
| 1275 | 1290 | } |
| 1276 | 1291 | |
| 1277 | 1292 | return $this->bstream->size(); |
@@ -1488,8 +1503,9 @@ discard block |
||
| 1488 | 1503 | //---------------------------------------------------------------------- |
| 1489 | 1504 | public static function checkModeKanji($size, $data) |
| 1490 | 1505 | {
|
| 1491 | - if($size & 1) |
|
| 1492 | - return false; |
|
| 1506 | + if($size & 1) { |
|
| 1507 | + return false; |
|
| 1508 | + } |
|
| 1493 | 1509 | |
| 1494 | 1510 | for($i=0; $i<$size; $i+=2) {
|
| 1495 | 1511 | $val = (ord($data[$i]) << 8) | ord($data[$i+1]); |
@@ -1509,8 +1525,9 @@ discard block |
||
| 1509 | 1525 | |
| 1510 | 1526 | public static function check($mode, $size, $data) |
| 1511 | 1527 | {
|
| 1512 | - if($size <= 0) |
|
| 1513 | - return false; |
|
| 1528 | + if($size <= 0) { |
|
| 1529 | + return false; |
|
| 1530 | + } |
|
| 1514 | 1531 | |
| 1515 | 1532 | switch($mode) {
|
| 1516 | 1533 | case QR_MODE_NUM: return self::checkModeNum($size, $data); break; |
@@ -1575,8 +1592,9 @@ discard block |
||
| 1575 | 1592 | $chunks = (int)($payload / 11); |
| 1576 | 1593 | $remain = $payload - $chunks * 11; |
| 1577 | 1594 | $size = $chunks * 2; |
| 1578 | - if($remain >= 6) |
|
| 1579 | - $size++; |
|
| 1595 | + if($remain >= 6) { |
|
| 1596 | + $size++; |
|
| 1597 | + } |
|
| 1580 | 1598 | break; |
| 1581 | 1599 | case QR_MODE_8: |
| 1582 | 1600 | $size = (int)($payload / 8); |
@@ -1593,8 +1611,12 @@ discard block |
||
| 1593 | 1611 | } |
| 1594 | 1612 | |
| 1595 | 1613 | $maxsize = QRspec::maximumWords($mode, $version); |
| 1596 | - if($size < 0) $size = 0; |
|
| 1597 | - if($size > $maxsize) $size = $maxsize; |
|
| 1614 | + if($size < 0) { |
|
| 1615 | + $size = 0; |
|
| 1616 | + } |
|
| 1617 | + if($size > $maxsize) { |
|
| 1618 | + $size = $maxsize; |
|
| 1619 | + } |
|
| 1598 | 1620 | |
| 1599 | 1621 | return $size; |
| 1600 | 1622 | } |
@@ -1607,8 +1629,9 @@ discard block |
||
| 1607 | 1629 | foreach($this->items as $item) {
|
| 1608 | 1630 | $bits = $item->encodeBitStream($this->version); |
| 1609 | 1631 | |
| 1610 | - if($bits < 0) |
|
| 1611 | - return -1; |
|
| 1632 | + if($bits < 0) { |
|
| 1633 | + return -1; |
|
| 1634 | + } |
|
| 1612 | 1635 | |
| 1613 | 1636 | $total += $bits; |
| 1614 | 1637 | } |
@@ -1627,8 +1650,9 @@ discard block |
||
| 1627 | 1650 | for(;;) {
|
| 1628 | 1651 | $bits = $this->createBitStream(); |
| 1629 | 1652 | |
| 1630 | - if($bits < 0) |
|
| 1631 | - return -1; |
|
| 1653 | + if($bits < 0) { |
|
| 1654 | + return -1; |
|
| 1655 | + } |
|
| 1632 | 1656 | |
| 1633 | 1657 | $ver = QRspec::getMinimumVersion((int)(($bits + 7) / 8), $this->level); |
| 1634 | 1658 | if($ver < 0) {
|
@@ -1665,8 +1689,9 @@ discard block |
||
| 1665 | 1689 | $padding = new QRbitstream(); |
| 1666 | 1690 | $ret = $padding->appendNum($words * 8 - $bits + 4, 0); |
| 1667 | 1691 | |
| 1668 | - if($ret < 0) |
|
| 1669 | - return $ret; |
|
| 1692 | + if($ret < 0) { |
|
| 1693 | + return $ret; |
|
| 1694 | + } |
|
| 1670 | 1695 | |
| 1671 | 1696 | $padlen = $maxwords - $words; |
| 1672 | 1697 | |
@@ -1679,8 +1704,9 @@ discard block |
||
| 1679 | 1704 | |
| 1680 | 1705 | $ret = $padding->appendBytes($padlen, $padbuf); |
| 1681 | 1706 | |
| 1682 | - if($ret < 0) |
|
| 1683 | - return $ret; |
|
| 1707 | + if($ret < 0) { |
|
| 1708 | + return $ret; |
|
| 1709 | + } |
|
| 1684 | 1710 | |
| 1685 | 1711 | } |
| 1686 | 1712 | |
@@ -1857,13 +1883,15 @@ discard block |
||
| 1857 | 1883 | //---------------------------------------------------------------------- |
| 1858 | 1884 | public function appendNum($bits, $num) |
| 1859 | 1885 | {
|
| 1860 | - if ($bits == 0) |
|
| 1861 | - return 0; |
|
| 1886 | + if ($bits == 0) { |
|
| 1887 | + return 0; |
|
| 1888 | + } |
|
| 1862 | 1889 | |
| 1863 | 1890 | $b = QRbitstream::newFromNum($bits, $num); |
| 1864 | 1891 | |
| 1865 | - if(is_null($b)) |
|
| 1866 | - return -1; |
|
| 1892 | + if(is_null($b)) { |
|
| 1893 | + return -1; |
|
| 1894 | + } |
|
| 1867 | 1895 | |
| 1868 | 1896 | $ret = $this->append($b); |
| 1869 | 1897 | unset($b); |
@@ -1874,13 +1902,15 @@ discard block |
||
| 1874 | 1902 | //---------------------------------------------------------------------- |
| 1875 | 1903 | public function appendBytes($size, $data) |
| 1876 | 1904 | {
|
| 1877 | - if ($size == 0) |
|
| 1878 | - return 0; |
|
| 1905 | + if ($size == 0) { |
|
| 1906 | + return 0; |
|
| 1907 | + } |
|
| 1879 | 1908 | |
| 1880 | 1909 | $b = QRbitstream::newFromBytes($size, $data); |
| 1881 | 1910 | |
| 1882 | - if(is_null($b)) |
|
| 1883 | - return -1; |
|
| 1911 | + if(is_null($b)) { |
|
| 1912 | + return -1; |
|
| 1913 | + } |
|
| 1884 | 1914 | |
| 1885 | 1915 | $ret = $this->append($b); |
| 1886 | 1916 | unset($b); |
@@ -1984,8 +2014,9 @@ discard block |
||
| 1984 | 2014 | //---------------------------------------------------------------------- |
| 1985 | 2015 | public static function isdigitat($str, $pos) |
| 1986 | 2016 | {
|
| 1987 | - if ($pos >= strlen($str)) |
|
| 1988 | - return false; |
|
| 2017 | + if ($pos >= strlen($str)) { |
|
| 2018 | + return false; |
|
| 2019 | + } |
|
| 1989 | 2020 | |
| 1990 | 2021 | return ((ord($str[$pos]) >= ord('0'))&&(ord($str[$pos]) <= ord('9')));
|
| 1991 | 2022 | } |
@@ -1993,8 +2024,9 @@ discard block |
||
| 1993 | 2024 | //---------------------------------------------------------------------- |
| 1994 | 2025 | public static function isalnumat($str, $pos) |
| 1995 | 2026 | {
|
| 1996 | - if ($pos >= strlen($str)) |
|
| 1997 | - return false; |
|
| 2027 | + if ($pos >= strlen($str)) { |
|
| 2028 | + return false; |
|
| 2029 | + } |
|
| 1998 | 2030 | |
| 1999 | 2031 | return (QRinput::lookAnTable(ord($str[$pos])) >= 0); |
| 2000 | 2032 | } |
@@ -2002,8 +2034,9 @@ discard block |
||
| 2002 | 2034 | //---------------------------------------------------------------------- |
| 2003 | 2035 | public function identifyMode($pos) |
| 2004 | 2036 | {
|
| 2005 | - if ($pos >= strlen($this->dataStr)) |
|
| 2006 | - return QR_MODE_NUL; |
|
| 2037 | + if ($pos >= strlen($this->dataStr)) { |
|
| 2038 | + return QR_MODE_NUL; |
|
| 2039 | + } |
|
| 2007 | 2040 | |
| 2008 | 2041 | $c = $this->dataStr[$pos]; |
| 2009 | 2042 | |
@@ -2057,8 +2090,9 @@ discard block |
||
| 2057 | 2090 | } |
| 2058 | 2091 | |
| 2059 | 2092 | $ret = $this->input->append(QR_MODE_NUM, $run, str_split($this->dataStr)); |
| 2060 | - if($ret < 0) |
|
| 2061 | - return -1; |
|
| 2093 | + if($ret < 0) { |
|
| 2094 | + return -1; |
|
| 2095 | + } |
|
| 2062 | 2096 | |
| 2063 | 2097 | return $run; |
| 2064 | 2098 | } |
@@ -2104,8 +2138,9 @@ discard block |
||
| 2104 | 2138 | } |
| 2105 | 2139 | |
| 2106 | 2140 | $ret = $this->input->append(QR_MODE_AN, $run, str_split($this->dataStr)); |
| 2107 | - if($ret < 0) |
|
| 2108 | - return -1; |
|
| 2141 | + if($ret < 0) { |
|
| 2142 | + return -1; |
|
| 2143 | + } |
|
| 2109 | 2144 | |
| 2110 | 2145 | return $run; |
| 2111 | 2146 | } |
@@ -2120,8 +2155,9 @@ discard block |
||
| 2120 | 2155 | } |
| 2121 | 2156 | |
| 2122 | 2157 | $ret = $this->input->append(QR_MODE_KANJI, $p, str_split($this->dataStr)); |
| 2123 | - if($ret < 0) |
|
| 2124 | - return -1; |
|
| 2158 | + if($ret < 0) { |
|
| 2159 | + return -1; |
|
| 2160 | + } |
|
| 2125 | 2161 | |
| 2126 | 2162 | return $run; |
| 2127 | 2163 | } |
@@ -2175,8 +2211,9 @@ discard block |
||
| 2175 | 2211 | $run = $p; |
| 2176 | 2212 | $ret = $this->input->append(QR_MODE_8, $run, str_split($this->dataStr)); |
| 2177 | 2213 | |
| 2178 | - if($ret < 0) |
|
| 2179 | - return -1; |
|
| 2214 | + if($ret < 0) { |
|
| 2215 | + return -1; |
|
| 2216 | + } |
|
| 2180 | 2217 | |
| 2181 | 2218 | return $run; |
| 2182 | 2219 | } |
@@ -2186,8 +2223,9 @@ discard block |
||
| 2186 | 2223 | {
|
| 2187 | 2224 | while (strlen($this->dataStr) > 0) |
| 2188 | 2225 | {
|
| 2189 | - if($this->dataStr == '') |
|
| 2190 | - return 0; |
|
| 2226 | + if($this->dataStr == '') { |
|
| 2227 | + return 0; |
|
| 2228 | + } |
|
| 2191 | 2229 | |
| 2192 | 2230 | $mode = $this->identifyMode(0); |
| 2193 | 2231 | |
@@ -2195,16 +2233,22 @@ discard block |
||
| 2195 | 2233 | case QR_MODE_NUM: $length = $this->eatNum(); break; |
| 2196 | 2234 | case QR_MODE_AN: $length = $this->eatAn(); break; |
| 2197 | 2235 | case QR_MODE_KANJI: |
| 2198 | - if ($this->modeHint == QR_MODE_KANJI) |
|
| 2199 | - $length = $this->eatKanji(); |
|
| 2200 | - else $length = $this->eat8(); |
|
| 2236 | + if ($this->modeHint == QR_MODE_KANJI) { |
|
| 2237 | + $length = $this->eatKanji(); |
|
| 2238 | + } else { |
|
| 2239 | + $length = $this->eat8(); |
|
| 2240 | + } |
|
| 2201 | 2241 | break; |
| 2202 | 2242 | default: $length = $this->eat8(); break; |
| 2203 | 2243 | |
| 2204 | 2244 | } |
| 2205 | 2245 | |
| 2206 | - if($length == 0) return 0; |
|
| 2207 | - if($length < 0) return -1; |
|
| 2246 | + if($length == 0) { |
|
| 2247 | + return 0; |
|
| 2248 | + } |
|
| 2249 | + if($length < 0) { |
|
| 2250 | + return -1; |
|
| 2251 | + } |
|
| 2208 | 2252 | |
| 2209 | 2253 | $this->dataStr = substr($this->dataStr, $length); |
| 2210 | 2254 | } |
@@ -2240,8 +2284,9 @@ discard block |
||
| 2240 | 2284 | |
| 2241 | 2285 | $split = new QRsplit($string, $input, $modeHint); |
| 2242 | 2286 | |
| 2243 | - if(!$casesensitive) |
|
| 2244 | - $split->toUpper(); |
|
| 2287 | + if(!$casesensitive) { |
|
| 2288 | + $split->toUpper(); |
|
| 2289 | + } |
|
| 2245 | 2290 | |
| 2246 | 2291 | return $split->splitString(); |
| 2247 | 2292 | } |
@@ -2318,11 +2363,23 @@ discard block |
||
| 2318 | 2363 | $rs = null; |
| 2319 | 2364 | |
| 2320 | 2365 | // Check parameter ranges |
| 2321 | - if($symsize < 0 || $symsize > 8) return $rs; |
|
| 2322 | - if($fcr < 0 || $fcr >= (1<<$symsize)) return $rs; |
|
| 2323 | - if($prim <= 0 || $prim >= (1<<$symsize)) return $rs; |
|
| 2324 | - if($nroots < 0 || $nroots >= (1<<$symsize)) return $rs; // Can't have more roots than symbol values! |
|
| 2325 | - if($pad < 0 || $pad >= ((1<<$symsize) -1 - $nroots)) return $rs; // Too much padding |
|
| 2366 | + if($symsize < 0 || $symsize > 8) { |
|
| 2367 | + return $rs; |
|
| 2368 | + } |
|
| 2369 | + if($fcr < 0 || $fcr >= (1<<$symsize)) { |
|
| 2370 | + return $rs; |
|
| 2371 | + } |
|
| 2372 | + if($prim <= 0 || $prim >= (1<<$symsize)) { |
|
| 2373 | + return $rs; |
|
| 2374 | + } |
|
| 2375 | + if($nroots < 0 || $nroots >= (1<<$symsize)) { |
|
| 2376 | + return $rs; |
|
| 2377 | + } |
|
| 2378 | + // Can't have more roots than symbol values! |
|
| 2379 | + if($pad < 0 || $pad >= ((1<<$symsize) -1 - $nroots)) { |
|
| 2380 | + return $rs; |
|
| 2381 | + } |
|
| 2382 | + // Too much padding |
|
| 2326 | 2383 | |
| 2327 | 2384 | $rs = new QRrsItem(); |
| 2328 | 2385 | $rs->mm = $symsize; |
@@ -2388,8 +2445,9 @@ discard block |
||
| 2388 | 2445 | } |
| 2389 | 2446 | |
| 2390 | 2447 | // convert rs->genpoly[] to index form for quicker encoding |
| 2391 | - for ($i = 0; $i <= $nroots; $i++) |
|
| 2392 | - $rs->genpoly[$i] = $rs->index_of[$rs->genpoly[$i]]; |
|
| 2448 | + for ($i = 0; $i <= $nroots; $i++) { |
|
| 2449 | + $rs->genpoly[$i] = $rs->index_of[$rs->genpoly[$i]]; |
|
| 2450 | + } |
|
| 2393 | 2451 | |
| 2394 | 2452 | return $rs; |
| 2395 | 2453 | } |
@@ -2447,12 +2505,24 @@ discard block |
||
| 2447 | 2505 | public static function init_rs($symsize, $gfpoly, $fcr, $prim, $nroots, $pad) |
| 2448 | 2506 | {
|
| 2449 | 2507 | foreach(self::$items as $rs) {
|
| 2450 | - if($rs->pad != $pad) continue; |
|
| 2451 | - if($rs->nroots != $nroots) continue; |
|
| 2452 | - if($rs->mm != $symsize) continue; |
|
| 2453 | - if($rs->gfpoly != $gfpoly) continue; |
|
| 2454 | - if($rs->fcr != $fcr) continue; |
|
| 2455 | - if($rs->prim != $prim) continue; |
|
| 2508 | + if($rs->pad != $pad) { |
|
| 2509 | + continue; |
|
| 2510 | + } |
|
| 2511 | + if($rs->nroots != $nroots) { |
|
| 2512 | + continue; |
|
| 2513 | + } |
|
| 2514 | + if($rs->mm != $symsize) { |
|
| 2515 | + continue; |
|
| 2516 | + } |
|
| 2517 | + if($rs->gfpoly != $gfpoly) { |
|
| 2518 | + continue; |
|
| 2519 | + } |
|
| 2520 | + if($rs->fcr != $fcr) { |
|
| 2521 | + continue; |
|
| 2522 | + } |
|
| 2523 | + if($rs->prim != $prim) { |
|
| 2524 | + continue; |
|
| 2525 | + } |
|
| 2456 | 2526 | |
| 2457 | 2527 | return $rs; |
| 2458 | 2528 | } |
@@ -2591,8 +2661,9 @@ discard block |
||
| 2591 | 2661 | {
|
| 2592 | 2662 | $codeArr = array(); |
| 2593 | 2663 | |
| 2594 | - foreach ($bitFrame as $line) |
|
| 2595 | - $codeArr[] = join('', $line);
|
|
| 2664 | + foreach ($bitFrame as $line) { |
|
| 2665 | + $codeArr[] = join('', $line); |
|
| 2666 | + } |
|
| 2596 | 2667 | |
| 2597 | 2668 | return gzcompress(join("\n", $codeArr), 9);
|
| 2598 | 2669 | } |
@@ -2603,8 +2674,9 @@ discard block |
||
| 2603 | 2674 | $codeArr = array(); |
| 2604 | 2675 | |
| 2605 | 2676 | $codeLines = explode("\n", gzuncompress($code));
|
| 2606 | - foreach ($codeLines as $line) |
|
| 2607 | - $codeArr[] = str_split($line); |
|
| 2677 | + foreach ($codeLines as $line) { |
|
| 2678 | + $codeArr[] = str_split($line); |
|
| 2679 | + } |
|
| 2608 | 2680 | |
| 2609 | 2681 | return $codeArr; |
| 2610 | 2682 | } |
@@ -2622,16 +2694,18 @@ discard block |
||
| 2622 | 2694 | $bitMask = self::unserial(file_get_contents($fileName)); |
| 2623 | 2695 | } else {
|
| 2624 | 2696 | $bitMask = $this->generateMaskNo($maskNo, $width, $s); |
| 2625 | - if (!file_exists(QR_CACHE_DIR.'mask_'.$maskNo)) |
|
| 2626 | - mkdir(QR_CACHE_DIR.'mask_'.$maskNo); |
|
| 2697 | + if (!file_exists(QR_CACHE_DIR.'mask_'.$maskNo)) { |
|
| 2698 | + mkdir(QR_CACHE_DIR.'mask_'.$maskNo); |
|
| 2699 | + } |
|
| 2627 | 2700 | file_put_contents($fileName, self::serial($bitMask)); |
| 2628 | 2701 | } |
| 2629 | 2702 | } else {
|
| 2630 | 2703 | $bitMask = $this->generateMaskNo($maskNo, $width, $s); |
| 2631 | 2704 | } |
| 2632 | 2705 | |
| 2633 | - if ($maskGenOnly) |
|
| 2634 | - return; |
|
| 2706 | + if ($maskGenOnly) { |
|
| 2707 | + return; |
|
| 2708 | + } |
|
| 2635 | 2709 | |
| 2636 | 2710 | $d = $s; |
| 2637 | 2711 | |
@@ -2698,8 +2772,9 @@ discard block |
||
| 2698 | 2772 | |
| 2699 | 2773 | $frameY = $frame[$y]; |
| 2700 | 2774 | |
| 2701 | - if ($y>0) |
|
| 2702 | - $frameYM = $frame[$y-1]; |
|
| 2775 | + if ($y>0) { |
|
| 2776 | + $frameYM = $frame[$y-1]; |
|
| 2777 | + } |
|
| 2703 | 2778 | |
| 2704 | 2779 | for($x=0; $x<$width; $x++) {
|
| 2705 | 2780 | if(($x > 0) && ($y > 0)) {
|
@@ -2912,14 +2987,17 @@ discard block |
||
| 2912 | 2987 | $blockNo++; |
| 2913 | 2988 | } |
| 2914 | 2989 | |
| 2915 | - if(QRspec::rsBlockNum2($spec) == 0) |
|
| 2916 | - return 0; |
|
| 2990 | + if(QRspec::rsBlockNum2($spec) == 0) { |
|
| 2991 | + return 0; |
|
| 2992 | + } |
|
| 2917 | 2993 | |
| 2918 | 2994 | $dl = QRspec::rsDataCodes2($spec); |
| 2919 | 2995 | $el = QRspec::rsEccCodes2($spec); |
| 2920 | 2996 | $rs = QRrs::init_rs(8, 0x11d, 0, 1, $el, 255 - $dl - $el); |
| 2921 | 2997 | |
| 2922 | - if($rs == NULL) return -1; |
|
| 2998 | + if($rs == NULL) { |
|
| 2999 | + return -1; |
|
| 3000 | + } |
|
| 2923 | 3001 | |
| 2924 | 3002 | for($i=0; $i<QRspec::rsBlockNum2($spec); $i++) {
|
| 2925 | 3003 | $ecc = array_slice($this->ecccode,$eccPos); |
@@ -3057,7 +3135,9 @@ discard block |
||
| 3057 | 3135 | } |
| 3058 | 3136 | |
| 3059 | 3137 | $input = new QRinput($version, $level); |
| 3060 | - if($input == NULL) return NULL; |
|
| 3138 | + if($input == NULL) { |
|
| 3139 | + return NULL; |
|
| 3140 | + } |
|
| 3061 | 3141 | |
| 3062 | 3142 | $ret = $input->append(QR_MODE_8, strlen($string), str_split($string)); |
| 3063 | 3143 | if($ret < 0) {
|
@@ -3077,7 +3157,9 @@ discard block |
||
| 3077 | 3157 | } |
| 3078 | 3158 | |
| 3079 | 3159 | $input = new QRinput($version, $level); |
| 3080 | - if($input == NULL) return NULL; |
|
| 3160 | + if($input == NULL) { |
|
| 3161 | + return NULL; |
|
| 3162 | + } |
|
| 3081 | 3163 | |
| 3082 | 3164 | $ret = QRsplit::splitStringToQRinput($string, $input, $hint, $casesensitive); |
| 3083 | 3165 | if($ret < 0) {
|
@@ -3187,7 +3269,9 @@ discard block |
||
| 3187 | 3269 | } |
| 3188 | 3270 | } |
| 3189 | 3271 | } |
| 3190 | - if($x < 0 || $y < 0) return null; |
|
| 3272 | + if($x < 0 || $y < 0) { |
|
| 3273 | + return null; |
|
| 3274 | + } |
|
| 3191 | 3275 | |
| 3192 | 3276 | $this->x = $x; |
| 3193 | 3277 | $this->y = $y; |
@@ -3294,8 +3378,9 @@ discard block |
||
| 3294 | 3378 | $err = ob_get_contents(); |
| 3295 | 3379 | ob_end_clean(); |
| 3296 | 3380 | |
| 3297 | - if ($err != '') |
|
| 3298 | - QRtools::log($outfile, $err); |
|
| 3381 | + if ($err != '') { |
|
| 3382 | + QRtools::log($outfile, $err); |
|
| 3383 | + } |
|
| 3299 | 3384 | |
| 3300 | 3385 | $maxSize = (int)(QR_PNG_MAXIMUM_SIZE / (count($tab)+2*$this->margin)); |
| 3301 | 3386 | |
@@ -115,8 +115,9 @@ discard block |
||
| 115 | 115 | |
| 116 | 116 | for($i=1; $i<= QRSPEC_VERSION_MAX; $i++) {
|
| 117 | 117 | $words = self::$capacity[$i][QRCAP_WORDS] - self::$capacity[$i][QRCAP_EC][$level]; |
| 118 | - if($words >= $size) |
|
| 119 | - return $i; |
|
| 118 | + if($words >= $size) { |
|
| 119 | + return $i; |
|
| 120 | + } |
|
| 120 | 121 | } |
| 121 | 122 | |
| 122 | 123 | return -1; |
@@ -134,8 +135,9 @@ discard block |
||
| 134 | 135 | //---------------------------------------------------------------------- |
| 135 | 136 | public static function lengthIndicator($mode, $version) |
| 136 | 137 | {
|
| 137 | - if ($mode == QR_MODE_STRUCTURE) |
|
| 138 | - return 0; |
|
| 138 | + if ($mode == QR_MODE_STRUCTURE) { |
|
| 139 | + return 0; |
|
| 140 | + } |
|
| 139 | 141 | |
| 140 | 142 | if ($version <= 9) {
|
| 141 | 143 | $l = 0; |
@@ -151,8 +153,9 @@ discard block |
||
| 151 | 153 | //---------------------------------------------------------------------- |
| 152 | 154 | public static function maximumWords($mode, $version) |
| 153 | 155 | {
|
| 154 | - if($mode == QR_MODE_STRUCTURE) |
|
| 155 | - return 3; |
|
| 156 | + if($mode == QR_MODE_STRUCTURE) { |
|
| 157 | + return 3; |
|
| 158 | + } |
|
| 156 | 159 | |
| 157 | 160 | if($version <= 9) {
|
| 158 | 161 | $l = 0; |
@@ -298,8 +301,9 @@ discard block |
||
| 298 | 301 | //---------------------------------------------------------------------- |
| 299 | 302 | public static function putAlignmentPattern($version, &$frame, $width) |
| 300 | 303 | {
|
| 301 | - if($version < 2) |
|
| 302 | - return; |
|
| 304 | + if($version < 2) { |
|
| 305 | + return; |
|
| 306 | + } |
|
| 303 | 307 | |
| 304 | 308 | $d = self::$alignmentPattern[$version][1] - self::$alignmentPattern[$version][0]; |
| 305 | 309 | if($d < 0) {
|
@@ -351,8 +355,9 @@ discard block |
||
| 351 | 355 | //---------------------------------------------------------------------- |
| 352 | 356 | public static function getVersionPattern($version) |
| 353 | 357 | {
|
| 354 | - if($version < 7 || $version > QRSPEC_VERSION_MAX) |
|
| 355 | - return 0; |
|
| 358 | + if($version < 7 || $version > QRSPEC_VERSION_MAX) { |
|
| 359 | + return 0; |
|
| 360 | + } |
|
| 356 | 361 | |
| 357 | 362 | return self::$versionPattern[$version -7]; |
| 358 | 363 | } |
@@ -369,11 +374,13 @@ discard block |
||
| 369 | 374 | |
| 370 | 375 | public static function getFormatInfo($mask, $level) |
| 371 | 376 | {
|
| 372 | - if($mask < 0 || $mask > 7) |
|
| 373 | - return 0; |
|
| 377 | + if($mask < 0 || $mask > 7) { |
|
| 378 | + return 0; |
|
| 379 | + } |
|
| 374 | 380 | |
| 375 | - if($level < 0 || $level > 3) |
|
| 376 | - return 0; |
|
| 381 | + if($level < 0 || $level > 3) { |
|
| 382 | + return 0; |
|
| 383 | + } |
|
| 377 | 384 | |
| 378 | 385 | return self::$formatInfo[$level][$mask]; |
| 379 | 386 | } |
@@ -553,8 +560,9 @@ discard block |
||
| 553 | 560 | //---------------------------------------------------------------------- |
| 554 | 561 | public static function newFrame($version) |
| 555 | 562 | {
|
| 556 | - if($version < 1 || $version > QRSPEC_VERSION_MAX) |
|
| 557 | - return null; |
|
| 563 | + if($version < 1 || $version > QRSPEC_VERSION_MAX) { |
|
| 564 | + return null; |
|
| 565 | + } |
|
| 558 | 566 | |
| 559 | 567 | if(!isset(self::$frames[$version])) {
|
| 560 | 568 | |
@@ -572,8 +580,9 @@ discard block |
||
| 572 | 580 | } |
| 573 | 581 | } |
| 574 | 582 | |
| 575 | - if(is_null(self::$frames[$version])) |
|
| 576 | - return null; |
|
| 583 | + if(is_null(self::$frames[$version])) { |
|
| 584 | + return null; |
|
| 585 | + } |
|
| 577 | 586 | |
| 578 | 587 | return self::$frames[$version]; |
| 579 | 588 | } |
@@ -119,8 +119,9 @@ discard block |
||
| 119 | 119 | {
|
| 120 | 120 | $codeArr = array(); |
| 121 | 121 | |
| 122 | - foreach ($bitFrame as $line) |
|
| 123 | - $codeArr[] = join('', $line);
|
|
| 122 | + foreach ($bitFrame as $line) { |
|
| 123 | + $codeArr[] = join('', $line); |
|
| 124 | + } |
|
| 124 | 125 | |
| 125 | 126 | return gzcompress(join("\n", $codeArr), 9);
|
| 126 | 127 | } |
@@ -131,8 +132,9 @@ discard block |
||
| 131 | 132 | $codeArr = array(); |
| 132 | 133 | |
| 133 | 134 | $codeLines = explode("\n", gzuncompress($code));
|
| 134 | - foreach ($codeLines as $line) |
|
| 135 | - $codeArr[] = str_split($line); |
|
| 135 | + foreach ($codeLines as $line) { |
|
| 136 | + $codeArr[] = str_split($line); |
|
| 137 | + } |
|
| 136 | 138 | |
| 137 | 139 | return $codeArr; |
| 138 | 140 | } |
@@ -150,16 +152,18 @@ discard block |
||
| 150 | 152 | $bitMask = self::unserial(file_get_contents($fileName)); |
| 151 | 153 | } else {
|
| 152 | 154 | $bitMask = $this->generateMaskNo($maskNo, $width, $s); |
| 153 | - if (!file_exists(QR_CACHE_DIR.'mask_'.$maskNo)) |
|
| 154 | - mkdir(QR_CACHE_DIR.'mask_'.$maskNo); |
|
| 155 | + if (!file_exists(QR_CACHE_DIR.'mask_'.$maskNo)) { |
|
| 156 | + mkdir(QR_CACHE_DIR.'mask_'.$maskNo); |
|
| 157 | + } |
|
| 155 | 158 | file_put_contents($fileName, self::serial($bitMask)); |
| 156 | 159 | } |
| 157 | 160 | } else {
|
| 158 | 161 | $bitMask = $this->generateMaskNo($maskNo, $width, $s); |
| 159 | 162 | } |
| 160 | 163 | |
| 161 | - if ($maskGenOnly) |
|
| 162 | - return; |
|
| 164 | + if ($maskGenOnly) { |
|
| 165 | + return; |
|
| 166 | + } |
|
| 163 | 167 | |
| 164 | 168 | $d = $s; |
| 165 | 169 | |
@@ -226,8 +230,9 @@ discard block |
||
| 226 | 230 | |
| 227 | 231 | $frameY = $frame[$y]; |
| 228 | 232 | |
| 229 | - if ($y>0) |
|
| 230 | - $frameYM = $frame[$y-1]; |
|
| 233 | + if ($y>0) { |
|
| 234 | + $frameYM = $frame[$y-1]; |
|
| 235 | + } |
|
| 231 | 236 | |
| 232 | 237 | for($x=0; $x<$width; $x++) {
|
| 233 | 238 | if(($x > 0) && ($y > 0)) {
|
@@ -175,7 +175,9 @@ discard block |
||
| 175 | 175 | if (!$this->readFileIntoString($fileName,$s)) {
|
| 176 | 176 | $this->triggerError ("Error while reading template file " . $fileName . ".");
|
| 177 | 177 | return false; } |
| 178 | - if (!$this->setTemplateString($s)) return false; |
|
| 178 | + if (!$this->setTemplateString($s)) { |
|
| 179 | + return false; |
|
| 180 | + } |
|
| 179 | 181 | return true; } |
| 180 | 182 | |
| 181 | 183 | /** |
@@ -280,16 +282,22 @@ discard block |
||
| 280 | 282 | $p = 0; |
| 281 | 283 | while (true) {
|
| 282 | 284 | $p0 = strpos($this->template,'<!--',$p); |
| 283 | - if ($p0 === false) break; |
|
| 285 | + if ($p0 === false) { |
|
| 286 | + break; |
|
| 287 | + } |
|
| 284 | 288 | $p = strpos($this->template,'-->',$p0); |
| 285 | 289 | if ($p === false) {
|
| 286 | 290 | $this->triggerError ("Invalid HTML comment in template at offset $p0.");
|
| 287 | 291 | return false; } |
| 288 | 292 | $p += 3; |
| 289 | 293 | $cmdL = substr($this->template,$p0+4,$p-$p0-7); |
| 290 | - if (!$this->processTemplateCommand($cmdL,$p0,$p,$resumeFromStart)) |
|
| 291 | - return false; |
|
| 292 | - if ($resumeFromStart) $p = $p0; } |
|
| 294 | + if (!$this->processTemplateCommand($cmdL,$p0,$p,$resumeFromStart)) { |
|
| 295 | + return false; |
|
| 296 | + } |
|
| 297 | + if ($resumeFromStart) { |
|
| 298 | + $p = $p0; |
|
| 299 | + } |
|
| 300 | + } |
|
| 293 | 301 | return true; } |
| 294 | 302 | |
| 295 | 303 | /** |
@@ -300,20 +308,25 @@ discard block |
||
| 300 | 308 | $resumeFromStart = false; |
| 301 | 309 | $p = 0; |
| 302 | 310 | $cmd = ''; |
| 303 | - if (!$this->parseWord($cmdL,$p,$cmd)) return true; |
|
| 311 | + if (!$this->parseWord($cmdL,$p,$cmd)) { |
|
| 312 | + return true; |
|
| 313 | + } |
|
| 304 | 314 | $parms = substr($cmdL,$p); |
| 305 | 315 | switch (strtoupper($cmd)) {
|
| 306 | 316 | case '$BEGINBLOCK': |
| 307 | - if (!$this->processBeginBlockCmd($parms,$cmdTPosBegin,$cmdTPosEnd)) |
|
| 308 | - return false; |
|
| 317 | + if (!$this->processBeginBlockCmd($parms,$cmdTPosBegin,$cmdTPosEnd)) { |
|
| 318 | + return false; |
|
| 319 | + } |
|
| 309 | 320 | break; |
| 310 | 321 | case '$ENDBLOCK': |
| 311 | - if (!$this->processEndBlockCmd($parms,$cmdTPosBegin,$cmdTPosEnd)) |
|
| 312 | - return false; |
|
| 322 | + if (!$this->processEndBlockCmd($parms,$cmdTPosBegin,$cmdTPosEnd)) { |
|
| 323 | + return false; |
|
| 324 | + } |
|
| 313 | 325 | break; |
| 314 | 326 | case '$INCLUDE': |
| 315 | - if (!$this->processincludeCmd($parms,$cmdTPosBegin,$cmdTPosEnd)) |
|
| 316 | - return false; |
|
| 327 | + if (!$this->processincludeCmd($parms,$cmdTPosBegin,$cmdTPosEnd)) { |
|
| 328 | + return false; |
|
| 329 | + } |
|
| 317 | 330 | $resumeFromStart = true; |
| 318 | 331 | break; |
| 319 | 332 | default: |
@@ -385,8 +398,9 @@ discard block |
||
| 385 | 398 | $btr =& $this->blockTab[$blockNo]; |
| 386 | 399 | $btr = array(); |
| 387 | 400 | $btr['blockName'] = $blockName; |
| 388 | - if (!$this->lookupBlockName($blockName,$btr['nextWithSameName'])) |
|
| 389 | - $btr['nextWithSameName'] = -1; |
|
| 401 | + if (!$this->lookupBlockName($blockName,$btr['nextWithSameName'])) { |
|
| 402 | + $btr['nextWithSameName'] = -1; |
|
| 403 | + } |
|
| 390 | 404 | $btr['definitionIsOpen'] = true; |
| 391 | 405 | $btr['instances'] = 0; |
| 392 | 406 | $btr['firstBlockInstNo'] = -1; |
@@ -436,7 +450,9 @@ discard block |
||
| 436 | 450 | if (strlen($this->template) > $this->maxInclTemplateSize) {
|
| 437 | 451 | $this->triggerError ("Subtemplate include aborted because the internal template string is longer than $this->maxInclTemplateSize characters.");
|
| 438 | 452 | return false; } |
| 439 | - if (!$this->loadSubtemplate($subtemplateName,$subtemplate)) return false; |
|
| 453 | + if (!$this->loadSubtemplate($subtemplateName,$subtemplate)) { |
|
| 454 | + return false; |
|
| 455 | + } |
|
| 440 | 456 | // (Copying the template to insert a subtemplate is a bit slow. In a future implementation of MiniTemplator, |
| 441 | 457 | // a table could be used that contains references to the string fragments.) |
| 442 | 458 | $this->template = substr($this->template,0,$tPos1) . $subtemplate . substr($this->template,$tPos2); |
@@ -451,7 +467,9 @@ discard block |
||
| 451 | 467 | $p = 0; |
| 452 | 468 | while (true) {
|
| 453 | 469 | $p = strpos($this->template, '${', $p);
|
| 454 | - if ($p === false) break; |
|
| 470 | + if ($p === false) { |
|
| 471 | + break; |
|
| 472 | + } |
|
| 455 | 473 | $p0 = $p; |
| 456 | 474 | $p = strpos($this->template, '}', $p); |
| 457 | 475 | if ($p === false) {
|
@@ -469,8 +487,9 @@ discard block |
||
| 469 | 487 | * @access private |
| 470 | 488 | */ |
| 471 | 489 | function registerVariableReference ($varName, $tPosBegin, $tPosEnd) {
|
| 472 | - if (!$this->lookupVariableName($varName,$varNo)) |
|
| 473 | - $this->registerVariable($varName,$varNo); |
|
| 490 | + if (!$this->lookupVariableName($varName,$varNo)) { |
|
| 491 | + $this->registerVariable($varName,$varNo); |
|
| 492 | + } |
|
| 474 | 493 | $varRefNo = $this->varRefTabCnt++; |
| 475 | 494 | $vrtr =& $this->varRefTab[$varRefNo]; |
| 476 | 495 | $vrtr = array(); |
@@ -510,12 +529,14 @@ discard block |
||
| 510 | 529 | $nextBlockNo += 1; |
| 511 | 530 | continue; }} |
| 512 | 531 | $btr =& $this->blockTab[$activeBlockNo]; |
| 513 | - if ($varRefTPos < $btr['tPosBegin']) |
|
| 514 | - $this->programLogicError(1); |
|
| 532 | + if ($varRefTPos < $btr['tPosBegin']) { |
|
| 533 | + $this->programLogicError(1); |
|
| 534 | + } |
|
| 515 | 535 | $blockVarNo = $btr['blockVarCnt']++; |
| 516 | 536 | $btr['blockVarNoToVarNoMap'][$blockVarNo] = $varNo; |
| 517 | - if ($btr['firstVarRefNo'] == -1) |
|
| 518 | - $btr['firstVarRefNo'] = $varRefNo; |
|
| 537 | + if ($btr['firstVarRefNo'] == -1) { |
|
| 538 | + $btr['firstVarRefNo'] = $varRefNo; |
|
| 539 | + } |
|
| 519 | 540 | $vrtr['blockNo'] = $activeBlockNo; |
| 520 | 541 | $vrtr['blockVarNo'] = $blockVarNo; |
| 521 | 542 | $varRefNo += 1; }} |
@@ -531,8 +552,9 @@ discard block |
||
| 531 | 552 | * @access public |
| 532 | 553 | */ |
| 533 | 554 | function reset() {
|
| 534 | - for ($varNo=0; $varNo<$this->varTabCnt; $varNo++) |
|
| 535 | - $this->varTab[$varNo]['varValue'] = ''; |
|
| 555 | + for ($varNo=0; $varNo<$this->varTabCnt; $varNo++) { |
|
| 556 | + $this->varTab[$varNo]['varValue'] = ''; |
|
| 557 | + } |
|
| 536 | 558 | for ($blockNo=0; $blockNo<$this->blockTabCnt; $blockNo++) {
|
| 537 | 559 | $btr =& $this->blockTab[$blockNo]; |
| 538 | 560 | $btr['instances'] = 0; |
@@ -559,7 +581,9 @@ discard block |
||
| 559 | 581 | function setVariable ($variableName, $variableValue, $isOptional=false) {
|
| 560 | 582 | if (!$this->templateValid) {$this->triggerError ("Template not valid."); return false; }
|
| 561 | 583 | if (!$this->lookupVariableName($variableName,$varNo)) {
|
| 562 | - if ($isOptional) return true; |
|
| 584 | + if ($isOptional) { |
|
| 585 | + return true; |
|
| 586 | + } |
|
| 563 | 587 | $this->triggerError ("Variable \"$variableName\" not defined in template.");
|
| 564 | 588 | return false; } |
| 565 | 589 | $this->varTab[$varNo]['varValue'] = $variableValue; |
@@ -627,20 +651,23 @@ discard block |
||
| 627 | 651 | $btr = & $this->blockTab[$blockNo]; |
| 628 | 652 | $this->registerBlockInstance($blockInstNo); |
| 629 | 653 | $bitr = & $this->blockInstTab[$blockInstNo]; |
| 630 | - if ($btr['firstBlockInstNo'] == -1) |
|
| 631 | - $btr['firstBlockInstNo'] = $blockInstNo; |
|
| 632 | - if ($btr['lastBlockInstNo'] != -1) |
|
| 633 | - $this->blockInstTab[$btr['lastBlockInstNo']]['nextBlockInstNo'] = $blockInstNo; |
|
| 654 | + if ($btr['firstBlockInstNo'] == -1) { |
|
| 655 | + $btr['firstBlockInstNo'] = $blockInstNo; |
|
| 656 | + } |
|
| 657 | + if ($btr['lastBlockInstNo'] != -1) { |
|
| 658 | + $this->blockInstTab[$btr['lastBlockInstNo']]['nextBlockInstNo'] = $blockInstNo; |
|
| 659 | + } |
|
| 634 | 660 | // set forward pointer of chain |
| 635 | 661 | $btr['lastBlockInstNo'] = $blockInstNo; |
| 636 | 662 | $parentBlockNo = $btr['parentBlockNo']; |
| 637 | 663 | $blockVarCnt = $btr['blockVarCnt']; |
| 638 | 664 | $bitr['blockNo'] = $blockNo; |
| 639 | 665 | $bitr['instanceLevel'] = $btr['instances']++; |
| 640 | - if ($parentBlockNo == -1) |
|
| 641 | - $bitr['parentInstLevel'] = -1; |
|
| 642 | - else |
|
| 643 | - $bitr['parentInstLevel'] = $this->blockTab[$parentBlockNo]['instances']; |
|
| 666 | + if ($parentBlockNo == -1) { |
|
| 667 | + $bitr['parentInstLevel'] = -1; |
|
| 668 | + } else { |
|
| 669 | + $bitr['parentInstLevel'] = $this->blockTab[$parentBlockNo]['instances']; |
|
| 670 | + } |
|
| 644 | 671 | $bitr['nextBlockInstNo'] = -1; |
| 645 | 672 | $bitr['blockVarTab'] = array(); |
| 646 | 673 | // copy instance variables for this block |
@@ -687,7 +714,9 @@ discard block |
||
| 687 | 714 | */ |
| 688 | 715 | function generateOutputToFile($fileName) {
|
| 689 | 716 | $fh = fopen($fileName, "wb"); |
| 690 | - if ($fh === false) return false; |
|
| 717 | + if ($fh === false) { |
|
| 718 | + return false; |
|
| 719 | + } |
|
| 691 | 720 | $this->outputMode = 1; |
| 692 | 721 | $this->outputFileHandle = $fh; |
| 693 | 722 | $ok = $this->generateOutputPage(); |
@@ -717,14 +746,18 @@ discard block |
||
| 717 | 746 | */ |
| 718 | 747 | function generateOutputPage() {
|
| 719 | 748 | if (!$this->templateValid) {$this->triggerError ("Template not valid."); return false; }
|
| 720 | - if ($this->blockTab[0]['instances'] == 0) |
|
| 721 | - $this->addBlockByNo (0); // add main block |
|
| 749 | + if ($this->blockTab[0]['instances'] == 0) { |
|
| 750 | + $this->addBlockByNo (0); |
|
| 751 | + } |
|
| 752 | + // add main block |
|
| 722 | 753 | for ($blockNo=0; $blockNo < $this->blockTabCnt; $blockNo++) {
|
| 723 | 754 | $btr =& $this->blockTab[$blockNo]; |
| 724 | 755 | $btr['currBlockInstNo'] = $btr['firstBlockInstNo']; } |
| 725 | 756 | $this->outputError = false; |
| 726 | 757 | $this->writeBlockInstances (0, -1); |
| 727 | - if ($this->outputError) return false; |
|
| 758 | + if ($this->outputError) { |
|
| 759 | + return false; |
|
| 760 | + } |
|
| 728 | 761 | return true; } |
| 729 | 762 | |
| 730 | 763 | /** |
@@ -737,11 +770,16 @@ discard block |
||
| 737 | 770 | $btr =& $this->blockTab[$blockNo]; |
| 738 | 771 | while (!$this->outputError) {
|
| 739 | 772 | $blockInstNo = $btr['currBlockInstNo']; |
| 740 | - if ($blockInstNo == -1) break; |
|
| 773 | + if ($blockInstNo == -1) { |
|
| 774 | + break; |
|
| 775 | + } |
|
| 741 | 776 | $bitr =& $this->blockInstTab[$blockInstNo]; |
| 742 | - if ($bitr['parentInstLevel'] < $parentInstLevel) |
|
| 743 | - $this->programLogicError (2); |
|
| 744 | - if ($bitr['parentInstLevel'] > $parentInstLevel) break; |
|
| 777 | + if ($bitr['parentInstLevel'] < $parentInstLevel) { |
|
| 778 | + $this->programLogicError (2); |
|
| 779 | + } |
|
| 780 | + if ($bitr['parentInstLevel'] > $parentInstLevel) { |
|
| 781 | + break; |
|
| 782 | + } |
|
| 745 | 783 | $this->writeBlockInstance ($blockInstNo); |
| 746 | 784 | $btr['currBlockInstNo'] = $bitr['nextBlockInstNo']; }} |
| 747 | 785 | |
@@ -774,15 +812,17 @@ discard block |
||
| 774 | 812 | if ($subBtr['tPosBegin'] < $tPos2) {
|
| 775 | 813 | $tPos2 = $subBtr['tPosBegin']; |
| 776 | 814 | $kind = 2; }} |
| 777 | - if ($tPos2 > $tPos) |
|
| 778 | - $this->writeString (substr($this->template,$tPos,$tPos2-$tPos)); |
|
| 815 | + if ($tPos2 > $tPos) { |
|
| 816 | + $this->writeString (substr($this->template,$tPos,$tPos2-$tPos)); |
|
| 817 | + } |
|
| 779 | 818 | switch ($kind) {
|
| 780 | 819 | case 0: // end of block |
| 781 | 820 | return; |
| 782 | 821 | case 1: // variable |
| 783 | 822 | $vrtr =& $this->varRefTab[$varRefNo]; |
| 784 | - if ($vrtr['blockNo'] != $blockNo) |
|
| 785 | - $this->programLogicError (4); |
|
| 823 | + if ($vrtr['blockNo'] != $blockNo) { |
|
| 824 | + $this->programLogicError (4); |
|
| 825 | + } |
|
| 786 | 826 | $variableValue = $bitr['blockVarTab'][$vrtr['blockVarNo']]; |
| 787 | 827 | $this->writeString ($variableValue); |
| 788 | 828 | $tPos = $vrtr['tPosEnd']; |
@@ -790,8 +830,9 @@ discard block |
||
| 790 | 830 | break; |
| 791 | 831 | case 2: // sub block |
| 792 | 832 | $subBtr =& $this->blockTab[$subBlockNo]; |
| 793 | - if ($subBtr['parentBlockNo'] != $blockNo) |
|
| 794 | - $this->programLogicError (3); |
|
| 833 | + if ($subBtr['parentBlockNo'] != $blockNo) { |
|
| 834 | + $this->programLogicError (3); |
|
| 835 | + } |
|
| 795 | 836 | $this->writeBlockInstances ($subBlockNo, $bitr['instanceLevel']); // recursive call |
| 796 | 837 | $tPos = $subBtr['tPosEnd']; |
| 797 | 838 | $subBlockNo += 1; |
@@ -801,15 +842,20 @@ discard block |
||
| 801 | 842 | * @access private |
| 802 | 843 | */ |
| 803 | 844 | function writeString($s) {
|
| 804 | - if ($this->outputError) return; |
|
| 845 | + if ($this->outputError) { |
|
| 846 | + return; |
|
| 847 | + } |
|
| 805 | 848 | switch ($this->outputMode) {
|
| 806 | 849 | case 0: // output to PHP output stream |
| 807 | - if (!print($s)) |
|
| 808 | - $this->outputError = true; |
|
| 850 | + if (!print($s)) { |
|
| 851 | + $this->outputError = true; |
|
| 852 | + } |
|
| 809 | 853 | break; |
| 810 | 854 | case 1: // output to file |
| 811 | 855 | $rc = fwrite($this->outputFileHandle, $s); |
| 812 | - if ($rc === false) $this->outputError = true; |
|
| 856 | + if ($rc === false) { |
|
| 857 | + $this->outputError = true; |
|
| 858 | + } |
|
| 813 | 859 | break; |
| 814 | 860 | case 2: // output to string |
| 815 | 861 | $this->outputString .= $s; |
@@ -824,7 +870,9 @@ discard block |
||
| 824 | 870 | */ |
| 825 | 871 | function lookupVariableName($varName, &$varNo) {
|
| 826 | 872 | $x = & $this->varNameToNoMap[strtoupper($varName)]; |
| 827 | - if (!isset($x)) return false; |
|
| 873 | + if (!isset($x)) { |
|
| 874 | + return false; |
|
| 875 | + } |
|
| 828 | 876 | $varNo = $x; |
| 829 | 877 | return true; } |
| 830 | 878 | |
@@ -837,7 +885,9 @@ discard block |
||
| 837 | 885 | */ |
| 838 | 886 | function lookupBlockName($blockName, &$blockNo) {
|
| 839 | 887 | $x = & $this->blockNameToNoMap[strtoupper($blockName)]; |
| 840 | - if (!isset($x)) return false; |
|
| 888 | + if (!isset($x)) { |
|
| 889 | + return false; |
|
| 890 | + } |
|
| 841 | 891 | $blockNo = $x; |
| 842 | 892 | return true; } |
| 843 | 893 | |
@@ -851,15 +901,21 @@ discard block |
||
| 851 | 901 | function readFileIntoString ($fileName, &$s) {
|
| 852 | 902 | if (function_exists('version_compare') && version_compare(phpversion(),"4.3.0",">=")) {
|
| 853 | 903 | $s = file_get_contents($fileName); |
| 854 | - if ($s === false) return false; |
|
| 904 | + if ($s === false) { |
|
| 905 | + return false; |
|
| 906 | + } |
|
| 855 | 907 | return true; } |
| 856 | 908 | $fh = fopen($fileName,"rb"); |
| 857 | - if ($fh === false) return false; |
|
| 909 | + if ($fh === false) { |
|
| 910 | + return false; |
|
| 911 | + } |
|
| 858 | 912 | $fileSize = filesize($fileName); |
| 859 | 913 | if ($fileSize === false) {fclose ($fh); return false; }
|
| 860 | 914 | $s = fread($fh,$fileSize); |
| 861 | 915 | fclose ($fh); |
| 862 | - if (strlen($s) != $fileSize) return false; |
|
| 916 | + if (strlen($s) != $fileSize) { |
|
| 917 | + return false; |
|
| 918 | + } |
|
| 863 | 919 | return true; } |
| 864 | 920 | |
| 865 | 921 | /** |
@@ -887,12 +943,22 @@ discard block |
||
| 887 | 943 | */ |
| 888 | 944 | function parseQuotedString($s, &$p, &$w) {
|
| 889 | 945 | $sLen = strlen($s); |
| 890 | - while ($p < $sLen && ord($s{$p}) <= 32) $p++;
|
|
| 891 | - if ($p >= $sLen) return false; |
|
| 892 | - if (substr($s, $p, 1) != '"') return false; |
|
| 946 | + while ($p < $sLen && ord($s{$p}) <= 32) { |
|
| 947 | + $p++; |
|
| 948 | + } |
|
| 949 | + if ($p >= $sLen) { |
|
| 950 | + return false; |
|
| 951 | + } |
|
| 952 | + if (substr($s, $p, 1) != '"') { |
|
| 953 | + return false; |
|
| 954 | + } |
|
| 893 | 955 | $p++; $p0 = $p; |
| 894 | - while ($p < $sLen && $s{$p} != '"') $p++;
|
|
| 895 | - if ($p >= $sLen) return false; |
|
| 956 | + while ($p < $sLen && $s{$p} != '"') { |
|
| 957 | + $p++; |
|
| 958 | + } |
|
| 959 | + if ($p >= $sLen) { |
|
| 960 | + return false; |
|
| 961 | + } |
|
| 896 | 962 | $w = substr($s, $p0, $p - $p0); |
| 897 | 963 | $p++; |
| 898 | 964 | return true; } |
@@ -903,25 +969,36 @@ discard block |
||
| 903 | 969 | */ |
| 904 | 970 | function parseWordOrQuotedString ($s, &$p, &$w) {
|
| 905 | 971 | $sLen = strlen($s); |
| 906 | - while ($p < $sLen && ord($s{$p}) <= 32) $p++;
|
|
| 907 | - if ($p >= $sLen) return false; |
|
| 908 | - if (substr($s,$p,1) == '"') |
|
| 909 | - return $this->parseQuotedString($s,$p,$w); |
|
| 910 | - else |
|
| 911 | - return $this->parseWord($s,$p,$w); } |
|
| 972 | + while ($p < $sLen && ord($s{$p}) <= 32) { |
|
| 973 | + $p++; |
|
| 974 | + } |
|
| 975 | + if ($p >= $sLen) { |
|
| 976 | + return false; |
|
| 977 | + } |
|
| 978 | + if (substr($s,$p,1) == '"') { |
|
| 979 | + return $this->parseQuotedString($s,$p,$w); |
|
| 980 | + } else { |
|
| 981 | + return $this->parseWord($s,$p,$w); |
|
| 982 | + } |
|
| 983 | + } |
|
| 912 | 984 | |
| 913 | 985 | /** |
| 914 | 986 | * Combine two file system paths. |
| 915 | 987 | * @access private |
| 916 | 988 | */ |
| 917 | 989 | function combineFileSystemPath ($path1, $path2) {
|
| 918 | - if ($path1 == '' || $path2 == '') return $path2; |
|
| 990 | + if ($path1 == '' || $path2 == '') { |
|
| 991 | + return $path2; |
|
| 992 | + } |
|
| 919 | 993 | $s = $path1; |
| 920 | - if (substr($s,-1) != '\\' && substr($s,-1) != '/') $s = $s . "/"; |
|
| 921 | - if (substr($path2,0,1) == '\\' || substr($path2,0,1) == '/') |
|
| 922 | - $s = $s . substr($path2,1); |
|
| 923 | - else |
|
| 924 | - $s = $s . $path2; |
|
| 994 | + if (substr($s,-1) != '\\' && substr($s,-1) != '/') { |
|
| 995 | + $s = $s . "/"; |
|
| 996 | + } |
|
| 997 | + if (substr($path2,0,1) == '\\' || substr($path2,0,1) == '/') { |
|
| 998 | + $s = $s . substr($path2,1); |
|
| 999 | + } else { |
|
| 1000 | + $s = $s . $path2; |
|
| 1001 | + } |
|
| 925 | 1002 | return $s; } |
| 926 | 1003 | |
| 927 | 1004 | /** |
@@ -445,10 +445,12 @@ discard block |
||
| 445 | 445 | |
| 446 | 446 | if (!$hide_images) { |
| 447 | 447 | $encsize = ''; |
| 448 | - if ($entry['height'] > 0) |
|
| 449 | - $encsize .= ' height="'.intval($entry['height']).'"'; |
|
| 450 | - if ($entry['width'] > 0) |
|
| 451 | - $encsize .= ' width="'.intval($entry['width']).'"'; |
|
| 448 | + if ($entry['height'] > 0) { |
|
| 449 | + $encsize .= ' height="'.intval($entry['height']).'"'; |
|
| 450 | + } |
|
| 451 | + if ($entry['width'] > 0) { |
|
| 452 | + $encsize .= ' width="'.intval($entry['width']).'"'; |
|
| 453 | + } |
|
| 452 | 454 | $rv .= "<p><img |
| 453 | 455 | alt=\"".htmlspecialchars($entry["filename"])."\" |
| 454 | 456 | src=\"" .htmlspecialchars($entry["url"])."\" |
@@ -480,15 +482,17 @@ discard block |
||
| 480 | 482 | $rv .= "<div dojoType=\"dijit.Menu\" style=\"display: none;\">"; |
| 481 | 483 | |
| 482 | 484 | foreach ($entries as $entry) { |
| 483 | - if ($entry["title"]) |
|
| 484 | - $title = " — ".truncate_string($entry["title"], 30); |
|
| 485 | - else |
|
| 486 | - $title = ""; |
|
| 485 | + if ($entry["title"]) { |
|
| 486 | + $title = " — ".truncate_string($entry["title"], 30); |
|
| 487 | + } else { |
|
| 488 | + $title = ""; |
|
| 489 | + } |
|
| 487 | 490 | |
| 488 | - if ($entry["filename"]) |
|
| 489 | - $filename = truncate_middle(htmlspecialchars($entry["filename"]), 60); |
|
| 490 | - else |
|
| 491 | - $filename = ""; |
|
| 491 | + if ($entry["filename"]) { |
|
| 492 | + $filename = truncate_middle(htmlspecialchars($entry["filename"]), 60); |
|
| 493 | + } else { |
|
| 494 | + $filename = ""; |
|
| 495 | + } |
|
| 492 | 496 | |
| 493 | 497 | $rv .= "<div onclick='popupOpenUrl(\"".htmlspecialchars($entry["url"])."\")' |
| 494 | 498 | dojoType=\"dijit.MenuItem\">".$filename.$title."</div>"; |
@@ -639,10 +643,11 @@ discard block |
||
| 639 | 643 | |
| 640 | 644 | // purge orphaned posts in main content table |
| 641 | 645 | |
| 642 | - if (DB_TYPE == "mysql") |
|
| 643 | - $limit_qpart = "LIMIT 5000"; |
|
| 644 | - else |
|
| 645 | - $limit_qpart = ""; |
|
| 646 | + if (DB_TYPE == "mysql") { |
|
| 647 | + $limit_qpart = "LIMIT 5000"; |
|
| 648 | + } else { |
|
| 649 | + $limit_qpart = ""; |
|
| 650 | + } |
|
| 646 | 651 | |
| 647 | 652 | $pdo = Db::pdo(); |
| 648 | 653 | $res = $pdo->query("DELETE FROM ttrss_entries WHERE |
@@ -26,8 +26,10 @@ |
||
| 26 | 26 | if ($error) { |
| 27 | 27 | foreach (libxml_get_errors() as $error) { |
| 28 | 28 | if ($error->level == LIBXML_ERR_FATAL) { |
| 29 | - if (!isset($this->error)) //currently only the first error is reported |
|
| 29 | + if (!isset($this->error)) { |
|
| 30 | + //currently only the first error is reported |
|
| 30 | 31 | $this->error = $this->format_error($error); |
| 32 | + } |
|
| 31 | 33 | $this->libxml_errors [] = $this->format_error($error); |
| 32 | 34 | } |
| 33 | 35 | } |
@@ -50,8 +50,9 @@ discard block |
||
| 50 | 50 | $reply .= strip_tags($feed_title); |
| 51 | 51 | } |
| 52 | 52 | |
| 53 | - if ($error) |
|
| 54 | - $reply .= " <i title=\"".htmlspecialchars($error)."\" class='material-icons icon-error'>error</i>"; |
|
| 53 | + if ($error) { |
|
| 54 | + $reply .= " <i title=\"".htmlspecialchars($error)."\" class='material-icons icon-error'>error</i>"; |
|
| 55 | + } |
|
| 55 | 56 | |
| 56 | 57 | $reply .= "</span>"; |
| 57 | 58 | $reply .= "<span id='feed_current_unread' style='display: none'></span>"; |
@@ -325,10 +326,11 @@ discard block |
||
| 325 | 326 | |
| 326 | 327 | $line['content'] = DiskCache::rewriteUrls($line['content']); |
| 327 | 328 | |
| 328 | - if ($line['note']) |
|
| 329 | - $line['note'] = Article::format_article_note($id, $line['note']); |
|
| 330 | - else |
|
| 331 | - $line['note'] = ""; |
|
| 329 | + if ($line['note']) { |
|
| 330 | + $line['note'] = Article::format_article_note($id, $line['note']); |
|
| 331 | + } else { |
|
| 332 | + $line['note'] = ""; |
|
| 333 | + } |
|
| 332 | 334 | |
| 333 | 335 | if (!get_pref("CDM_EXPANDED")) { |
| 334 | 336 | $line["cdm_excerpt"] = "<span class='collapse'> |
@@ -395,8 +397,9 @@ discard block |
||
| 395 | 397 | /* we don't need those */ |
| 396 | 398 | |
| 397 | 399 | foreach (["date_entered", "guid", "last_published", "last_marked", "tag_cache", "favicon_avg_color", |
| 398 | - "uuid", "label_cache", "yyiw"] as $k) |
|
| 399 | - unset($line[$k]); |
|
| 400 | + "uuid", "label_cache", "yyiw"] as $k) { |
|
| 401 | + unset($line[$k]); |
|
| 402 | + } |
|
| 400 | 403 | |
| 401 | 404 | array_push($reply['content'], $line); |
| 402 | 405 | } |
@@ -1310,13 +1313,17 @@ discard block |
||
| 1310 | 1313 | |
| 1311 | 1314 | public static function getCategoryUnread($cat, $owner_uid = false) { |
| 1312 | 1315 | |
| 1313 | - if (!$owner_uid) $owner_uid = $_SESSION["uid"]; |
|
| 1316 | + if (!$owner_uid) { |
|
| 1317 | + $owner_uid = $_SESSION["uid"]; |
|
| 1318 | + } |
|
| 1314 | 1319 | |
| 1315 | 1320 | $pdo = Db::pdo(); |
| 1316 | 1321 | |
| 1317 | 1322 | if ($cat >= 0) { |
| 1318 | 1323 | |
| 1319 | - if (!$cat) $cat = null; |
|
| 1324 | + if (!$cat) { |
|
| 1325 | + $cat = null; |
|
| 1326 | + } |
|
| 1320 | 1327 | |
| 1321 | 1328 | $sth = $pdo->prepare("SELECT id FROM ttrss_feeds |
| 1322 | 1329 | WHERE (cat_id = :cat OR (:cat IS NULL AND cat_id IS NULL)) |
@@ -2279,56 +2286,68 @@ discard block |
||
| 2279 | 2286 | break; |
| 2280 | 2287 | case "note": |
| 2281 | 2288 | if ($commandpair[1]) { |
| 2282 | - if ($commandpair[1] == "true") |
|
| 2283 | - array_push($query_keywords, "($not (note IS NOT NULL AND note != ''))"); |
|
| 2284 | - else if ($commandpair[1] == "false") |
|
| 2285 | - array_push($query_keywords, "($not (note IS NULL OR note = ''))"); |
|
| 2286 | - else |
|
| 2287 | - array_push($query_keywords, "($not (LOWER(note) LIKE ". |
|
| 2289 | + if ($commandpair[1] == "true") { |
|
| 2290 | + array_push($query_keywords, "($not (note IS NOT NULL AND note != ''))"); |
|
| 2291 | + } else if ($commandpair[1] == "false") { |
|
| 2292 | + array_push($query_keywords, "($not (note IS NULL OR note = ''))"); |
|
| 2293 | + } else { |
|
| 2294 | + array_push($query_keywords, "($not (LOWER(note) LIKE ". |
|
| 2288 | 2295 | $pdo->quote('%' . mb_strtolower($commandpair[1]) . '%')."))"); |
| 2296 | + } |
|
| 2289 | 2297 | } else { |
| 2290 | 2298 | array_push($query_keywords, "(UPPER(ttrss_entries.title) $not LIKE UPPER(".$pdo->quote("%$k%").") |
| 2291 | 2299 | OR UPPER(ttrss_entries.content) $not LIKE UPPER(".$pdo->quote("%$k%")."))"); |
| 2292 | - if (!$not) array_push($search_words, $k); |
|
| 2300 | + if (!$not) { |
|
| 2301 | + array_push($search_words, $k); |
|
| 2302 | + } |
|
| 2293 | 2303 | } |
| 2294 | 2304 | break; |
| 2295 | 2305 | case "star": |
| 2296 | 2306 | |
| 2297 | 2307 | if ($commandpair[1]) { |
| 2298 | - if ($commandpair[1] == "true") |
|
| 2299 | - array_push($query_keywords, "($not (marked = true))"); |
|
| 2300 | - else |
|
| 2301 | - array_push($query_keywords, "($not (marked = false))"); |
|
| 2308 | + if ($commandpair[1] == "true") { |
|
| 2309 | + array_push($query_keywords, "($not (marked = true))"); |
|
| 2310 | + } else { |
|
| 2311 | + array_push($query_keywords, "($not (marked = false))"); |
|
| 2312 | + } |
|
| 2302 | 2313 | } else { |
| 2303 | 2314 | array_push($query_keywords, "(UPPER(ttrss_entries.title) $not LIKE UPPER(".$pdo->quote("%$k%").") |
| 2304 | 2315 | OR UPPER(ttrss_entries.content) $not LIKE UPPER(".$pdo->quote("%$k%")."))"); |
| 2305 | - if (!$not) array_push($search_words, $k); |
|
| 2316 | + if (!$not) { |
|
| 2317 | + array_push($search_words, $k); |
|
| 2318 | + } |
|
| 2306 | 2319 | } |
| 2307 | 2320 | break; |
| 2308 | 2321 | case "pub": |
| 2309 | 2322 | if ($commandpair[1]) { |
| 2310 | - if ($commandpair[1] == "true") |
|
| 2311 | - array_push($query_keywords, "($not (published = true))"); |
|
| 2312 | - else |
|
| 2313 | - array_push($query_keywords, "($not (published = false))"); |
|
| 2323 | + if ($commandpair[1] == "true") { |
|
| 2324 | + array_push($query_keywords, "($not (published = true))"); |
|
| 2325 | + } else { |
|
| 2326 | + array_push($query_keywords, "($not (published = false))"); |
|
| 2327 | + } |
|
| 2314 | 2328 | |
| 2315 | 2329 | } else { |
| 2316 | 2330 | array_push($query_keywords, "(UPPER(ttrss_entries.title) $not LIKE UPPER('%$k%') |
| 2317 | 2331 | OR UPPER(ttrss_entries.content) $not LIKE UPPER(".$pdo->quote("%$k%")."))"); |
| 2318 | - if (!$not) array_push($search_words, $k); |
|
| 2332 | + if (!$not) { |
|
| 2333 | + array_push($search_words, $k); |
|
| 2334 | + } |
|
| 2319 | 2335 | } |
| 2320 | 2336 | break; |
| 2321 | 2337 | case "unread": |
| 2322 | 2338 | if ($commandpair[1]) { |
| 2323 | - if ($commandpair[1] == "true") |
|
| 2324 | - array_push($query_keywords, "($not (unread = true))"); |
|
| 2325 | - else |
|
| 2326 | - array_push($query_keywords, "($not (unread = false))"); |
|
| 2339 | + if ($commandpair[1] == "true") { |
|
| 2340 | + array_push($query_keywords, "($not (unread = true))"); |
|
| 2341 | + } else { |
|
| 2342 | + array_push($query_keywords, "($not (unread = false))"); |
|
| 2343 | + } |
|
| 2327 | 2344 | |
| 2328 | 2345 | } else { |
| 2329 | 2346 | array_push($query_keywords, "(UPPER(ttrss_entries.title) $not LIKE UPPER(".$pdo->quote("%$k%").") |
| 2330 | 2347 | OR UPPER(ttrss_entries.content) $not LIKE UPPER(".$pdo->quote("%$k%")."))"); |
| 2331 | - if (!$not) array_push($search_words, $k); |
|
| 2348 | + if (!$not) { |
|
| 2349 | + array_push($search_words, $k); |
|
| 2350 | + } |
|
| 2332 | 2351 | } |
| 2333 | 2352 | break; |
| 2334 | 2353 | default: |
@@ -2351,7 +2370,9 @@ discard block |
||
| 2351 | 2370 | OR UPPER(ttrss_entries.content) $not LIKE UPPER(".$pdo->quote("%$k%")."))"); |
| 2352 | 2371 | } |
| 2353 | 2372 | |
| 2354 | - if (!$not) array_push($search_words, $k); |
|
| 2373 | + if (!$not) { |
|
| 2374 | + array_push($search_words, $k); |
|
| 2375 | + } |
|
| 2355 | 2376 | } |
| 2356 | 2377 | } |
| 2357 | 2378 | } |