intraface / intraface.dk

src/Intraface/modules/procurement/Procurement.php

Indentation +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
 
     function loadItem($id = 0)
     {
-         $this->item = new ProcurementItem($this, (int)$id);
+            $this->item = new ProcurementItem($this, (int)$id);
     }
 
     function update($input)

Doc Comments +10 added lines, -2 removed lines

@@ -280,6 +280,9 @@ 
         return true;
     }
 
+    /**
+     * @param string $dk_paid_date
+     */
     function setPaid($dk_paid_date)
     {
         if ($this->get('id') == 0) {
@@ -369,6 +372,11 @@ 
      *
      * @param object year year object
      * @param integer voucher_number
+     * @param Year $year
+     * @param integer $voucher_number
+     * @param string $voucher_date
+     * @param integer $credit_account_id
+     * @param Stub_Translation $translation
      *
      * @return boolean
      */
@@ -685,7 +693,7 @@ 
      *
      * @todo: duplicate in Procurement class
      *
-     * @return array status types
+     * @return string[] status types
      */
     private function getStatusTypes()
     {
@@ -700,7 +708,7 @@ 
      * returns the possible regions where procurement is bought
      * @todo: duplicate in Procurement class
      *
-     * @return array possible regions
+     * @return string[] possible regions
      */
     public function getRegionTypes()
     {

Spacing +17 added lines, -17 removed lines

@@ -414,7 +414,7 @@ 
         }
 
 
-        $text = $translation->get('procurement').'# ' . $this->get('number') . ': ' . $this->get('description');
+        $text = $translation->get('procurement').'# '.$this->get('number').': '.$this->get('description');
         require_once 'Intraface/modules/accounting/Voucher.php';
         $voucher = Voucher::factory($year, $voucher_number);
         $voucher->save(array(
@@ -433,7 +433,7 @@ 
             $credit_total += $amount;
 
             if (!empty($line['text'])) {
-                $line_text = $text. ' - ' . $line['text'];
+                $line_text = $text.' - '.$line['text'];
             } else {
                 $line_text = $text;
             }
@@ -487,7 +487,7 @@ 
             $voucher_date->convert2db();
         }
 
-        $db->query("UPDATE procurement SET date_stated = '".$voucher_date->get()."', voucher_id = ".intval($voucher_number)." WHERE id = " . $this->id . " AND intranet_id = " . $this->kernel->intranet->get('id'));
+        $db->query("UPDATE procurement SET date_stated = '".$voucher_date->get()."', voucher_id = ".intval($voucher_number)." WHERE id = ".$this->id." AND intranet_id = ".$this->kernel->intranet->get('id'));
         return 1;
     }
 
@@ -570,15 +570,15 @@ 
         $total = 0;
         $vat = 0;
         foreach ($debet_accounts as $key => $debet_account) {
-            if ($validator->isNumeric($debet_account['amount'], 'Ugyldig beløb i linje ' . ($key + 1) . ' "' . $debet_account['text'] . '"', 'greater_than_zero')) {
+            if ($validator->isNumeric($debet_account['amount'], 'Ugyldig beløb i linje '.($key + 1).' "'.$debet_account['text'].'"', 'greater_than_zero')) {
                 $amount = new Intraface_Amount($debet_account['amount']);
                 $amount->convert2db();
                 $total += $amount->get();
 
-                $validator->isString($debet_account['text'], 'Ugyldig tekst i linje ' . ($key + 1) . ' "' . $debet_account['text'] . '"', '', 'allow_empty');
+                $validator->isString($debet_account['text'], 'Ugyldig tekst i linje '.($key + 1).' "'.$debet_account['text'].'"', '', 'allow_empty');
 
                 if (empty($debet_account['state_account_id'])) {
-                    $this->error->set('Linje ' . ($key + 1) . ' "' . $debet_account['text'] . '" ved ikke hvor den skal bogføres');
+                    $this->error->set('Linje '.($key + 1).' "'.$debet_account['text'].'" ved ikke hvor den skal bogføres');
                 } else {
                     require_once 'Intraface/modules/accounting/Account.php';
                     $account = Account::factory($year, $debet_account['state_account_id']);
@@ -586,9 +586,9 @@ 
                     // @todo check this. I changed it to make sure that we are able to state varekøb til videresalg
                     // || $account->get('type') != 'operating'
                     if ($account->get('id') == 0) {
-                        $this->error->set('Ugyldig konto for bogføring af linje ' . ($key + 1) . ' "' . $debet_account['text'] . '"');
+                        $this->error->set('Ugyldig konto for bogføring af linje '.($key + 1).' "'.$debet_account['text'].'"');
                     } elseif ($account->get('vat') == 'in') {
-                        $vat += $amount->get() / 100 * $account->get('vat_percent');
+                        $vat += $amount->get()/100*$account->get('vat_percent');
                     }
                 }
             }
@@ -605,8 +605,8 @@ 
             }
 
             if (round($vat, 2) != $this->get('vat')) {
-                $expected = number_format($this->get('vat') * 4, 2, ',', '.');
-                $this->error->set('Momsen af de beløb du bogfører på konti med moms stemmer ikke overens med momsen på det samlede indkøb (vi forventede '.number_format($vat, 2, ',', '.').' i moms og det samlede beløb burde have været ' . $expected . '). Har du fået alle varer med? Har du husket at skrive beløbet uden moms for varerne?');
+                $expected = number_format($this->get('vat')*4, 2, ',', '.');
+                $this->error->set('Momsen af de beløb du bogfører på konti med moms stemmer ikke overens med momsen på det samlede indkøb (vi forventede '.number_format($vat, 2, ',', '.').' i moms og det samlede beløb burde have været '.$expected.'). Har du fået alle varer med? Har du husket at skrive beløbet uden moms for varerne?');
             }
         }
 
@@ -635,15 +635,15 @@ 
         $total = 0;
         $vat = 0;
         foreach ($debet_accounts as $key => $debet_account) {
-            if ($validator->isNumeric($debet_account['amount'], 'Ugyldig beløb i linje ' . ($key + 1) . ' "' . $debet_account['text'] . '"', 'greater_than_zero')) {
+            if ($validator->isNumeric($debet_account['amount'], 'Ugyldig beløb i linje '.($key + 1).' "'.$debet_account['text'].'"', 'greater_than_zero')) {
                 $amount = new Intraface_Amount($debet_account['amount']);
                 $amount->convert2db();
                 $total += $amount->get();
 
-                $validator->isString($debet_account['text'], 'Ugyldig tekst i linje ' . ($key + 1) . ' "' . $debet_account['text'] . '"', '', 'allow_empty');
+                $validator->isString($debet_account['text'], 'Ugyldig tekst i linje '.($key + 1).' "'.$debet_account['text'].'"', '', 'allow_empty');
 
                 if (empty($debet_account['state_account_id'])) {
-                    $this->error->set('Linje ' . ($key + 1) . ' "' . $debet_account['text'] . '" ved ikke hvor den skal bogføres');
+                    $this->error->set('Linje '.($key + 1).' "'.$debet_account['text'].'" ved ikke hvor den skal bogføres');
                 } else {
                     require_once 'Intraface/modules/accounting/Account.php';
                     $account = Account::factory($year, $debet_account['state_account_id']);
@@ -651,9 +651,9 @@ 
                     // @todo check this. I changed it to make sure that we are able to state varekøb til videresalg
                     // || $account->get('type') != 'operating'
                     if ($account->get('id') == 0) {
-                        $this->error->set('Ugyldig konto for bogføring af linje ' . ($key + 1) . ' "' . $debet_account['text'] . '"');
+                        $this->error->set('Ugyldig konto for bogføring af linje '.($key + 1).' "'.$debet_account['text'].'"');
                     } elseif ($account->get('vat') == 'in') {
-                        $vat += $amount->get() / 100 * $account->get('vat_percent');
+                        $vat += $amount->get()/100*$account->get('vat_percent');
                     }
                 }
             }
@@ -670,8 +670,8 @@ 
 
         if (round($vat, 2) != $this->get('vat')) {
             $expected_vat = number_format($vat, 2, ',', '.');
-            $expected_value = number_format($this->get('vat') * 4, 2, ',', '.');
-            $this->error->set('Momsen af de beløb du bogfører på konti med moms stemmer ikke overens med momsen på det samlede indkøb (vi forventede ' . $expected_vat . ' i moms og det samlede beløb burde have været ' . $expected_value . '). Har du fået alle varer med? Har du husket at skrive beløbet uden moms for varerne?');
+            $expected_value = number_format($this->get('vat')*4, 2, ',', '.');
+            $this->error->set('Momsen af de beløb du bogfører på konti med moms stemmer ikke overens med momsen på det samlede indkøb (vi forventede '.$expected_vat.' i moms og det samlede beløb burde have været '.$expected_value.'). Har du fået alle varer med? Har du husket at skrive beløbet uden moms for varerne?');
         }
         
         if ($this->error->isError()) {

src/Intraface/modules/procurement/Controller/Index.php

Spacing +2 added lines, -2 removed lines

@@ -67,7 +67,7 @@ 
             'procurements' => $procurements
         );
 
-        $smarty = $this->template->create(dirname(__FILE__) . '/templates/index');
+        $smarty = $this->template->create(dirname(__FILE__).'/templates/index');
         return $smarty->render($this, $data);
     }
 
@@ -82,7 +82,7 @@ 
             'gateway' => $this->getProcurementGateway()
         );
 
-        $smarty = $this->template->create(dirname(__FILE__) . '/templates/procurement-edit');
+        $smarty = $this->template->create(dirname(__FILE__).'/templates/procurement-edit');
         return $smarty->render($this, $data);
     }
 

src/Intraface/modules/procurement/Controller/Item.php

Spacing +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
 
         if (isset($_GET['change_product'])) {
             $redirect = Intraface_Redirect::factory($this->getKernel(), 'go');
-            $url = $redirect->setDestination(NET_SCHEME . NET_HOST . $this->url('selectproduct'), NET_SCHEME . NET_HOST . $this->url());
+            $url = $redirect->setDestination(NET_SCHEME.NET_HOST.$this->url('selectproduct'), NET_SCHEME.NET_HOST.$this->url());
             $redirect->askParameter('product_id');
             return new k_SeeOther($url);
         }
@@ -81,7 +81,7 @@ 
 
         $data = array('procurement' => $procurement, 'values' => $values);
 
-        $tpl = $this->template->create(dirname(__FILE__) . '/templates/item-edit');
+        $tpl = $this->template->create(dirname(__FILE__).'/templates/item-edit');
         return $tpl->render($this, $data);
 
     }

src/Intraface/modules/debtor/Visitor/Text.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
                 $currency_iso_code = 'DKK';
             }
 
