Inspection of "content background" - injitools/cms-Inji - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e7c2e1...989877 )

by Alexey

created 2016-08-09 04:33 UTC

Status

Indentation +87 added lines, -87 removed lines patch added patch discarded remove patch

@@ -8,11 +8,11 @@  discard block
 block discarded – undo
         </tr>
       </thead>
       <?php
-      $orderDeatilCols = [
-          'id' => '№ Заказа',
-          'complete_data' => 'Дата заказа'
-      ];
-      ?>
+        $orderDeatilCols = [
+            'id' => '№ Заказа',
+            'complete_data' => 'Дата заказа'
+        ];
+        ?>
       <tbody>
         <tr>
           <td class="text-left" style="width: 50%;">              
@@ -65,91 +65,91 @@  discard block
 block discarded – undo
         </thead>
         <tbody>
           <?php
-          $sums = [];
-          foreach ($cart->cartItems as $cartItem) {
-              $item = $cartItem->item;
-              $itemName = $item->name();
-              if (!isset($sums[$cartItem->price->currency_id])) {
-                  $sums[$cartItem->price->currency_id] = $cartItem->price->price;
-              } else {
-                  $sums[$cartItem->price->currency_id] += $cartItem->price->price;
-              }
-              ?>
+            $sums = [];
+            foreach ($cart->cartItems as $cartItem) {
+                $item = $cartItem->item;
+                $itemName = $item->name();
+                if (!isset($sums[$cartItem->price->currency_id])) {
+                    $sums[$cartItem->price->currency_id] = $cartItem->price->price;
+                } else {
+                    $sums[$cartItem->price->currency_id] += $cartItem->price->price;
+                }
+                ?>
               <tr>
                 <td class="text-left"><?= $itemName; ?></td>
                 <td class="text-right"><?= $cartItem->count; ?></td>
                 <td class="text-right"><?= $cartItem->price->price; ?> 
                   <?php
-                  if (App::$cur->money) {
-                      $currency = Money\Currency::get($cartItem->price->currency_id);
-                      if ($currency) {
-                          echo $currency->acronym();
-                      } else {
-                          echo 'руб.';
-                      }
-                  } else {
-                      echo 'руб.';
-                  }
-                  ?></td>
+                    if (App::$cur->money) {
+                        $currency = Money\Currency::get($cartItem->price->currency_id);
+                        if ($currency) {
+                            echo $currency->acronym();
+                        } else {
+                            echo 'руб.';
+                        }
+                    } else {
+                        echo 'руб.';
+                    }
+                    ?></td>
                 <td class="text-right"><?= ($cartItem->price->price * $cartItem->count); ?>
                   <?php
-                  if (App::$cur->money) {
-                      $currency = Money\Currency::get($cartItem->price->currency_id);
-                      if ($currency) {
-                          echo $currency->acronym();
-                      } else {
-                          echo 'руб.';
-                      }
-                  } else {
-                      echo 'руб.';
-                  }
-                  ?></td>
+                    if (App::$cur->money) {
+                        $currency = Money\Currency::get($cartItem->price->currency_id);
+                        if ($currency) {
+                            echo $currency->acronym();
+                        } else {
+                            echo 'руб.';
+                        }
+                    } else {
+                        echo 'руб.';
+                    }
+                    ?></td>
                 <td class="text-right" style="white-space: nowrap;">
                   <a onclick = 'inji.Ecommerce.Cart.addItem(<?= $item->getPrice()->id; ?>, 1);
                           return false;' data-original-title="Добавить в корзину" href="#" data-toggle="tooltip" title="" class="btn btn-primary"><i class="glyphicon glyphicon-shopping-cart"></i></a>
               </tr>
               <?php
-          }
-          ?>
+            }
+            ?>
         </tbody>
         <tfoot>
           <tr>
             <td colspan="2"></td>
             <td class="text-right"><b>Сумма</b></td>
             <td class="text-right"><?php
