public-album.php ➔ convertorderbytrans() - Code Metrics - Inspection of "Updates (Yoda, Helper, Traits)" - mambax7/extgallery - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 72613d...069c91 )

by Michael

created 2017-09-24 19:17 UTC

public-album.php ➔ convertorderbytrans() F

↳ Parent: Project

Complexity

Conditions	11
Paths	1024

Size

Total Lines	36
Code Lines	23

Duplication

Lines	36
Ratio	100 %

Importance

Changes

Metric	Value
cc	11
eloc	23
nc	1024
nop	1
dl	36
loc	36
rs	3.1764
c	0
b	0
f	0

How to fix Complexity

<?php

/**
 * ExtGallery User area
 *
 * You may not change or alter any portion of this comment or credits
 * of supporting developers from this source code or any supporting source code
 * which is considered copyrighted (c) material of the original comment or credit authors.
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 *
 * @copyright   {@link https://xoops.org/ XOOPS Project}
 * @license     GNU GPL 2 (http://www.gnu.org/licenses/old-licenses/gpl-2.0.html)
 * @author      Zoullou (http://www.zoullou.net)
 * @package     ExtGallery
 */

include __DIR__ . '/header.php';
require_once XOOPS_ROOT_PATH . '/class/pagenav.php';
require_once XOOPS_ROOT_PATH . '/modules/extgallery/class/publicPerm.php';

$GLOBALS['xoopsOption']['template_main'] = 'extgallery_public-album.tpl';
include XOOPS_ROOT_PATH . '/header.php';

if (!isset($_GET['id'])) {

    $catId = 0;
} else {
    $catId = (int)$_GET['id'];
}
if (!isset($_GET['start'])) {

    $start = 0;
} else {
    $start = (int)$_GET['start'];
}

// HACK BLUETEEN TO SORT PHOTO BY USERS
//photo_date - photo_title - photo_hits - photo_rating
if (isset($_GET['sortby'])

    && ('photo_date' === $_GET['sortby']
        || 'photo_title' === $_GET['sortby']
        || 'photo_hits' === $_GET['sortby']
        || 'photo_rating' === $_GET['sortby'])) {
    $sortby = $_GET['sortby'];
} else {
    $sortby = 'photo_date';
}

//ASC ou DESC
if (isset($_GET['orderby']) && ('DESC' === $_GET['orderby'] || 'ASC' === $_GET['orderby'])) {

    $orderby = $_GET['orderby'];
} else {
    $orderby = $GLOBALS['xoopsModuleConfig']['display_set_order'];
}

$SortbyOrderby = $sortby . ' ' . $orderby;

/**
 * @param $SortbyOrderby
 *
 * @return array|string
 */
function convertorderbytrans($SortbyOrderby)

{
    $orderbyTrans = [];
    if ('photo_date DESC' === $SortbyOrderby) {
        $orderbyTrans = _MD_EXTGALLERY_ORDERBY_DATEASC;
    }
    if ('photo_date ASC' === $SortbyOrderby) {
        $orderbyTrans = _MD_EXTGALLERY_ORDERBY_DATEDESC;
    }
    if ('photo_title ASC' === $SortbyOrderby) {
        $orderbyTrans = _MD_EXTGALLERY_ORDERBY_TITREASC;
    }
    if ('photo_title DESC' === $SortbyOrderby) {
        $orderbyTrans = _MD_EXTGALLERY_ORDERBY_TITREDESC;
    }
    if ('uid ASC' === $SortbyOrderby) {
        $orderbyTrans = _MD_EXTGALLERY_ORDERBY_DESIGNERASC;
    }
    if ('uid DESC' === $SortbyOrderby) {
        $orderbyTrans = _MD_EXTGALLERY_ORDERBY_DESIGNERDESC;
    }
    if ('photo_hits DESC' === $SortbyOrderby) {
        $orderbyTrans = _MD_EXTGALLERY_ORDERBY_HITSASC;
    }
    if ('photo_hits ASC' === $SortbyOrderby) {
        $orderbyTrans = _MD_EXTGALLERY_ORDERBY_HITSDESC;
    }
    if ('photo_rating DESC' === $SortbyOrderby) {
        $orderbyTrans = _MD_EXTGALLERY_ORDERBY_NOTEASC;
    }
    if ('photo_rating ASC' === $SortbyOrderby) {
        $orderbyTrans = _MD_EXTGALLERY_ORDERBY_NOTEDESC;
    }

    return $orderbyTrans;
}