-            $table->addRow(array(round($item["quantity"]), substr($item["name"], 0, 40), $currency_iso_code.' ' . $amount));
+            $table->addRow(array(round($item["quantity"]), substr($item["name"], 0, 40), $currency_iso_code.' '.$amount));
         }
         return $table->getTable();
     }

src/Intraface/modules/debtor/Visitor/Pdf.php

Spacing +19 added lines, -19 removed lines

@@ -74,7 +74,7 @@ 
 
         $this->doc->setY('-'.$this->doc->get("font_spacing"));
 
-        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
             $this->doc->nextPage(true);
         }
 
@@ -83,7 +83,7 @@ 
             foreach ($text AS $line) {
                 if ($line == "") {
                     $this->doc->setY('-'.$this->doc->get("font_spacing"));
-                    if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+                    if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
                         $this->doc->nextPage(true);
                     }
                 } else {
@@ -91,7 +91,7 @@ 
                         $this->doc->setY('-'.($this->doc->get("font_padding_top") + $this->doc->get("font_size")));
                         $line = $this->doc->addTextWrap($this->doc->get('margin_left'), $this->doc->get('y'), $this->doc->get('content_width'), $this->doc->get("font_size"), $line);
                         $this->doc->setY('-'.$this->doc->get("font_padding_bottom"));
-                        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+                        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
                             $this->doc->nextPage(true);
                         }
                     }
