mambax7 / references

admin/about.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
  * @author       XOOPS Development Team
  */
 
-include_once __DIR__ . '/admin_header.php';
+include_once __DIR__.'/admin_header.php';
 
 xoops_cp_header();
 
@@ -25,4 +25,4 @@ 
 echo $aboutAdmin->addNavigation(basename(__FILE__));
 echo $aboutAdmin->renderAbout('xoopsfoundation@gmail.com', false);
 
-include_once __DIR__ . '/admin_footer.php';
+include_once __DIR__.'/admin_footer.php';

class/references_articles.php

Spacing +45 added lines, -45 removed lines

@@ -20,13 +20,13 @@ 
  */
 defined('XOOPS_ROOT_PATH') || exit('XOOPS root path not defined');
 
-require_once XOOPS_ROOT_PATH . '/kernel/object.php';
+require_once XOOPS_ROOT_PATH.'/kernel/object.php';
 if (!class_exists('references_XoopsPersistableObjectHandler')) {
-    require_once XOOPS_ROOT_PATH . '/modules/references/class/PersistableObjectHandler.php';
+    require_once XOOPS_ROOT_PATH.'/modules/references/class/PersistableObjectHandler.php';
 }
 
-define('REFERENCES_STATUS_ONLINE', 1);    // Articles en ligne
-define('REFERENCES_STATUS_OFFLINE', 0);    // Articles hors ligne
+define('REFERENCES_STATUS_ONLINE', 1); // Articles en ligne
+define('REFERENCES_STATUS_OFFLINE', 0); // Articles hors ligne
 
 class references_articles extends references_Object
 {
@@ -87,9 +87,9 @@ 
     public function getOnlinePicture()
     {
         if ($this->isArticleOnline()) {
-            return REFERENCES_IMAGES_URL . 'status_online.png';
+            return REFERENCES_IMAGES_URL.'status_online.png';
         } else {
-            return REFERENCES_IMAGES_URL . 'status_offline.png';
+            return REFERENCES_IMAGES_URL.'status_offline.png';
         }
     }
 
@@ -112,8 +112,8 @@ 
     public function pictureExists($indice)
     {
         $return    = false;
-        $fieldName = 'article_picture' . $indice;
-        if (xoops_trim($this->getVar($fieldName)) != '' && file_exists(references_utils::getModuleOption('images_path') . DIRECTORY_SEPARATOR . $this->getVar($fieldName))) {
+        $fieldName = 'article_picture'.$indice;
+        if (xoops_trim($this->getVar($fieldName)) != '' && file_exists(references_utils::getModuleOption('images_path').DIRECTORY_SEPARATOR.$this->getVar($fieldName))) {
             $return = true;
         }
 
@@ -128,9 +128,9 @@ 
      */
     public function deletePicture($indice)
     {
-        $fieldName = 'article_picture' . $indice;
+        $fieldName = 'article_picture'.$indice;
         if ($this->pictureExists($indice)) {
-            @unlink(references_utils::getModuleOption('images_path') . references_utils::getModuleOption('images_path') . $this->getVar($fieldName));
+            @unlink(references_utils::getModuleOption('images_path').references_utils::getModuleOption('images_path').$this->getVar($fieldName));
         }
         $this->setVar($fieldName, '');
     }
@@ -143,11 +143,11 @@ 
      */
     public function getPictureUrl($indice)
     {
-        $fieldName = 'article_picture' . $indice;
+        $fieldName = 'article_picture'.$indice;
         if (xoops_trim($this->getVar($fieldName)) != '' && $this->pictureExists($indice)) {
-            return references_utils::getModuleOption('images_url') . '/' . $this->getVar($fieldName);
+            return references_utils::getModuleOption('images_url').'/'.$this->getVar($fieldName);
         } else {
-            return REFERENCES_IMAGES_URL . 'blank.gif';
+            return REFERENCES_IMAGES_URL.'blank.gif';
         }
     }
 
@@ -159,9 +159,9 @@ 
      */
     public function getPicturePath($indice)
     {
-        $fieldName = 'article_picture' . $indice;
+        $fieldName = 'article_picture'.$indice;
         if (xoops_trim($this->getVar($fieldName)) != '') {
-            return references_utils::getModuleOption('images_path') . DIRECTORY_SEPARATOR . $this->getVar($fieldName);
+            return references_utils::getModuleOption('images_path').DIRECTORY_SEPARATOR.$this->getVar($fieldName);
         } else {
             return '';
         }
@@ -175,9 +175,9 @@ 
      */
     public function thumbExists($indice)
     {
-        $fieldName = 'article_picture' . $indice;
+        $fieldName = 'article_picture'.$indice;
         $return    = false;
-        if (xoops_trim($this->getVar($fieldName)) != '' && file_exists(references_utils::getModuleOption('images_path') . DIRECTORY_SEPARATOR . REFERENCES_THUMBS_PREFIX . $this->getVar($fieldName))) {
+        if (xoops_trim($this->getVar($fieldName)) != '' && file_exists(references_utils::getModuleOption('images_path').DIRECTORY_SEPARATOR.REFERENCES_THUMBS_PREFIX.$this->getVar($fieldName))) {
             $return = true;
         }
 
@@ -192,11 +192,11 @@ 
      */
     public function getThumbUrl($indice)
     {
-        $fieldName = 'article_picture' . $indice;
+        $fieldName = 'article_picture'.$indice;
         if (xoops_trim($this->getVar($fieldName)) != '') {
-            return references_utils::getModuleOption('images_url') . '/' . REFERENCES_THUMBS_PREFIX . $this->getVar($fieldName);
+            return references_utils::getModuleOption('images_url').'/'.REFERENCES_THUMBS_PREFIX.$this->getVar($fieldName);
         } else {
-            return REFERENCES_IMAGES_URL . 'blank.gif';
+            return REFERENCES_IMAGES_URL.'blank.gif';
         }
     }
 
@@ -208,9 +208,9 @@ 
      */
     public function getThumbPath($indice)
     {
-        $fieldName = 'article_picture' . $indice;
+        $fieldName = 'article_picture'.$indice;
         if (xoops_trim($this->getVar($fieldName)) != '') {
-            return references_utils::getModuleOption('images_path') . DIRECTORY_SEPARATOR . REFERENCES_THUMBS_PREFIX . $this->getVar($fieldName);
+            return references_utils::getModuleOption('images_path').DIRECTORY_SEPARATOR.REFERENCES_THUMBS_PREFIX.$this->getVar($fieldName);
         } else {
             return '';
         }
@@ -224,7 +224,7 @@ 
     public function attachmentExists()
     {
         $return = false;
-        if (xoops_trim($this->getVar('article_attached_file')) != '' && file_exists(references_utils::getModuleOption('attached_path') . DIRECTORY_SEPARATOR . $this->getVar('article_attached_file'))) {
+        if (xoops_trim($this->getVar('article_attached_file')) != '' && file_exists(references_utils::getModuleOption('attached_path').DIRECTORY_SEPARATOR.$this->getVar('article_attached_file'))) {
             $return = true;
         }
 
@@ -239,7 +239,7 @@ 
     public function getAttachmentUrl()
     {
         if (xoops_trim($this->getVar('article_attached_file')) != '' && $this->attachmentExists()) {
-            return references_utils::getModuleOption('attached_url') . '/' . $this->getVar('article_attached_file');
+            return references_utils::getModuleOption('attached_url').'/'.$this->getVar('article_attached_file');
         } else {
             return '';
         }
@@ -253,7 +253,7 @@ 
     public function getAttachmentPath()
     {
         if (xoops_trim($this->getVar('article_attached_file')) != '' && $this->attachmentExists()) {
-            return references_utils::getModuleOption('attached_path') . DIRECTORY_SEPARATOR . $this->getVar('article_attached_file');
+            return references_utils::getModuleOption('attached_path').DIRECTORY_SEPARATOR.$this->getVar('article_attached_file');
         } else {
             return '';
         }
@@ -265,7 +265,7 @@ 
     public function deleteAttachment()
     {
         if ($this->attachmentExists()) {
-            @unlink(references_utils::getModuleOption('attached_path') . DIRECTORY_SEPARATOR . $this->getVar('article_attached_file'));
+            @unlink(references_utils::getModuleOption('attached_path').DIRECTORY_SEPARATOR.$this->getVar('article_attached_file'));
         }
         $this->setVar('article_attached_file', '');
     }
@@ -278,9 +278,9 @@ 
      */
     public function deleteThumb($indice)
     {
-        $fieldName = 'article_picture' . $indice;
+        $fieldName = 'article_picture'.$indice;
         if ($this->thumbExists($indice)) {
-            @unlink(references_utils::getModuleOption('images_path') . DIRECTORY_SEPARATOR . REFERENCES_THUMBS_PREFIX . $this->getVar($fieldName));
+            @unlink(references_utils::getModuleOption('images_path').DIRECTORY_SEPARATOR.REFERENCES_THUMBS_PREFIX.$this->getVar($fieldName));
         }
     }
 
@@ -348,9 +348,9 @@ 
     public function getUrl($shortVersion = false)
     {
         if (!$shortVersion) {
-            return REFERENCES_URL . 'reference.php?article_id=' . $this->getVar('article_id');
+            return REFERENCES_URL.'reference.php?article_id='.$this->getVar('article_id');
         } else {
-            return 'reference.php?article_id=' . $this->getVar('article_id');
+            return 'reference.php?article_id='.$this->getVar('article_id');
         }
     }
 
@@ -373,11 +373,11 @@ 
             $ret['article_picture_exists'] = true;
             for ($i = 1; $i <= 10; ++$i) {
                 if ($this->pictureExists($i)) {
-                    $ret['article_picture_url' . $i]  = $this->getPictureUrl($i);
-                    $ret['article_picture_path' . $i] = $this->getPicturePath($i);
+                    $ret['article_picture_url'.$i]  = $this->getPictureUrl($i);
+                    $ret['article_picture_path'.$i] = $this->getPicturePath($i);
                     $ret['article_pictures_urls'][]   = $this->getPictureUrl($i);
                     $ret['article_pictures_paths'][]  = $this->getPicturePath($i);
-                    $fieldName                        = 'article_picture' . $i . '_text';
+                    $fieldName                        = 'article_picture'.$i.'_text';
                     if (xoops_trim($this->getVar($fieldName)) != '') {
                         $ret['article_pictures_texts'][] = references_utils::makeHrefTitle($this->getVar($fieldName));
                     } else {
@@ -402,8 +402,8 @@ 
             $ret['article_thumb_exists'] = true;
             for ($i = 1; $i <= 10; ++$i) {
                 if ($this->thumbExists($i)) {
-                    $ret['article_thumb_url' . $i]  = $this->getThumbUrl($i);
-                    $ret['article_thumb_path' . $i] = $this->getThumbPath($i);
+                    $ret['article_thumb_url'.$i]  = $this->getThumbUrl($i);
+                    $ret['article_thumb_path'.$i] = $this->getThumbPath($i);
                     $ret['article_thumbs_urls'][]   = $this->getThumbUrl($i);
                     $ret['article_thumbs_paths'][]  = $this->getThumbPath($i);
                 }
@@ -444,7 +444,7 @@ 
         $gperm_handler = xoops_getHandler('groupperm');
         $categories    = $gperm_handler->getItemIds($permissionsType, $groups, $currentModule->getVar('mid'));
         if (is_array($categories) && count($categories) > 0) {
-            $permissions[$permissionsType] = new Criteria('article_category_id', '(' . implode(',', $categories) . ')', 'IN');
+            $permissions[$permissionsType] = new Criteria('article_category_id', '('.implode(',', $categories).')', 'IN');
         } else {    // Ne peut rien voir
             $permissions[$permissionsType] = new Criteria('article_category_id', '0', '=');
         }
@@ -472,7 +472,7 @@ 
             $criteria->add(new Criteria('article_online', REFERENCES_STATUS_ONLINE, '='));
         }
         if (is_array($categoryId) && count($categoryId) > 0) {
-            $criteria->add(new Criteria('article_category_id', '(' . implode(',', $categoryId) . ')', 'IN'));
+            $criteria->add(new Criteria('article_category_id', '('.implode(',', $categoryId).')', 'IN'));
         } elseif ($categoryId > 0) {
             $criteria->add(new Criteria('article_category_id', $categoryId, '='));
         }
@@ -529,8 +529,8 @@ 
     public function getDistinctCategoriesIds()
     {
         $ret = array();
-        $sql = 'SELECT distinct(article_category_id) FROM ' . $this->table;
-        $sql .= ' ' . $this->getPermissionsCategoriesCriteria()->renderWhere();    // Permissions
+        $sql = 'SELECT distinct(article_category_id) FROM '.$this->table;
+        $sql .= ' '.$this->getPermissionsCategoriesCriteria()->renderWhere(); // Permissions
         $sql .= ' GROUP BY article_category_id ORDER BY article_category_id';
         $result = $this->db->query($sql);
         if (!$result) {
@@ -598,17 +598,17 @@ 
      */
     public function moveUp($currentId, $currentOrder)
     {
-        $sql_plus = 'SELECT article_id FROM ' . $this->table . ' WHERE article_weight = ' . ((int)$currentOrder - 1);
+        $sql_plus = 'SELECT article_id FROM '.$this->table.' WHERE article_weight = '.((int) $currentOrder - 1);
         $res_plus = $this->db->query($sql_plus);
         if ($this->db->getRowsNum($res_plus) == 0) {
             return;
         }
         $row_plus = $this->db->fetchArray($res_plus);
 
-        $upd1 = 'UPDATE ' . $this->table . ' SET article_weight = (article_weight + 1) WHERE article_id = ' . $row_plus['article_id'];
+        $upd1 = 'UPDATE '.$this->table.' SET article_weight = (article_weight + 1) WHERE article_id = '.$row_plus['article_id'];
         $this->db->queryF($upd1);
 
-        $upd2 = 'UPDATE ' . $this->table . ' SET article_weight = (article_weight - 1) WHERE article_id = ' . (int)$currentId;
+        $upd2 = 'UPDATE '.$this->table.' SET article_weight = (article_weight - 1) WHERE article_id = '.(int) $currentId;
         $this->db->queryF($upd2);
         $this->forceCacheClean();
     }
@@ -622,17 +622,17 @@ 
      */
     public function moveDown($currentId, $currentOrder)
     {
-        $sql_moins = 'SELECT article_id FROM ' . $this->table . ' WHERE article_weight = ' . ((int)$currentOrder + 1);
+        $sql_moins = 'SELECT article_id FROM '.$this->table.' WHERE article_weight = '.((int) $currentOrder + 1);
         $res_moins = $this->db->query($sql_moins);
         if ($this->db->getRowsNum($res_moins) == 0) {
             return;
         }
 
         $row_moins = $this->db->fetchArray($res_moins);
-        $upd1      = 'UPDATE ' . $this->table . ' SET article_weight = (article_weight - 1) WHERE article_id = ' . $row_moins['article_id'];
+        $upd1      = 'UPDATE '.$this->table.' SET article_weight = (article_weight - 1) WHERE article_id = '.$row_moins['article_id'];
         $this->db->queryF($upd1);
 
-        $upd2 = 'UPDATE ' . $this->table . ' SET article_weight =( article_weight + 1) WHERE article_id = ' . (int)$currentId;
+        $upd2 = 'UPDATE '.$this->table.' SET article_weight =( article_weight + 1) WHERE article_id = '.(int) $currentId;
         $this->db->queryF($upd2);
         $this->forceCacheClean();
     }

class/references_categories.php

Spacing +7 added lines, -7 removed lines

@@ -20,9 +20,9 @@ 
  */
 defined('XOOPS_ROOT_PATH') || exit('XOOPS root path not defined');
 
-require_once XOOPS_ROOT_PATH . '/kernel/object.php';
+require_once XOOPS_ROOT_PATH.'/kernel/object.php';
 if (!class_exists('references_XoopsPersistableObjectHandler')) {
-    require_once XOOPS_ROOT_PATH . '/modules/references/class/PersistableObjectHandler.php';
+    require_once XOOPS_ROOT_PATH.'/modules/references/class/PersistableObjectHandler.php';
 }
 
 class references_categories extends references_Object
@@ -56,9 +56,9 @@ 
     public function getUrl($shortVersion = false)
     {
         if (!$shortVersion) {
-            return REFERENCES_URL . 'category.php?category_id=' . $this->getVar('category_id');
+            return REFERENCES_URL.'category.php?category_id='.$this->getVar('category_id');
         } else {
-            return 'category.php?category_id=' . $this->getVar('category_id');
+            return 'category.php?category_id='.$this->getVar('category_id');
         }
     }
 
@@ -108,7 +108,7 @@ 
         $gperm_handler = xoops_getHandler('groupperm');
         $categories    = $gperm_handler->getItemIds($permissionsType, $groups, $currentModule->getVar('mid'));
         if (is_array($categories) && count($categories) > 0) {
-            $permissions[$permissionsType] = new Criteria('category_id', '(' . implode(',', $categories) . ')', 'IN');
+            $permissions[$permissionsType] = new Criteria('category_id', '('.implode(',', $categories).')', 'IN');
         } else {    // Ne peut rien voir
             $permissions[$permissionsType] = new Criteria('category_id', '0', '=');
         }
@@ -174,8 +174,8 @@ 
         if (count($categories) == 0) {
             return $ret;
         }
-        $jump  = REFERENCES_URL . 'category.php?category_id=';
-        $extra = " onchange='location=\"" . $jump . "\"+this.options[this.selectedIndex].value'";
+        $jump  = REFERENCES_URL.'category.php?category_id=';
+        $extra = " onchange='location=\"".$jump."\"+this.options[this.selectedIndex].value'";
 
         return references_utils::htmlSelect($selectName, $categories, $selected, true, '', false, 1, $extra);
     }