// Check the access permission
$permHandler = ExtgalleryPublicPermHandler::getInstance();
if ((null === $GLOBALS['xoopsUser'] || !is_object($GLOBALS['xoopsUser'])) || !$permHandler->isAllowed($GLOBALS['xoopsUser'], 'public_access', $catId)) {
    redirect_header('index.php', 3, _NOPERM);
}
/** @var ExtgalleryPublicCatHandler $catHandler */
$catHandler = xoops_getModuleHandler('publiccat', 'extgallery');
/** @var ExtgalleryPublicPhotoHandler $photoHandler */
$photoHandler = xoops_getModuleHandler('publicphoto', 'extgallery');

$catObj = $catHandler->getCat($catId);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();

if (null === $catObj) {
    include XOOPS_ROOT_PATH . '/footer.php';
    exit;
}

$ajaxeffect = $xoopsModuleConfig['use_ajax_effects'];
$xoopsTpl->assign('use_ajax_effects', $ajaxeffect);

$cat = $catHandler->objectToArray($catObj);
$xoopsTpl->assign('cat', $cat);

$catPath = $photoHandler->objectToArray($catHandler->getPath($catId));
$xoopsTpl->assign('catPath', $catPath);

$photos = $photoHandler->objectToArray($photoHandler->getAlbumPhotoPage($catId, $start, $sortby, $orderby), ['uid']); //xoops - blueteen - tri de l'affichage

// Plugin traitement
$plugin  = xoops_getModuleHandler('plugin', 'extgallery');
$nbPhoto = count($photos);
for ($i = 0; $i < $nbPhoto; ++$i) {
    $params = ['catId' => $catId, 'photoId' => $photos[$i]['photo_id'], 'link' => []];
    $plugin->triggerEvent('photoAlbumLink', $params);
    $photos[$i]['link'] = $params['link'];
}

$k = $xoopsModuleConfig['nb_column'] - (count($photos) % $xoopsModuleConfig['nb_column']);
if ($k != $xoopsModuleConfig['nb_column']) {

    for ($i = 0; $i < $k; ++$i) {
        $photos[] = [];
    }
}

// HACK DATE BY MAGE : DISPLAY PUBLICATION DATE
foreach (array_keys($photos) as $i) {

    if (isset($photos[$i]['photo_date'])) {
        $photos[$i]['photo_date'] = date(_SHORTDATESTRING, $photos[$i]['photo_date']);
    }
}
// END HACK DATE BY MAGE : DISPLAY PUBLICATION DATE

$xoopsTpl->assign('photos', $photos);
/** @var xos_opal_Theme $xoTheme */

$pageNav = new XoopsPageNav($photoHandler->getAlbumCount($catId), $xoopsModuleConfig['nb_column'] * $xoopsModuleConfig['nb_line'], $start, 'start', 'id=' . $catId . '&orderby=' . $orderby . '&sortby=' . $sortby); //xoops - blueteen - tri de l'affichage
$xoopsTpl->assign('pageNav', $pageNav->renderNav());
if (isset($catObj)) {

    $xoopsTpl->assign('xoops_pagetitle', $catObj->getVar('cat_name'));
    $xoTheme->addMeta('meta', 'description', $catObj->getVar('cat_desc'));
}

$jquery = $xoopsModuleConfig['enable_jquery'];
$xoopsTpl->assign('jquery', $jquery);
if (1 == $jquery && 'none' !== $ajaxeffect) {

    $xoTheme->addScript('browse.php?Frameworks/jquery/jquery.js');
    switch ($ajaxeffect) {
        case 'lightbox':
            $xoTheme->addScript('browse.php?Frameworks/jquery/plugins/jquery.lightbox.js');
            $xoTheme->addStylesheet('browse.php?modules/system/css/lightbox.css');
            break;

        case 'tooltip':
            $xoTheme->addScript('browse.php?modules/extgallery/assets/js/tooltip/image.tooltip.js');
            $xoTheme->addStylesheet('browse.php?modules/extgallery/assets/js/tooltip/image.tooltip.css');
            break;

        case 'overlay':
            $xoTheme->addScript('browse.php?modules/extgallery/assets/js/overlay/overlay.jquery.tools.min.js');
            $xoTheme->addStylesheet('browse.php?modules/extgallery/assets/js/overlay/overlay.css');
            break;

        case 'fancybox':
            $xoTheme->addScript('browse.php?modules/extgallery/assets/js/fancybox/mousewheel.js');
            $xoTheme->addScript('browse.php?modules/extgallery/assets/js/fancybox/fancybox.pack.js');
            $xoTheme->addStylesheet('browse.php?modules/extgallery/assets/js/fancybox/fancybox.css');
            break;

        case 'prettyphoto':
            $xoTheme->addScript('browse.php?modules/extgallery/assets/js/prettyphoto/jquery.prettyPhoto.js');
            $xoTheme->addStylesheet('browse.php?modules/extgallery/assets/js/prettyphoto/prettyPhoto.css');
            break;
    }
}