@@ -116,7 +116,7 @@ 
         $this->doc->addText($apointX["tekst"], $this->doc->get('y'), $this->doc->get("font_size"), "Tekst");
         $this->doc->addText($apointX["antal"] - $this->doc->getTextWidth($this->doc->get("font_size"), "Antal"), $this->doc->get('y'), $this->doc->get("font_size"), "Antal");
         $this->doc->addText($apointX["pris"] - $this->doc->getTextWidth($this->doc->get("font_size"), "Pris"), $this->doc->get('y'), $this->doc->get("font_size"), "Pris");
-        $this->doc->addText($apointX["beloeb"] - $this->doc->getTextWidth($this->doc->get("font_size"), "Beløb") -3, $this->doc->get('y'), $this->doc->get("font_size"), "Beløb");
+        $this->doc->addText($apointX["beloeb"] - $this->doc->getTextWidth($this->doc->get("font_size"), "Beløb") - 3, $this->doc->get('y'), $this->doc->get("font_size"), "Beløb");
 
         $this->doc->setY('-'.($this->doc->get("font_spacing") - $this->doc->get("font_size")));
 
@@ -138,7 +138,7 @@ 
 
         $bg_color = 0;
 
-        for ($i = 0, $max = count($items); $i <  $max; $i++) {
+        for ($i = 0, $max = count($items); $i < $max; $i++) {
             $vat = $items[$i]["vat"];
 
             if ($bg_color == 1) {
@@ -161,9 +161,9 @@ 
             }
             
             if ($debtor->getCurrency()) {
-                $amount =  $items[$i]["quantity"] * $items[$i]["price_currency"]->getAsIso(2);
+                $amount = $items[$i]["quantity"]*$items[$i]["price_currency"]->getAsIso(2);
             } else {
-                $amount =  $items[$i]["quantity"] * $items[$i]["price"]->getAsIso(2);
+                $amount = $items[$i]["quantity"]*$items[$i]["price"]->getAsIso(2);
             }
             $total += $amount;
 
@@ -188,7 +188,7 @@ 
 
                 $tekst = $this->doc->addTextWrap($apointX["tekst"], $this->doc->get('y'), $apointX["tekst_width_small"], $this->doc->get("font_size"), $tekst);
                 $this->doc->setY('-'.$this->doc->get("font_padding_bottom"));
-                if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+                if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
                     $this->doc->nextPage(true);
                 }
             }