-              foreach ($sums as $currency_id => $sum) {
-                  if (!$sum) {
-                      continue;
-                  }
-                  echo number_format($sum, 2, '.', ' ');
-                  if (App::$cur->money) {
-                      $currency = Money\Currency::get($currency_id);
-                      if ($currency) {
-                          echo '&nbsp;' . $currency->acronym();
-                      } else {
-                          echo '&nbsp;руб.';
-                      }
-                  } else {
-                      echo '&nbsp;руб.';
-                  }
-                  echo '<br />';
-              }
-              ?></td>
+                foreach ($sums as $currency_id => $sum) {
+                    if (!$sum) {
+                        continue;
+                    }
+                    echo number_format($sum, 2, '.', ' ');
+                    if (App::$cur->money) {
+                        $currency = Money\Currency::get($currency_id);
+                        if ($currency) {
+                            echo '&nbsp;' . $currency->acronym();
+                        } else {
+                            echo '&nbsp;руб.';
+                        }
+                    } else {
+                        echo '&nbsp;руб.';
+                    }
+                    echo '<br />';
+                }
+                ?></td>
             <td></td>
           </tr>
           <?php
-          $deliveryCurrency = $cart->delivery ? $cart->delivery->currency_id : 0;
-          $deliveryPrice = $cart->delivery->price;
-          if (!isset($sums[$deliveryCurrency])) {
-              $sums[$deliveryCurrency] = $deliveryPrice;
-          } else {
-              $sums[$deliveryCurrency] += $deliveryPrice;
-          }
-          $delCurrency = false;
-          if (App::$cur->money) {
-              $delCurrency = Money\Currency::get($deliveryPrice);
-          }
-          ?>
+            $deliveryCurrency = $cart->delivery ? $cart->delivery->currency_id : 0;
+            $deliveryPrice = $cart->delivery->price;
+            if (!isset($sums[$deliveryCurrency])) {
+                $sums[$deliveryCurrency] = $deliveryPrice;
+            } else {
+                $sums[$deliveryCurrency] += $deliveryPrice;
+            }
+            $delCurrency = false;
+            if (App::$cur->money) {
+                $delCurrency = Money\Currency::get($deliveryPrice);
+            }
+            ?>
           <tr>
             <td colspan="2"></td>
             <td class="text-right"><b><?= $cart->delivery ? $cart->delivery->name : 'Доставка'; ?></b></td>
@@ -160,24 +160,24 @@  discard block
 block discarded – undo
             <td colspan="2"></td>
             <td class="text-right"><b>Итого</b></td>
             <td class="text-right"><?php
-              foreach ($sums as $currency_id => $sum) {
-                  if (!$sum) {
-                      continue;
-                  }
-                  echo number_format($sum, 2, '.', ' ');
-                  if (App::$cur->money) {
-                      $currency = Money\Currency::get($currency_id);
-                      if ($currency) {
-                          echo '&nbsp;' . $currency->acronym();
-                      } else {
-                          echo '&nbsp;руб.';
-                      }
-                  } else {
-                      echo '&nbsp;руб.';
-                  }
-                  echo '<br />';
-              }
-              ?></td>
+                foreach ($sums as $currency_id => $sum) {
+                    if (!$sum) {
+                        continue;
+                    }
+                    echo number_format($sum, 2, '.', ' ');
+                    if (App::$cur->money) {
+                        $currency = Money\Currency::get($currency_id);
+                        if ($currency) {
+                            echo '&nbsp;' . $currency->acronym();
+                        } else {
+                            echo '&nbsp;руб.';
+                        }
+                    } else {
+                        echo '&nbsp;руб.';
+                    }
+                    echo '<br />';
+                }
+                ?></td>
             <td></td>
           </tr>
         </tfoot>

Please login to merge, or discard this patch.

		@@ -8,11 +8,11 @@ discard block
		block discarded – undo
8	8	</tr>
9	9	</thead>
10	10	<?php
11		- $orderDeatilCols = [
12		- 'id' => '№ Заказа',
13		- 'complete_data' => 'Дата заказа'
14		- ];
15		- ?>
	11	+ $orderDeatilCols = [
	12	+ 'id' => '№ Заказа',
	13	+ 'complete_data' => 'Дата заказа'
	14	+ ];
	15	+ ?>
