intraface / intraface.dk

src/Intraface/modules/cms/ElementGateway.php

Doc Comments +9 added lines

@@ -11,6 +11,9 @@ 
         $this->kernel = $kernel;
     }
 
+    /**
+     * @param string $type
+     */
     function findBySectionAndType($section, $type)
     {
         $class = $this->class_prefix . ucfirst($type);
@@ -37,6 +40,9 @@ 
         return new $class(CMS_Section::factory($this->kernel, 'id', $this->db->f('section_id')), $this->db->f('id'));
     }
 
+    /**
+     * @param string $id
+     */
     function findByKernelAndId($kernel, $id)
     {
         $cms_module = $kernel->getModule('cms');
@@ -53,6 +59,9 @@ 
         return new $class(CMS_Section::factory($kernel, 'id', $this->db->f('section_id')), $this->db->f('id'));
     }
 
+    /**
+     * @param string $id
+     */
     function findBySectionAndId($section, $id)
     {
         // FIXME - jeg tror den her kan skabe en del

Spacing +7 added lines, -7 removed lines

@@ -13,7 +13,7 @@ 
 
     function findBySectionAndType($section, $type)
     {
-        $class = $this->class_prefix . ucfirst($type);
+        $class = $this->class_prefix.ucfirst($type);
 
         if (!class_exists($class)) {
             throw new Exception('Class does not exist');
@@ -26,11 +26,11 @@ 
         $cms_module = $this->kernel->getModule('cms');
         $element_types = $cms_module->getSetting('element_types');
 
-        $this->db->query("SELECT id, section_id, type_key FROM cms_element WHERE id = " . $id . " AND intranet_id = " . $this->kernel->intranet->get('id'));
+        $this->db->query("SELECT id, section_id, type_key FROM cms_element WHERE id = ".$id." AND intranet_id = ".$this->kernel->intranet->get('id'));
         if (!$this->db->nextRecord()) {
             return false;
         }
-        $class = $this->class_prefix . ucfirst($element_types[$this->db->f('type_key')]);
+        $class = $this->class_prefix.ucfirst($element_types[$this->db->f('type_key')]);
         if (!class_exists($class)) {
             return false;
         }
@@ -42,11 +42,11 @@ 
         $cms_module = $kernel->getModule('cms');
         $element_types = $cms_module->getSetting('element_types');
 
-        $this->db->query("SELECT id, section_id, type_key FROM cms_element WHERE id = " . $id . " AND intranet_id = " . $kernel->intranet->get('id'));
+        $this->db->query("SELECT id, section_id, type_key FROM cms_element WHERE id = ".$id." AND intranet_id = ".$kernel->intranet->get('id'));
         if (!$this->db->nextRecord()) {
             return false;
         }
-        $class = $this->class_prefix . ucfirst($element_types[$this->db->f('type_key')]);
+        $class = $this->class_prefix.ucfirst($element_types[$this->db->f('type_key')]);
         if (!class_exists($class)) {
             return false;
         }
@@ -61,12 +61,12 @@ 
         $cms_module = $section->kernel->getModule('cms');
         $element_types = $cms_module->getSetting('element_types');
 
-        $this->db->query("SELECT id, section_id, type_key FROM cms_element WHERE id = " . $id . " AND intranet_id = " . $section->kernel->intranet->get('id'));
+        $this->db->query("SELECT id, section_id, type_key FROM cms_element WHERE id = ".$id." AND intranet_id = ".$section->kernel->intranet->get('id'));
         if (!$this->db->nextRecord()) {
             return false;
         }
 
-        $class = $this->class_prefix . ucfirst($element_types[$this->db->f('type_key')]);
+        $class = $this->class_prefix.ucfirst($element_types[$this->db->f('type_key')]);
         if (!class_exists($class)) {
             return false;
         }

src/Intraface/modules/cms/HTML_Parser.php

Doc Comments +3 added lines

@@ -21,6 +21,9 @@
         $this->translation = $translation;

     }

 

+    /**
+     * @param string $phrase
+     */
     function t($phrase)

     {

         return utf8_encode($this->translation->get($phrase));

Spacing +4 added lines, -4 removed lines

@@ -58,7 +58,7 @@ 
                 $element['extra_style'] = ' style="'.$element['extra_style'].'"';
             }
 
-            $function =  'parse'.$element['type'] . 'Element';
+            $function = 'parse'.$element['type'].'Element';
             $output = $this->$function($element);
 
             $display .= '<div id="element-'.$element['id'].'"'.$element['extra_class'].$element['extra_style'].'>';
@@ -76,9 +76,9 @@ 
             } else {
                 $display .= '		<li><a href="'.url(null).'?action=move&amp;element_id='.$element['id'].'&amp;id='.$element['section_id'].'">'.$this->t('move').'</a></li>';
             }
-            $display .= '		<li><a class="confirm" href="' . url(null) . '?delete='.$element['id'].'">'.$this->t('delete').'</a></li>';
-            $display .=     '	</ul>';
-            $display .= '<div>' .           $output . '</div>';
+            $display .= '		<li><a class="confirm" href="'.url(null).'?delete='.$element['id'].'">'.$this->t('delete').'</a></li>';
+            $display .= '	</ul>';
+            $display .= '<div>'.$output.'</div>';
             $display .= '</div>';
         }
         return $display;