$rel                 = 'alternate';
$attributes['rel']   = $rel;
$attributes['type']  = 'application/rss+xml';
$attributes['title'] = _MD_EXTGALLERY_RSS;
$attributes['href']  = XOOPS_URL . '/modules/extgallery/public-rss.php';
$xoTheme->addMeta('link', $rel, $attributes);
$xoTheme->addStylesheet('modules/extgallery/assets/css/style.css');

$lang = [
    'hits'       => _MD_EXTGALLERY_HITS,
    'comments'   => _MD_EXTGALLERY_COMMENTS,
    'rate_score' => _MD_EXTGALLERY_RATING_SCORE
];
$xoopsTpl->assign('lang', $lang);

$xoopsTpl->assign('enableExtra', $xoopsModuleConfig['display_extra_field']);
$xoopsTpl->assign('enableRating', $xoopsModuleConfig['enable_rating']);
$xoopsTpl->assign('nbColumn', $xoopsModuleConfig['nb_column']);
$xoopsTpl->assign('extgalleryName', $xoopsModule->getVar('name'));
$xoopsTpl->assign('disp_ph_title', $xoopsModuleConfig['disp_ph_title']);

$xoopsTpl->assign('extgalleryID', $catId); //xoops - blueteen - tri de l'affichage
$xoopsTpl->assign('extgalleryStart', $start); //xoops -blueteen - tri de l'affichage
$xoopsTpl->assign('extgallerySortbyOrderby', _MD_EXTGALLERY_ORDERBY . convertorderbytrans($SortbyOrderby)); //xoops - blueteen - tri de l'affichage

//DNPROSSI - VOLTAN - added preferences option
//  enable_info, enable_submitter_lnk, enable_photo_hits
if ('album' === $xoopsModuleConfig['info_view'] || 'both' === $xoopsModuleConfig['info_view']) {

    if ('public' === $xoopsModuleConfig['pubusr_info_view'] || 'both' === $xoopsModuleConfig['pubusr_info_view']) {
        if (0 == $xoopsModuleConfig['enable_info']) {
            $enable_info = $xoopsModuleConfig['enable_info'];
        } else {
            $enable_info = 1;
        }
    } else {
        $enable_info = 1;
    }
} else {
    $enable_info = 1;
}
$xoopsTpl->assign('enable_info', $enable_info);
$xoopsTpl->assign('enable_photo_hits', $xoopsModuleConfig['enable_photo_hits']);
$xoopsTpl->assign('enable_submitter_lnk', $xoopsModuleConfig['enable_submitter_lnk']);
$xoopsTpl->assign('enable_show_comments', $xoopsModuleConfig['enable_show_comments']);
$xoopsTpl->assign('enable_date', $xoopsModuleConfig['enable_date']);
$xoopsTpl->assign('show_rss', $xoopsModuleConfig['show_rss']);

//for tooltip
$xoopsTpl->assign('album_tooltip_borderwidth', $xoopsModuleConfig['album_tooltip_borderwidth']);
$xoopsTpl->assign('album_tooltip_bordercolor', $xoopsModuleConfig['album_tooltip_bordercolor']);
$xoopsTpl->assign('album_tooltip_width', $xoopsModuleConfig['album_tooltip_width']);

//for overlay
$xoopsTpl->assign('album_overlay_bg', $xoopsModuleConfig['album_overlay_bg']);
$xoopsTpl->assign('album_overlay_width', $xoopsModuleConfig['album_overlay_width']);
$xoopsTpl->assign('album_overlay_height', $xoopsModuleConfig['album_overlay_height']);