16	16	<tbody>
17	17	<tr>
18	18	<td class="text-left" style="width: 50%;">
		@@ -65,91 +65,91 @@ discard block
		block discarded – undo
65	65	</thead>
66	66	<tbody>
67	67	<?php
68		- $sums = [];
69		- foreach ($cart->cartItems as $cartItem) {
70		- $item = $cartItem->item;
71		- $itemName = $item->name();
72		- if (!isset($sums[$cartItem->price->currency_id])) {
73		- $sums[$cartItem->price->currency_id] = $cartItem->price->price;
74		- } else {
75		- $sums[$cartItem->price->currency_id] += $cartItem->price->price;
76		- }
77		- ?>
	68	+ $sums = [];
	69	+ foreach ($cart->cartItems as $cartItem) {
	70	+ $item = $cartItem->item;
	71	+ $itemName = $item->name();
	72	+ if (!isset($sums[$cartItem->price->currency_id])) {
	73	+ $sums[$cartItem->price->currency_id] = $cartItem->price->price;
	74	+ } else {
	75	+ $sums[$cartItem->price->currency_id] += $cartItem->price->price;
	76	+ }
	77	+ ?>
78	78	<tr>
79	79	<td class="text-left"><?= $itemName; ?></td>
80	80	<td class="text-right"><?= $cartItem->count; ?></td>
81	81	<td class="text-right"><?= $cartItem->price->price; ?>
82	82	<?php
83		- if (App::$cur->money) {
84		- $currency = Money\Currency::get($cartItem->price->currency_id);
85		- if ($currency) {
86		- echo $currency->acronym();
87		- } else {
88		- echo 'руб.';
89		- }
90		- } else {
91		- echo 'руб.';
92		- }
93		- ?></td>
	83	+ if (App::$cur->money) {
	84	+ $currency = Money\Currency::get($cartItem->price->currency_id);
	85	+ if ($currency) {
	86	+ echo $currency->acronym();
	87	+ } else {
	88	+ echo 'руб.';
	89	+ }
	90	+ } else {
	91	+ echo 'руб.';
	92	+ }
	93	+ ?></td>
94	94	<td class="text-right"><?= ($cartItem->price->price * $cartItem->count); ?>
95	95	<?php
96		- if (App::$cur->money) {
97		- $currency = Money\Currency::get($cartItem->price->currency_id);
98		- if ($currency) {
99		- echo $currency->acronym();
100		- } else {
101		- echo 'руб.';
102		- }
103		- } else {
104		- echo 'руб.';
105		- }
106		- ?></td>
	96	+ if (App::$cur->money) {
	97	+ $currency = Money\Currency::get($cartItem->price->currency_id);
	98	+ if ($currency) {
	99	+ echo $currency->acronym();
	100	+ } else {
	101	+ echo 'руб.';
	102	+ }
	103	+ } else {
	104	+ echo 'руб.';
	105	+ }
	106	+ ?></td>
107	107	<td class="text-right" style="white-space: nowrap;">
108	108	<a onclick = 'inji.Ecommerce.Cart.addItem(<?= $item->getPrice()->id; ?>, 1);
109	109	return false;' data-original-title="Добавить в корзину" href="#" data-toggle="tooltip" title="" class="btn btn-primary"><i class="glyphicon glyphicon-shopping-cart"></i></a>
110	110	</tr>
111	111	<?php
112		- }
113		- ?>
	112	+ }
	113	+ ?>