@@ -211,7 +211,7 @@ 
                             $this->doc->setColor(0, 0, 0);
                         }
                         $this->doc->setY('-'.$this->doc->get("font_spacing"));
-                        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+                        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
                             $this->doc->nextPage(true);
                         }
                     } else {
@@ -227,7 +227,7 @@ 
                             $line = $this->doc->addTextWrap($apointX["tekst"], $this->doc->get('y') + 1, $apointX["tekst_width"], $this->doc->get("font_size"), $line); // Ups Ups, hvor kommer '+ 1' fra - jo ser du, ellers kappes det nederste af teksten!
                             $this->doc->setY('-'.$this->doc->get("font_padding_bottom"));
 
-                            if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+                            if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
                                 $this->doc->nextPage(true);
                             }
                         }
@@ -236,12 +236,12 @@ 
 
             }
 
-            if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+            if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
                 $this->doc->nextPage(true);
             }
 
             // If products with VAT and next post is without we add VAT.
-            if (($vat == 1 && isset($items[$i+1]["vat"]) && $items[$i+1]["vat"] == 0) || ($vat == 1 && $i+1 >= $max)) {
+            if (($vat == 1 && isset($items[$i + 1]["vat"]) && $items[$i + 1]["vat"] == 0) || ($vat == 1 && $i + 1 >= $max)) {
                 // If VAT on current product, but next has no VAT OR if VAT and last product
 
                 ($bg_color == 1) ? $bg_color = 0 : $bg_color = 1;
@@ -256,8 +256,8 @@ 
                 $this->doc->line($this->doc->get("margin_left"), $this->doc->get('y'), $this->doc->get('right_margin_position'), $this->doc->get('y'));
                 $this->doc->setY('-'.($this->doc->get("font_size") + $this->doc->get("font_padding_top")));
                 $this->doc->addText($apointX["tekst"], $this->doc->get('y'), $this->doc->get("font_size"), "<b>25% moms af ".number_format($total, 2, ",", ".")."</b>");
-                $this->doc->addText($apointX["beloeb"] - $this->doc->getTextWidth($this->doc->get("font_size"), "<b>".number_format($total * 0.25, 2, ",", ".")."</b>"), $this->doc->get('y'), $this->doc->get("font_size"), "<b>".number_format($total * 0.25, 2, ",", ".")."</b>");
-                $total = $total * 1.25;
+                $this->doc->addText($apointX["beloeb"] - $this->doc->getTextWidth($this->doc->get("font_size"), "<b>".number_format($total*0.25, 2, ",", ".")."</b>"), $this->doc->get('y'), $this->doc->get("font_size"), "<b>".number_format($total*0.25, 2, ",", ".")."</b>");
+                $total = $total*1.25;
                 $this->doc->setY('-'.$this->doc->get("font_padding_bottom"));
                 $this->doc->line($this->doc->get("margin_left"), $this->doc->get('y'), $this->doc->get('right_margin_position'), $this->doc->get('y'));
                 $this->doc->setLineStyle(1);
@@ -268,7 +268,7 @@ 
         }
 
 
-        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
             $this->doc->nextPage();
         }
 