//for fancybox
$xoopsTpl->assign('album_fancybox_color', $xoopsModuleConfig['album_fancybox_color']);
$xoopsTpl->assign('album_fancybox_opacity', $xoopsModuleConfig['album_fancybox_opacity']);
$xoopsTpl->assign('album_fancybox_tin', $xoopsModuleConfig['album_fancybox_tin']);
$xoopsTpl->assign('album_fancybox_tout', $xoopsModuleConfig['album_fancybox_tout']);
$xoopsTpl->assign('album_fancybox_title', $xoopsModuleConfig['album_fancybox_title']);
$xoopsTpl->assign('album_fancybox_showtype', $xoopsModuleConfig['album_fancybox_showtype']);

//for prettyphoto
$xoopsTpl->assign('album_prettyphoto_speed', $xoopsModuleConfig['album_prettyphoto_speed']);
$xoopsTpl->assign('album_prettyphoto_theme', $xoopsModuleConfig['album_prettyphoto_theme']);
$xoopsTpl->assign('album_prettyphoto_slidspeed', $xoopsModuleConfig['album_prettyphoto_slidspe']);
$xoopsTpl->assign('album_prettyphoto_autoplay', $xoopsModuleConfig['album_prettyphoto_autopla']);

include XOOPS_ROOT_PATH . '/footer.php';


1		<?php
		0 ignored issues – show Coding Style Compatibility introduced 2016-03-22 05:38 UTC by Report Bug Copy Issue Report For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 62 and the first side effect is on line 18. The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file. The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place. To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1. Loading history...
2		/**
3		* ExtGallery User area
4		*
5		* You may not change or alter any portion of this comment or credits
6		* of supporting developers from this source code or any supporting source code
7		* which is considered copyrighted (c) material of the original comment or credit authors.
8		* This program is distributed in the hope that it will be useful,
9		* but WITHOUT ANY WARRANTY; without even the implied warranty of
10		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
11		*
12		* @copyright {@link https://xoops.org/ XOOPS Project}
13		* @license GNU GPL 2 (http://www.gnu.org/licenses/old-licenses/gpl-2.0.html)
14		* @author Zoullou (http://www.zoullou.net)
15		* @package ExtGallery
16		*/
17
18		include __DIR__ . '/header.php';
19		require_once XOOPS_ROOT_PATH . '/class/pagenav.php';
20		require_once XOOPS_ROOT_PATH . '/modules/extgallery/class/publicPerm.php';
21
22		$GLOBALS['xoopsOption']['template_main'] = 'extgallery_public-album.tpl';
23		include XOOPS_ROOT_PATH . '/header.php';
24
25	View Code Duplication	if (!isset($_GET['id'])) {
		0 ignored issues – show Duplication introduced 2016-03-22 05:38 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
26		$catId = 0;
27		} else {
28		$catId = (int)$_GET['id'];
29		}
30	View Code Duplication	if (!isset($_GET['start'])) {
		0 ignored issues – show Duplication introduced 2016-03-22 05:38 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
31		$start = 0;
32		} else {
33		$start = (int)$_GET['start'];
34		}
35
36		// HACK BLUETEEN TO SORT PHOTO BY USERS
37		//photo_date - photo_title - photo_hits - photo_rating
38	View Code Duplication	if (isset($_GET['sortby'])
		0 ignored issues – show Duplication introduced 2016-08-10 20:00 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
39		&& ('photo_date' === $_GET['sortby']
40		\|\| 'photo_title' === $_GET['sortby']
41		\|\| 'photo_hits' === $_GET['sortby']
42		\|\| 'photo_rating' === $_GET['sortby'])) {
43		$sortby = $_GET['sortby'];
44		} else {
45		$sortby = 'photo_date';
46		}
47
48		//ASC ou DESC
49	View Code Duplication	if (isset($_GET['orderby']) && ('DESC' === $_GET['orderby'] \|\| 'ASC' === $_GET['orderby'])) {
		0 ignored issues – show Duplication introduced 2017-09-24 19:19 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
50		$orderby = $_GET['orderby'];
51		} else {
52		$orderby = $GLOBALS['xoopsModuleConfig']['display_set_order'];
53		}
54
55		$SortbyOrderby = $sortby . ' ' . $orderby;
56
57		/**
58		* @param $SortbyOrderby
59		*
60		* @return array\|string
61		*/
62	View Code Duplication	function convertorderbytrans($SortbyOrderby)
		0 ignored issues – show Duplication introduced 2016-03-22 05:38 UTC by Report Bug Copy Issue Report This function seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