114	114	</tbody>
115	115	<tfoot>
116	116	<tr>
117	117	<td colspan="2"></td>
118	118	<td class="text-right"><b>Сумма</b></td>
119	119	<td class="text-right"><?php
120		- foreach ($sums as $currency_id => $sum) {
121		- if (!$sum) {
122		- continue;
123		- }
124		- echo number_format($sum, 2, '.', ' ');
125		- if (App::$cur->money) {
126		- $currency = Money\Currency::get($currency_id);
127		- if ($currency) {
128		- echo ' ' . $currency->acronym();
129		- } else {
130		- echo ' руб.';
131		- }
132		- } else {
133		- echo ' руб.';
134		- }
135		- echo '<br />';
136		- }
137		- ?></td>
	120	+ foreach ($sums as $currency_id => $sum) {
	121	+ if (!$sum) {
	122	+ continue;
	123	+ }
	124	+ echo number_format($sum, 2, '.', ' ');
	125	+ if (App::$cur->money) {
	126	+ $currency = Money\Currency::get($currency_id);
	127	+ if ($currency) {
	128	+ echo ' ' . $currency->acronym();
	129	+ } else {
	130	+ echo ' руб.';
	131	+ }
	132	+ } else {
	133	+ echo ' руб.';
	134	+ }
	135	+ echo '<br />';
	136	+ }
	137	+ ?></td>
138	138	<td></td>
139	139	</tr>
140	140	<?php
141		- $deliveryCurrency = $cart->delivery ? $cart->delivery->currency_id : 0;
142		- $deliveryPrice = $cart->delivery->price;
143		- if (!isset($sums[$deliveryCurrency])) {
144		- $sums[$deliveryCurrency] = $deliveryPrice;
145		- } else {
146		- $sums[$deliveryCurrency] += $deliveryPrice;
147		- }
148		- $delCurrency = false;
149		- if (App::$cur->money) {
150		- $delCurrency = Money\Currency::get($deliveryPrice);
151		- }
152		- ?>
	141	+ $deliveryCurrency = $cart->delivery ? $cart->delivery->currency_id : 0;
	142	+ $deliveryPrice = $cart->delivery->price;
	143	+ if (!isset($sums[$deliveryCurrency])) {
	144	+ $sums[$deliveryCurrency] = $deliveryPrice;
	145	+ } else {
	146	+ $sums[$deliveryCurrency] += $deliveryPrice;
	147	+ }
	148	+ $delCurrency = false;
	149	+ if (App::$cur->money) {
	150	+ $delCurrency = Money\Currency::get($deliveryPrice);
	151	+ }
	152	+ ?>
153	153	<tr>
154	154	<td colspan="2"></td>
155	155	<td class="text-right"><b><?= $cart->delivery ? $cart->delivery->name : 'Доставка'; ?></b></td>
		@@ -160,24 +160,24 @@ discard block
		block discarded – undo
160	160	<td colspan="2"></td>
161	161	<td class="text-right"><b>Итого</b></td>
162	162	<td class="text-right"><?php
163		- foreach ($sums as $currency_id => $sum) {
164		- if (!$sum) {
165		- continue;
166		- }
167		- echo number_format($sum, 2, '.', ' ');
168		- if (App::$cur->money) {
169		- $currency = Money\Currency::get($currency_id);
170		- if ($currency) {
171		- echo ' ' . $currency->acronym();
172		- } else {
173		- echo ' руб.';
174		- }
175		- } else {
176		- echo ' руб.';
177		- }
178		- echo '<br />';
179		- }
180		- ?></td>
	163	+ foreach ($sums as $currency_id => $sum) {
	164	+ if (!$sum) {
	165	+ continue;
	166	+ }
	167	+ echo number_format($sum, 2, '.', ' ');
	168	+ if (App::$cur->money) {
	169	+ $currency = Money\Currency::get($currency_id);
	170	+ if ($currency) {
	171	+ echo ' ' . $currency->acronym();
	172	+ } else {
	173	+ echo ' руб.';
	174	+ }
	175	+ } else {
	176	+ echo ' руб.';
	177	+ }
	178	+ echo '<br />';
	179	+ }
	180	+ ?></td>
181	181	<td></td>
182	182	</tr>
183	183	</tfoot>

injitools / cms-Inji

Push — master ( e7c2e1...989877 )

Status

Category

Indentation +87 added lines, -87 removed lines patch added patch discarded remove patch