src/Intraface/modules/cms/Navigation.php

Doc Comments +3 added lines

@@ -10,6 +10,9 @@
     private $cmspage;

     public $value;

 

+    /**
+     * @param CMS_Page $cmspage
+     */
     function __construct($cmspage)

     {

         $this->cmspage = $cmspage;

src/Intraface/modules/cms/PageGateway.php

Doc Comments +5 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
     /**
      * Returns the possible page types
      *
-     * @return array possible page types
+     * @return string[] possible page types
      */
     public function getTypes()
     {
@@ -59,6 +59,9 @@ 
         return $object;
     }
 
+    /**
+     * @param string $identifier
+     */
     function findBySiteIdAndIdentifier($site_id, $identifier)
     {
         $identifier = strip_tags($identifier);
@@ -282,7 +285,7 @@ 
     /**
      * Returns the possible page types but with a binary index
      *
-     * @return array possible page types with binary index
+     * @return string[] possible page types with binary index
      */
     static public function getTypesWithBinaryIndex()
     {

Spacing +7 added lines, -7 removed lines

@@ -40,7 +40,7 @@ 
         if ($this->dbquery) {
             return $this->dbquery;
         }
-        return ($this->dbquery = new Intraface_DBQuery($this->kernel, 'cms_page', 'cms_page.intranet_id = '.$this->kernel->intranet->get('id').' AND cms_page.active = 1 AND site_id = ' . $this->cmssite->get('id')));
+        return ($this->dbquery = new Intraface_DBQuery($this->kernel, 'cms_page', 'cms_page.intranet_id = '.$this->kernel->intranet->get('id').' AND cms_page.active = 1 AND site_id = '.$this->cmssite->get('id')));
     }
 
     function setDBQuery($dbquery)
@@ -50,7 +50,7 @@ 
 
     function findById($id)
     {
-        $this->db->query("SELECT id, site_id FROM cms_page WHERE id = " . (int)$id . " AND intranet_id = " . $this->kernel->intranet->get('id'));
+        $this->db->query("SELECT id, site_id FROM cms_page WHERE id = ".(int)$id." AND intranet_id = ".$this->kernel->intranet->get('id'));
         if (!$this->db->nextRecord()) {
             return false;
         }
@@ -64,13 +64,13 @@ 
         $identifier = strip_tags($identifier);
 
         if (!empty($identifier)) {
-            $this->db->query("SELECT site_id, id FROM cms_page WHERE identifier = '" . $identifier . "' AND intranet_id = " . $this->kernel->intranet->get('id') . " AND active = 1 AND site_id = " . $site_id);
+            $this->db->query("SELECT site_id, id FROM cms_page WHERE identifier = '".$identifier."' AND intranet_id = ".$this->kernel->intranet->get('id')." AND active = 1 AND site_id = ".$site_id);
         } else {
             // @todo choose the default page - vi skal lige have noget med publish og expire date her ogs�
-            $this->db->query("SELECT site_id, id FROM cms_page WHERE intranet_id = " . $this->kernel->intranet->get('id') . " AND active = 1 AND status_key = 1 AND site_id = " . $site_id . " ORDER BY position ASC LIMIT 1");
+            $this->db->query("SELECT site_id, id FROM cms_page WHERE intranet_id = ".$this->kernel->intranet->get('id')." AND active = 1 AND status_key = 1 AND site_id = ".$site_id." ORDER BY position ASC LIMIT 1");
         }
         if (!$this->db->nextRecord()) {
-            $this->db->query("SELECT site_id, id FROM cms_page WHERE id = " . (int)$identifier . " AND intranet_id = " . $this->kernel->intranet->get('id') . " AND active = 1 AND site_id = " . $site_id);
+            $this->db->query("SELECT site_id, id FROM cms_page WHERE id = ".(int)$identifier." AND intranet_id = ".$this->kernel->intranet->get('id')." AND active = 1 AND site_id = ".$site_id);
             if (!$this->db->nextRecord()) {
                 return false;
             }
@@ -211,8 +211,8 @@ 
                 // til google sitemaps
                 // sp�rgsm�let er om vi ikke skal starte et objekt op for hver pages
 
-                $pages[$i]['url'] = $this->cmssite->get('url') . $pages[$i]['identifier'] . '/';
-                $pages[$i]['url_self'] = $pages[$i]['identifier'] . '/';
+                $pages[$i]['url'] = $this->cmssite->get('url').$pages[$i]['identifier'].'/';
+                $pages[$i]['url_self'] = $pages[$i]['identifier'].'/';
                 $pages[$i]['changefreq'] = 'weekly';
                 $pages[$i]['priority'] = 0.5;
 

src/Intraface/modules/cms/Stylesheet.php

Doc Comments +3 added lines

@@ -7,6 +7,9 @@
     private $cmssite;

     public $error;

 

+    /**
+     * @param CMS_Site $cmssite
+     */
     function __construct($cmssite)

     {

         if (!is_object($cmssite) OR strtolower(get_class($cmssite)) != 'cms_site') {

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     function __construct($cmssite)
     {
         if (!is_object($cmssite) or strtolower(get_class($cmssite)) != 'cms_site') {
-            throw new Exception('CMS_Stylesheet::__construct needs CMS_Site - got ' . get_class($cmssite));
+            throw new Exception('CMS_Stylesheet::__construct needs CMS_Site - got '.get_class($cmssite));
         }
         $this->cmssite = $cmssite;
         $this->error = new Intraface_Error;

src/Intraface/modules/cms/TemplateSection.php

Doc Comments +6 added lines

@@ -62,11 +62,17 @@
         return new CMS_Parameter($this);

     }

 

+    /**
+     * @param string $key
+     */
     function addParameter($key, $value)

     {

         return $this->parameter->save($key, $value);

     }

 

+    /**
+     * @param string $type
+     */
     function factory($object, $type, $value)

     {

         $class_prefix = 'Intraface_modules_cms_templatesection_';

Spacing +15 added lines, -15 removed lines

@@ -31,11 +31,11 @@ 
             throw new Exception('TemplateSection::__construct skal bruge CMS_Template');
         }
         $this->error    = new Intraface_Error;
-        $this->id       = (int) $id;
+        $this->id       = (int)$id;
         $this->template = $template;
         $this->kernel   = $template->kernel;
 
-        $this->value['identify_as'] = 'cms_template_section';  // bruges af parameter
+        $this->value['identify_as'] = 'cms_template_section'; // bruges af parameter
 
         $this->parameter = $this->createParameterObject();
 
@@ -75,7 +75,7 @@ 
                 // validering p� value
                 // kun v�re gyldige elementtyper
                 // object skal vre cmspage
-                $class = $class_prefix . ucfirst($value);
+                $class = $class_prefix.ucfirst($value);
                 return new $class($object);
                 break;
             case 'id':
@@ -84,12 +84,12 @@ 
                 $section_types = $cms_module->getSetting('section_types');
 
                 $db = new DB_Sql;
-                $db->query("SELECT id, type_key, template_id FROM cms_template_section WHERE id = " . $value . " AND intranet_id = " . $object->intranet->get('id'));
+                $db->query("SELECT id, type_key, template_id FROM cms_template_section WHERE id = ".$value." AND intranet_id = ".$object->intranet->get('id'));
                 if (!$db->nextRecord()) {
                     return false;
                 }
 
-                $class = $class_prefix . ucfirst($section_types[$db->f('type_key')]);
+                $class = $class_prefix.ucfirst($section_types[$db->f('type_key')]);
                 return new $class(CMS_Template::factory($object, 'id', $db->f('template_id')), $db->f('id'));
 
                 break;
@@ -99,12 +99,12 @@ 
                 $section_types = $cms_module->getSetting('section_types');
 
                 $db = new DB_Sql;
-                $db->query("SELECT id, type_key FROM cms_template_section WHERE id = " . $value . " AND intranet_id = " . $object->kernel->intranet->get('id'));
+                $db->query("SELECT id, type_key FROM cms_template_section WHERE id = ".$value." AND intranet_id = ".$object->kernel->intranet->get('id'));
                 if (!$db->nextRecord()) {
                     return false;
                 }
 
-                $class = $class_prefix . ucfirst($section_types[$db->f('type_key')]);
+                $class = $class_prefix.ucfirst($section_types[$db->f('type_key')]);
                 return new $class($object, $db->f('id'));
 
                 break;
@@ -117,7 +117,7 @@ 
     function load()
     {
         $db = new DB_Sql;
-        $db->query("SELECT id, name, identifier, type_key, locked FROM cms_template_section WHERE cms_template_section.intranet_id = ".$this->template->cmssite->kernel->intranet->get('id')." AND cms_template_section.id = " . $this->id);
+        $db->query("SELECT id, name, identifier, type_key, locked FROM cms_template_section WHERE cms_template_section.intranet_id = ".$this->template->cmssite->kernel->intranet->get('id')." AND cms_template_section.id = ".$this->id);
         if (!$db->nextRecord()) {
             return 0;
         }
@@ -145,7 +145,7 @@ 
             $this->error->set('error in identifier - cannot be empty');
         }
 
-        if (!Validate::string($var['identifier'], array('format' => VALIDATE_ALPHA . VALIDATE_NUM . '-_'))) {
+        if (!Validate::string($var['identifier'], array('format' => VALIDATE_ALPHA.VALIDATE_NUM.'-_'))) {
             $this->error->set('error in identfier - allowed characters are a-z and 1-9');
         }
         if ($this->isIdentifierUnique($var['identifier'])) {
@@ -176,12 +176,12 @@ 
             $sql_end = ", date_created = NOW()";
         } else {
             $sql_type = "UPDATE ";
-            $sql_end = " WHERE id = " . $this->id;
+            $sql_end = " WHERE id = ".$this->id;
         }
-        $sql = $sql_type . " cms_template_section SET
+        $sql = $sql_type." cms_template_section SET
                 name = '".safeToDb($var['name'])."',
                 identifier = '".safeToDb($var['identifier'])."',
-                site_id = " . $this->template->cmssite->get('id') . ",
+                site_id = " . $this->template->cmssite->get('id').",
                 intranet_id = ".$this->template->kernel->intranet->get('id').",
                 template_id = ".$this->template->get('id').",
                 type_key = ".$this->value['type_key'].",
@@ -211,14 +211,14 @@ 
     function delete()
     {
         $db = new DB_Sql;
-        $db->query("UPDATE cms_template_section SET active = 0 WHERE id = " . $this->id);
+        $db->query("UPDATE cms_template_section SET active = 0 WHERE id = ".$this->id);
         return true;
     }
 
     function getList()
     {
         $db = new DB_Sql;
-        $db->query("SELECT id, name, identifier, type_key FROM cms_template_section WHERE template_id = " . $this->template->get('id') . " AND intranet_id = " . $this->kernel->intranet->get('id') . " AND active = 1 ORDER BY position ASC");
+        $db->query("SELECT id, name, identifier, type_key FROM cms_template_section WHERE template_id = ".$this->template->get('id')." AND intranet_id = ".$this->kernel->intranet->get('id')." AND active = 1 ORDER BY position ASC");
 
         $i = 0;
         $sections = array();
@@ -237,7 +237,7 @@ 
     function isIdentifierUnique($identifier)
     {
         $db = new DB_Sql;
-        $db->query("SELECT count(*) AS antal FROM cms_template_section WHERE identifier = '".$identifier."' AND intranet_id = " . $this->kernel->intranet->get('id') . " AND template_id = " . $this->template->get('id') . " AND active = 1 AND id != " . $this->id);
+        $db->query("SELECT count(*) AS antal FROM cms_template_section WHERE identifier = '".$identifier."' AND intranet_id = ".$this->kernel->intranet->get('id')." AND template_id = ".$this->template->get('id')." AND active = 1 AND id != ".$this->id);
         if (!$db->nextRecord()) {
             return 0;
         }

src/Intraface/modules/cms/templatesection/Mixed.php

Doc Comments +3 added lines

@@ -6,6 +6,9 @@
  */

 class Intraface_modules_cms_templatesection_Mixed extends CMS_TemplateSection

 {

+    /**
+     * @param CMS_Template $cmspage
+     */
     function __construct($cmspage, $id = 0)

     {

         $this->value['type'] = 'mixed';

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

Doc Comments +3 added lines

@@ -87,6 +87,9 @@
         return 'Create';
     }
 
+    /**
+     * @return string
+     */
     function getType()
     {
         return $this->context->context->getType();

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
         if ($this->query('contact_id') == '') {
             return new k_SeeOther($this->url('contact'));
         }
-        $smarty = $this->template->create(dirname(__FILE__) . '/templates/edit');
+        $smarty = $this->template->create(dirname(__FILE__).'/templates/edit');
         return $smarty->render($this);
     }
 
@@ -55,7 +55,7 @@ 
         }
 
         if ($debtor->update($_POST)) {
-            return new k_SeeOther($this->url('../' . $debtor->get('id')));
+            return new k_SeeOther($this->url('../'.$debtor->get('id')));
         }
 
         return $this->render();
@@ -67,7 +67,7 @@ 
             return $this->body();
         }
 
-        $due_time = time() + $this->getContact()->get('paymentcondition') * 24 * 60 * 60;
+        $due_time = time() + $this->getContact()->get('paymentcondition')*24*60*60;
         $due_date = date('d-m-Y', $due_time);
 
         return array(

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

Doc Comments +3 added lines

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

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
         $depreciation = $this->getDepreciation();
         if ($id = $depreciation->update($_POST)) {
             if ($this->getKernel()->user->hasModuleAccess('accounting')) {
-                return new k_SeeOther($this->url($id . '/state'));
+                return new k_SeeOther($this->url($id.'/state'));
             } else {
                 return new k_SeeOther($this->url('../'));
             }
@@ -64,7 +64,7 @@ 
     {
         $invoice_module = $this->getKernel()->useModule('invoice');
         $depreciation = $this->getDepreciation();
-        $smarty = $this->template->create(dirname(__FILE__) . '/templates/depreciation');
+        $smarty = $this->template->create(dirname(__FILE__).'/templates/depreciation');
         return $smarty->render($this);
     }
 }