63		{
64		$orderbyTrans = [];
65		if ('photo_date DESC' === $SortbyOrderby) {
66		$orderbyTrans = _MD_EXTGALLERY_ORDERBY_DATEASC;
67		}
68		if ('photo_date ASC' === $SortbyOrderby) {
69		$orderbyTrans = _MD_EXTGALLERY_ORDERBY_DATEDESC;
70		}
71		if ('photo_title ASC' === $SortbyOrderby) {
72		$orderbyTrans = _MD_EXTGALLERY_ORDERBY_TITREASC;
73		}
74		if ('photo_title DESC' === $SortbyOrderby) {
75		$orderbyTrans = _MD_EXTGALLERY_ORDERBY_TITREDESC;
76		}
77		if ('uid ASC' === $SortbyOrderby) {
78		$orderbyTrans = _MD_EXTGALLERY_ORDERBY_DESIGNERASC;
79		}
80		if ('uid DESC' === $SortbyOrderby) {
81		$orderbyTrans = _MD_EXTGALLERY_ORDERBY_DESIGNERDESC;
82		}
83		if ('photo_hits DESC' === $SortbyOrderby) {
84		$orderbyTrans = _MD_EXTGALLERY_ORDERBY_HITSASC;
85		}
86		if ('photo_hits ASC' === $SortbyOrderby) {
87		$orderbyTrans = _MD_EXTGALLERY_ORDERBY_HITSDESC;
88		}
89		if ('photo_rating DESC' === $SortbyOrderby) {
90		$orderbyTrans = _MD_EXTGALLERY_ORDERBY_NOTEASC;
91		}
92		if ('photo_rating ASC' === $SortbyOrderby) {
93		$orderbyTrans = _MD_EXTGALLERY_ORDERBY_NOTEDESC;
94		}
95
96		return $orderbyTrans;
97		}
98
99		// Check the access permission
100		$permHandler = ExtgalleryPublicPermHandler::getInstance();
101		if ((null === $GLOBALS['xoopsUser'] \|\| !is_object($GLOBALS['xoopsUser'])) \|\| !$permHandler->isAllowed($GLOBALS['xoopsUser'], 'public_access', $catId)) {
102		redirect_header('index.php', 3, _NOPERM);
103		}
104		/** @var ExtgalleryPublicCatHandler $catHandler */
105		$catHandler = xoops_getModuleHandler('publiccat', 'extgallery');
106		/** @var ExtgalleryPublicPhotoHandler $photoHandler */
107		$photoHandler = xoops_getModuleHandler('publicphoto', 'extgallery');
108
109		$catObj = $catHandler->getCat($catId);
		0 ignored issues – show Bug introduced 2016-11-19 19:25 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$catObj` is correct as `$catHandler->getCat($catId)` (which targets `ExtgalleryCatHandler::getCat()`) seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
110
111		if (null === $catObj) {
112		include XOOPS_ROOT_PATH . '/footer.php';
113		exit;
114		}
115
116		$ajaxeffect = $xoopsModuleConfig['use_ajax_effects'];
117		$xoopsTpl->assign('use_ajax_effects', $ajaxeffect);
118
119		$cat = $catHandler->objectToArray($catObj);
120		$xoopsTpl->assign('cat', $cat);
121
122		$catPath = $photoHandler->objectToArray($catHandler->getPath($catId));
123		$xoopsTpl->assign('catPath', $catPath);
124
125		$photos = $photoHandler->objectToArray($photoHandler->getAlbumPhotoPage($catId, $start, $sortby, $orderby), ['uid']); //xoops - blueteen - tri de l'affichage
126
127		// Plugin traitement
128		$plugin = xoops_getModuleHandler('plugin', 'extgallery');
129		$nbPhoto = count($photos);
130		for ($i = 0; $i < $nbPhoto; ++$i) {
131		$params = ['catId' => $catId, 'photoId' => $photos[$i]['photo_id'], 'link' => []];
132		$plugin->triggerEvent('photoAlbumLink', $params);
133		$photos[$i]['link'] = $params['link'];
134		}
135
136		$k = $xoopsModuleConfig['nb_column'] - (count($photos) % $xoopsModuleConfig['nb_column']);
137	View Code Duplication	if ($k != $xoopsModuleConfig['nb_column']) {
		0 ignored issues – show Duplication introduced 2016-03-22 05:38 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
138		for ($i = 0; $i < $k; ++$i) {
139		$photos[] = [];
140		}
141		}
142
143		// HACK DATE BY MAGE : DISPLAY PUBLICATION DATE
144	View Code Duplication	foreach (array_keys($photos) as $i) {
		0 ignored issues – show Duplication introduced 2016-03-22 05:38 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
145		if (isset($photos[$i]['photo_date'])) {
146		$photos[$i]['photo_date'] = date(_SHORTDATESTRING, $photos[$i]['photo_date']);
147		}
148		}
149		// END HACK DATE BY MAGE : DISPLAY PUBLICATION DATE
150
151		$xoopsTpl->assign('photos', $photos);
152		/** @var xos_opal_Theme $xoTheme */
153
154		$pageNav = new XoopsPageNav($photoHandler->getAlbumCount($catId), $xoopsModuleConfig['nb_column'] * $xoopsModuleConfig['nb_line'], $start, 'start', 'id=' . $catId . '&orderby=' . $orderby . '&sortby=' . $sortby); //xoops - blueteen - tri de l'affichage
155		$xoopsTpl->assign('pageNav', $pageNav->renderNav());
156	View Code Duplication	if (isset($catObj)) {
		0 ignored issues – show Duplication introduced 2016-03-22 05:38 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
157		$xoopsTpl->assign('xoops_pagetitle', $catObj->getVar('cat_name'));
158		$xoTheme->addMeta('meta', 'description', $catObj->getVar('cat_desc'));
159		}
160
161		$jquery = $xoopsModuleConfig['enable_jquery'];
162		$xoopsTpl->assign('jquery', $jquery);
163	View Code Duplication	if (1 == $jquery && 'none' !== $ajaxeffect) {
		0 ignored issues – show Duplication introduced 2017-09-24 19:19 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
164		$xoTheme->addScript('browse.php?Frameworks/jquery/jquery.js');
165		switch ($ajaxeffect) {
166		case 'lightbox':
167		$xoTheme->addScript('browse.php?Frameworks/jquery/plugins/jquery.lightbox.js');
168		$xoTheme->addStylesheet('browse.php?modules/system/css/lightbox.css');
169		break;
170
171		case 'tooltip':
172		$xoTheme->addScript('browse.php?modules/extgallery/assets/js/tooltip/image.tooltip.js');
173		$xoTheme->addStylesheet('browse.php?modules/extgallery/assets/js/tooltip/image.tooltip.css');
174		break;
175
176		case 'overlay':
177		$xoTheme->addScript('browse.php?modules/extgallery/assets/js/overlay/overlay.jquery.tools.min.js');
178		$xoTheme->addStylesheet('browse.php?modules/extgallery/assets/js/overlay/overlay.css');
179		break;
180
181		case 'fancybox':
182		$xoTheme->addScript('browse.php?modules/extgallery/assets/js/fancybox/mousewheel.js');
183		$xoTheme->addScript('browse.php?modules/extgallery/assets/js/fancybox/fancybox.pack.js');
184		$xoTheme->addStylesheet('browse.php?modules/extgallery/assets/js/fancybox/fancybox.css');
185		break;
186
187		case 'prettyphoto':
188		$xoTheme->addScript('browse.php?modules/extgallery/assets/js/prettyphoto/jquery.prettyPhoto.js');
189		$xoTheme->addStylesheet('browse.php?modules/extgallery/assets/js/prettyphoto/prettyPhoto.css');
190		break;
191		}
192		}
193
194		$rel = 'alternate';
195		$attributes['rel'] = $rel;
196		$attributes['type'] = 'application/rss+xml';
197		$attributes['title'] = _MD_EXTGALLERY_RSS;
198		$attributes['href'] = XOOPS_URL . '/modules/extgallery/public-rss.php';
199		$xoTheme->addMeta('link', $rel, $attributes);
200		$xoTheme->addStylesheet('modules/extgallery/assets/css/style.css');
201
202		$lang = [
203		'hits' => _MD_EXTGALLERY_HITS,
204		'comments' => _MD_EXTGALLERY_COMMENTS,
205		'rate_score' => _MD_EXTGALLERY_RATING_SCORE
206		];
207		$xoopsTpl->assign('lang', $lang);
208
209		$xoopsTpl->assign('enableExtra', $xoopsModuleConfig['display_extra_field']);
210		$xoopsTpl->assign('enableRating', $xoopsModuleConfig['enable_rating']);
211		$xoopsTpl->assign('nbColumn', $xoopsModuleConfig['nb_column']);
212		$xoopsTpl->assign('extgalleryName', $xoopsModule->getVar('name'));
213		$xoopsTpl->assign('disp_ph_title', $xoopsModuleConfig['disp_ph_title']);
214
215		$xoopsTpl->assign('extgalleryID', $catId); //xoops - blueteen - tri de l'affichage
216		$xoopsTpl->assign('extgalleryStart', $start); //xoops -blueteen - tri de l'affichage
217		$xoopsTpl->assign('extgallerySortbyOrderby', _MD_EXTGALLERY_ORDERBY . convertorderbytrans($SortbyOrderby)); //xoops - blueteen - tri de l'affichage
218
219		//DNPROSSI - VOLTAN - added preferences option
220		// enable_info, enable_submitter_lnk, enable_photo_hits
221	View Code Duplication	if ('album' === $xoopsModuleConfig['info_view'] \|\| 'both' === $xoopsModuleConfig['info_view']) {
		0 ignored issues – show Duplication introduced 2017-09-24 19:19 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
222		if ('public' === $xoopsModuleConfig['pubusr_info_view'] \|\| 'both' === $xoopsModuleConfig['pubusr_info_view']) {
223		if (0 == $xoopsModuleConfig['enable_info']) {
224		$enable_info = $xoopsModuleConfig['enable_info'];
225		} else {
226		$enable_info = 1;
227		}
228		} else {
229		$enable_info = 1;
230		}
231		} else {
232		$enable_info = 1;
233		}
234		$xoopsTpl->assign('enable_info', $enable_info);
235		$xoopsTpl->assign('enable_photo_hits', $xoopsModuleConfig['enable_photo_hits']);
236		$xoopsTpl->assign('enable_submitter_lnk', $xoopsModuleConfig['enable_submitter_lnk']);
237		$xoopsTpl->assign('enable_show_comments', $xoopsModuleConfig['enable_show_comments']);
238		$xoopsTpl->assign('enable_date', $xoopsModuleConfig['enable_date']);
239		$xoopsTpl->assign('show_rss', $xoopsModuleConfig['show_rss']);
240
241		//for tooltip
242		$xoopsTpl->assign('album_tooltip_borderwidth', $xoopsModuleConfig['album_tooltip_borderwidth']);
243		$xoopsTpl->assign('album_tooltip_bordercolor', $xoopsModuleConfig['album_tooltip_bordercolor']);
244		$xoopsTpl->assign('album_tooltip_width', $xoopsModuleConfig['album_tooltip_width']);
245
246		//for overlay
247		$xoopsTpl->assign('album_overlay_bg', $xoopsModuleConfig['album_overlay_bg']);
248		$xoopsTpl->assign('album_overlay_width', $xoopsModuleConfig['album_overlay_width']);
249		$xoopsTpl->assign('album_overlay_height', $xoopsModuleConfig['album_overlay_height']);
250
251		//for fancybox
252		$xoopsTpl->assign('album_fancybox_color', $xoopsModuleConfig['album_fancybox_color']);
253		$xoopsTpl->assign('album_fancybox_opacity', $xoopsModuleConfig['album_fancybox_opacity']);
254		$xoopsTpl->assign('album_fancybox_tin', $xoopsModuleConfig['album_fancybox_tin']);
255		$xoopsTpl->assign('album_fancybox_tout', $xoopsModuleConfig['album_fancybox_tout']);
256		$xoopsTpl->assign('album_fancybox_title', $xoopsModuleConfig['album_fancybox_title']);
257		$xoopsTpl->assign('album_fancybox_showtype', $xoopsModuleConfig['album_fancybox_showtype']);
258
259		//for prettyphoto
260		$xoopsTpl->assign('album_prettyphoto_speed', $xoopsModuleConfig['album_prettyphoto_speed']);
261		$xoopsTpl->assign('album_prettyphoto_theme', $xoopsModuleConfig['album_prettyphoto_theme']);
262		$xoopsTpl->assign('album_prettyphoto_slidspeed', $xoopsModuleConfig['album_prettyphoto_slidspe']);
263		$xoopsTpl->assign('album_prettyphoto_autoplay', $xoopsModuleConfig['album_prettyphoto_autopla']);
264
265		include XOOPS_ROOT_PATH . '/footer.php';
266

mambax7 / extgallery

Push — master ( 72613d...069c91 )

public-album.php ➔ convertorderbytrans() F

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like