@@ -293,7 +293,7 @@ 
             $total_text = "Total ".$currency_iso_code.":";
         }
 
-        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
             $this->doc->nextPage(true);
         }
 
@@ -330,7 +330,7 @@ 
 
             $this->doc->setY('-'.$this->doc->get("font_spacing"));
 
-            if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+            if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
                 $this->doc->nextPage(true);
             }
 
@@ -338,7 +338,7 @@ 
             foreach ($text AS $line) {
                 if ($line == "") {
                     $this->doc->setY('-'.$this->doc->get("font_spacing"));
-                    if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+                    if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
                         $this->doc->nextPage(true);
                     }
                 } else {
@@ -347,7 +347,7 @@ 
                         $line = $this->doc->addTextWrap($this->doc->get('margin_left'), $this->doc->get('y'), $this->doc->get('content_width'), $this->doc->get("font_size"), $line);
                         $this->doc->setY('-'.$this->doc->get("font_padding_bottom"));
 
-                        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") * 2) {
+                        if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing")*2) {
                             $this->doc->nextPage(true);
                         }
                     }

src/Intraface/modules/debtor/Pdf.php

Spacing +17 added lines, -17 removed lines

@@ -87,7 +87,7 @@ 
         $box_padding_top = 8; // space from top box to first line
         $box_padding_bottom = 9;
         $box_width = 275; // ($page_width - $margin_left - 10)/2;
-        $box_small_height = $this->doc->get("font_spacing") * 3 + $box_padding_top + $box_padding_bottom + 2;
+        $box_small_height = $this->doc->get("font_spacing")*3 + $box_padding_top + $box_padding_bottom + 2;
 
         // Write the receiver
         $this->doc->setY('-'.$this->doc->get("font_spacing")); // $pointY -= $box_padding_top;
@@ -151,7 +151,7 @@ 
                 if ($i == 2) $i = count($line);
             }
             $this->doc->addText($this->doc->get('x') + 10, $this->doc->get('y'), $this->doc->get("font_size"), $intranet["postcode"]." ".$intranet["city"]);
-            $this->doc->setY('-'.($this->doc->get("font_spacing") * 2)); // $pointY -= $this->doc->get("font_spacing") * 2;
+            $this->doc->setY('-'.($this->doc->get("font_spacing")*2)); // $pointY -= $this->doc->get("font_spacing") * 2;
 
             $this->doc->addText($this->doc->get('x') + 10, $this->doc->get('y'), $this->doc->get("font_size"), "CVR.:");
             $this->doc->addText($this->doc->get('x') + 10 + 60, $this->doc->get('y'), $this->doc->get("font_size"), $intranet["cvr"]);
@@ -188,7 +188,7 @@ 
         if (is_array($docinfo) && count($docinfo) > 0) {
             $this->doc->setY('-10'); // $pointY -= 10;
             $box_small_top = $this->doc->get('y');
-            $box_small_height = count($docinfo) * $this->doc->get("font_spacing") + $box_padding_top + $box_padding_bottom;
+            $box_small_height = count($docinfo)*$this->doc->get("font_spacing") + $box_padding_top + $box_padding_bottom;
             $this->doc->setY('-'.$box_padding_top); // $pointY -= $box_padding_top;
 
             for ($i = 0; $i < count($docinfo); $i++) {
@@ -280,7 +280,7 @@ 
             $payment_left = 230;
             $payment_right = $this->doc->get("right_margin_position") - $this->doc->get("margin_left") - $payment_left;
 
-            if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") + 4 + $payment_line * 3) {
+            if ($this->doc->get('y') < $this->doc->get("margin_bottom") + $this->doc->get("font_spacing") + 4 + $payment_line*3) {
                 $this->doc->nextPage(true);
             }
 
@@ -298,10 +298,10 @@ 
 
             if ($payment_method == 1) {
 
-                $this->doc->rectangle($this->doc->get('x'), $this->doc->get('y') - $payment_line * 2, $this->doc->get("right_margin_position") - $this->doc->get("margin_left"), $payment_line * 2);
-                $this->doc->line($this->doc->get('x') + $payment_left, $this->doc->get('y') - $payment_line * 2, $this->doc->get('x') + $payment_left, $this->doc->get('y'));
+                $this->doc->rectangle($this->doc->get('x'), $this->doc->get('y') - $payment_line*2, $this->doc->get("right_margin_position") - $this->doc->get("margin_left"), $payment_line*2);
+                $this->doc->line($this->doc->get('x') + $payment_left, $this->doc->get('y') - $payment_line*2, $this->doc->get('x') + $payment_left, $this->doc->get('y'));
                 $this->doc->line($this->doc->get('x'), $this->doc->get('y') - $payment_line, $this->doc->get("right_margin_position"), $this->doc->get('y') - $payment_line);
-                $this->doc->line($this->doc->get('x') + $payment_left / 2, $this->doc->get('y') - $payment_line * 2, $this->doc->get('x') + $payment_left / 2, $this->doc->get('y') - $payment_line);
+                $this->doc->line($this->doc->get('x') + $payment_left/2, $this->doc->get('y') - $payment_line*2, $this->doc->get('x') + $payment_left/2, $this->doc->get('y') - $payment_line);
 
                 $this->doc->setY('-7'); // $pointY -= 7;
                 $this->doc->addText($this->doc->get('x') + 4, $this->doc->get('y'), $this->doc->get("font_size") - 4, "Bank:");
@@ -325,9 +325,9 @@ 
                 $this->doc->setValue('y', $payment_start - $payment_line); // Sets exact position
                 $this->doc->setY('-7'); // $pointY -= 7;
 
-                $this->doc->addText($this->doc->get('x') + $payment_left / 2 + 4, $this->doc->get('y'), $this->doc->get("font_size") - 4, "Betalingsdato:");
+                $this->doc->addText($this->doc->get('x') + $payment_left/2 + 4, $this->doc->get('y'), $this->doc->get("font_size") - 4, "Betalingsdato:");
                 $this->doc->setY('-'.($payment_line - 12));
-                $this->doc->addText($this->doc->get('x') + $payment_left / 2 + 10, $this->doc->get('y'), $this->doc->get("font_size"), $parameter["due_date"]);
+                $this->doc->addText($this->doc->get('x') + $payment_left/2 + 10, $this->doc->get('y'), $this->doc->get("font_size"), $parameter["due_date"]);
 
                 $this->doc->setValue('y', $payment_start - $payment_line); // Sets exact position
                 $this->doc->setY('-7');
@@ -339,11 +339,11 @@ 
 
             } elseif ($payment_method == 2) {
 
-                $this->doc->rectangle($this->doc->get('x'), $this->doc->get('y') - $payment_line * 3, $this->doc->get("right_margin_position") - $this->doc->get("margin_left"), $payment_line * 3);
-                $this->doc->line($this->doc->get('x') + $payment_left, $this->doc->get('y') - $payment_line * 3, $this->doc->get('x') + $payment_left, $this->doc->get('y'));
+                $this->doc->rectangle($this->doc->get('x'), $this->doc->get('y') - $payment_line*3, $this->doc->get("right_margin_position") - $this->doc->get("margin_left"), $payment_line*3);
+                $this->doc->line($this->doc->get('x') + $payment_left, $this->doc->get('y') - $payment_line*3, $this->doc->get('x') + $payment_left, $this->doc->get('y'));
                 $this->doc->line($this->doc->get('x') + $payment_left, $this->doc->get('y') - $payment_line, $this->doc->get("right_margin_position"), $this->doc->get('y') - $payment_line);
-                $this->doc->line($this->doc->get('x') + $payment_left, $this->doc->get('y') - $payment_line * 2, $this->doc->get("right_margin_position"), $this->doc->get('y') - $payment_line * 2);
-                $this->doc->line($this->doc->get('x') + $payment_left + $payment_right / 2, $this->doc->get('y') - $payment_line * 2, $this->doc->get('x') + $payment_left + $payment_right / 2, $this->doc->get('y') - $payment_line);
+                $this->doc->line($this->doc->get('x') + $payment_left, $this->doc->get('y') - $payment_line*2, $this->doc->get("right_margin_position"), $this->doc->get('y') - $payment_line*2);
+                $this->doc->line($this->doc->get('x') + $payment_left + $payment_right/2, $this->doc->get('y') - $payment_line*2, $this->doc->get('x') + $payment_left + $payment_right/2, $this->doc->get('y') - $payment_line);
 
                 $this->doc->setY('-7');
                 $this->doc->addText($this->doc->get('x') + 4, $this->doc->get('y'), $this->doc->get("font_size") - 4, "Indbetaler:");
@@ -376,11 +376,11 @@ 
                 $this->doc->setValue('y', $payment_start - $payment_line); // Sets exact position
                 $this->doc->setY('-7');
 
-                $this->doc->addText($this->doc->get('x') + $payment_left + $payment_right / 2 + 4, $this->doc->get('y'), $this->doc->get("font_size") - 4, "Betalingsdato:");
+                $this->doc->addText($this->doc->get('x') + $payment_left + $payment_right/2 + 4, $this->doc->get('y'), $this->doc->get("font_size") - 4, "Betalingsdato:");
                 $this->doc->setY('-'.($payment_line - 12));
-                $this->doc->addText($this->doc->get('x') + $payment_left + $payment_right / 2 + 10, $this->doc->get('y'), $this->doc->get("font_size"), $parameter["due_date"]);
+                $this->doc->addText($this->doc->get('x') + $payment_left + $payment_right/2 + 10, $this->doc->get('y'), $this->doc->get("font_size"), $parameter["due_date"]);
 
-                $this->doc->setValue('y', $payment_start - $payment_line * 2); // Sets exact position
+                $this->doc->setValue('y', $payment_start - $payment_line*2); // Sets exact position
                 $this->doc->setY('-7');
 
 
@@ -392,7 +392,7 @@ 
                 $this->doc->addText($this->doc->get('x') + $payment_left + 10, $this->doc->get('y'), $this->doc->get('font_size'), $this_text);
             } elseif ($payment_method == 3) {
 
-                $this->doc->rectangle($this->doc->get('x'), $this->doc->get('y') - $payment_line * 2, $this->doc->get("right_margin_position") - $this->doc->get("margin_left"), $payment_line * 2);
+                $this->doc->rectangle($this->doc->get('x'), $this->doc->get('y') - $payment_line*2, $this->doc->get("right_margin_position") - $this->doc->get("margin_left"), $payment_line*2);
                 $this->doc->line($this->doc->get("margin_left"), $this->doc->get('y') - $payment_line, $this->doc->get("right_margin_position"), $this->doc->get('y') - $payment_line);
                 $this->doc->line($this->doc->get('x') + $payment_left, $this->doc->get('y'), $this->doc->get('x') + $payment_left, $this->doc->get('y') - $payment_line);
 

Doc Comments +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
     /**
      * Creates the document to write
      *
-     * @return PdfMaker object
+     * @return Intraface_Pdf object
      */
     protected function getDocument()
     {
@@ -48,7 +48,7 @@ 
      * @param string $type     Output to type (string or file)
      * @param string $filename Filename
      *
-     * @return void
+     * @return string|null
      */
     function output($type = 'string', $filename = 'debtor.pdf')
     {

src/Intraface/modules/debtor/Debtor/Item.php

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
      */
     public function getAmount()
     {
-        return new Ilib_Variable_Float($this->getQuantity()->getAsIso(2) * $this->getProductPriceObject()->getPriceIncludingVat($this->getProduct())->getAsIso(2), 'iso');
+        return new Ilib_Variable_Float($this->getQuantity()->getAsIso(2)*$this->getProductPriceObject()->getPriceIncludingVat($this->getProduct())->getAsIso(2), 'iso');
     }
     
     public function getQuantity()

src/Intraface/modules/debtor/Controller/Depreciation.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     {
         $invoice_module = $this->getKernel()->useModule('invoice');
         $depreciation = $this->getModel();
-        $smarty = $this->template->create(dirname(__FILE__) . '/templates/depreciation');
+        $smarty = $this->template->create(dirname(__FILE__).'/templates/depreciation');
         return $smarty->render($this);
     }
 

Doc Comments +3 added lines

@@ -23,6 +23,9 @@
         return $this->context->getKernel();
     }
 
+    /**
+     * @return Invoice
+     */
     function getModel()
     {
         return $this->context->getModel();

src/Intraface/modules/debtor/Controller/Settings.php

Spacing +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
             'kernel' => $this->context->getKernel(),
             'scan_in_contact' => $scan_in_contact);
 
-        $smarty = $this->template->create(dirname(__FILE__) . '/templates/settings');
+        $smarty = $this->template->create(dirname(__FILE__).'/templates/settings');
         return $smarty->render($this, $data);
     }
 
@@ -115,7 +115,7 @@ 
                 $contact_module = $this->context->getKernel()->useModule('contact');
 
                 $redirect = Intraface_Redirect::factory($this->context->getKernel(), 'go');
-                $url = $redirect->setDestination(NET_SCHEME . NET_HOST . $this->url('contact'), $debtor_module->getPath()."settings");
+                $url = $redirect->setDestination(NET_SCHEME.NET_HOST.$this->url('contact'), $debtor_module->getPath()."settings");
                 $redirect->askParameter('contact_id');
                 $redirect->setIdentifier('contact');