mambax7 / smartobject

admin/adsense.php

Indentation +76 added lines, -76 removed lines

@@ -12,32 +12,32 @@ 
 
 function editclass($showmenu = false, $adsenseid = 0, $clone = false)
 {
-    global $smartobjectAdsenseHandler;
-
-    $adsenseObj = $smartobjectAdsenseHandler->get($adsenseid);
-
-    if (!$clone && !$adsenseObj->isNew()) {
-        if ($showmenu) {
-            //smart_adminMenu(3, _AM_SOBJECT_ADSENSES . " > " . _AM_SOBJECT_EDITING);
-        }
-        smart_collapsableBar('adsenseedit', _AM_SOBJECT_ADSENSES_EDIT, _AM_SOBJECT_ADSENSES_EDIT_INFO);
-
-        $sform = $adsenseObj->getForm(_AM_SOBJECT_ADSENSES_EDIT, 'addadsense');
-        $sform->display();
-        smart_close_collapsable('adsenseedit');
-    } else {
-        $adsenseObj->setVar('adsenseid', 0);
-        $adsenseObj->setVar('tag', '');
-
-        if ($showmenu) {
-            //smart_adminMenu(3, _AM_SOBJECT_ADSENSES . " > " . _CO_SOBJECT_CREATINGNEW);
-        }
-
-        smart_collapsableBar('adsensecreate', _AM_SOBJECT_ADSENSES_CREATE, _AM_SOBJECT_ADSENSES_CREATE_INFO);
-        $sform = $adsenseObj->getForm(_AM_SOBJECT_ADSENSES_CREATE, 'addadsense', false, false, false, true);
-        $sform->display();
-        smart_close_collapsable('adsensecreate');
-    }
+	global $smartobjectAdsenseHandler;
+
+	$adsenseObj = $smartobjectAdsenseHandler->get($adsenseid);
+
+	if (!$clone && !$adsenseObj->isNew()) {
+		if ($showmenu) {
+			//smart_adminMenu(3, _AM_SOBJECT_ADSENSES . " > " . _AM_SOBJECT_EDITING);
+		}
+		smart_collapsableBar('adsenseedit', _AM_SOBJECT_ADSENSES_EDIT, _AM_SOBJECT_ADSENSES_EDIT_INFO);
+
+		$sform = $adsenseObj->getForm(_AM_SOBJECT_ADSENSES_EDIT, 'addadsense');
+		$sform->display();
+		smart_close_collapsable('adsenseedit');
+	} else {
+		$adsenseObj->setVar('adsenseid', 0);
+		$adsenseObj->setVar('tag', '');
+
+		if ($showmenu) {
+			//smart_adminMenu(3, _AM_SOBJECT_ADSENSES . " > " . _CO_SOBJECT_CREATINGNEW);
+		}
+
+		smart_collapsableBar('adsensecreate', _AM_SOBJECT_ADSENSES_CREATE, _AM_SOBJECT_ADSENSES_CREATE_INFO);
+		$sform = $adsenseObj->getForm(_AM_SOBJECT_ADSENSES_CREATE, 'addadsense', false, false, false, true);
+		$sform->display();
+		smart_close_collapsable('adsensecreate');
+	}
 }
 
 require_once __DIR__ . '/admin_header.php';
@@ -50,72 +50,72 @@ 
 $op = '';
 
 if (isset($_GET['op'])) {
-    $op = $_GET['op'];
+	$op = $_GET['op'];
 }
 if (isset($_POST['op'])) {
-    $op = $_POST['op'];
+	$op = $_POST['op'];
 }
 
 switch ($op) {
-    case 'mod':
+	case 'mod':
 
-        $adsenseid = isset($_GET['adsenseid']) ? (int)$_GET['adsenseid'] : 0;
+		$adsenseid = isset($_GET['adsenseid']) ? (int)$_GET['adsenseid'] : 0;
 
-        smart_xoops_cp_header();
-        $adminObject->displayNavigation(basename(__FILE__));
+		smart_xoops_cp_header();
+		$adminObject->displayNavigation(basename(__FILE__));
 
-        editclass(true, $adsenseid);
-        break;
+		editclass(true, $adsenseid);
+		break;
 
-    case 'clone':
+	case 'clone':
 
-        $adsenseid = isset($_GET['adsenseid']) ? (int)$_GET['adsenseid'] : 0;
+		$adsenseid = isset($_GET['adsenseid']) ? (int)$_GET['adsenseid'] : 0;
 
-        smart_xoops_cp_header();
-        $adminObject->displayNavigation(basename(__FILE__));
+		smart_xoops_cp_header();
+		$adminObject->displayNavigation(basename(__FILE__));
 
-        editclass(true, $adsenseid, true);
-        break;
+		editclass(true, $adsenseid, true);
+		break;
 
-    case 'addadsense':
-        if (@require_once SMARTOBJECT_ROOT_PATH . 'include/captcha/captcha.php') {
-            $xoopsCaptcha = XoopsCaptcha::getInstance();
-            if (!$xoopsCaptcha->verify()) {
-                redirect_header('javascript:history.go(-1);', 3, $xoopsCaptcha->getMessage());
-            }
-        }
-        require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
-        $controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectAdsenseHandler);
-        $controller->storeFromDefaultForm(_AM_SOBJECT_ADSENSES_CREATED, _AM_SOBJECT_ADSENSES_MODIFIED);
-        break;
+	case 'addadsense':
+		if (@require_once SMARTOBJECT_ROOT_PATH . 'include/captcha/captcha.php') {
+			$xoopsCaptcha = XoopsCaptcha::getInstance();
+			if (!$xoopsCaptcha->verify()) {
+				redirect_header('javascript:history.go(-1);', 3, $xoopsCaptcha->getMessage());
+			}
+		}
+		require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
+		$controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectAdsenseHandler);
+		$controller->storeFromDefaultForm(_AM_SOBJECT_ADSENSES_CREATED, _AM_SOBJECT_ADSENSES_MODIFIED);
+		break;
 
-    case 'del':
+	case 'del':
 
-        require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
-        $controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectAdsenseHandler);
-        $controller->handleObjectDeletion();
+		require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
+		$controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectAdsenseHandler);
+		$controller->handleObjectDeletion();
 
-        break;
+		break;
 
-    default:
+	default:
 
-        smart_xoops_cp_header();
-        $adminObject->displayNavigation(basename(__FILE__));
+		smart_xoops_cp_header();
+		$adminObject->displayNavigation(basename(__FILE__));
 
-        //smart_adminMenu(3, _AM_SOBJECT_ADSENSES);
+		//smart_adminMenu(3, _AM_SOBJECT_ADSENSES);
 
-        smart_collapsableBar('createdadsenses', _AM_SOBJECT_ADSENSES, _AM_SOBJECT_ADSENSES_DSC);
+		smart_collapsableBar('createdadsenses', _AM_SOBJECT_ADSENSES, _AM_SOBJECT_ADSENSES_DSC);
 
-        require_once SMARTOBJECT_ROOT_PATH . 'class/smartobjecttable.php';
-        $objectTable = new XoopsModules\Smartobject\SmartObjectTable($smartobjectAdsenseHandler);
-        $objectTable->addColumn(new XoopsModules\Smartobject\SmartObjectColumn('description', 'left'));
-        $objectTable->addColumn(new XoopsModules\Smartobject\SmartObjectColumn(_AM_SOBJECT_ADSENSE_TAG, 'center', 200, 'getXoopsCode'));
+		require_once SMARTOBJECT_ROOT_PATH . 'class/smartobjecttable.php';
+		$objectTable = new XoopsModules\Smartobject\SmartObjectTable($smartobjectAdsenseHandler);
+		$objectTable->addColumn(new XoopsModules\Smartobject\SmartObjectColumn('description', 'left'));
+		$objectTable->addColumn(new XoopsModules\Smartobject\SmartObjectColumn(_AM_SOBJECT_ADSENSE_TAG, 'center', 200, 'getXoopsCode'));
 
-        //      $objectTable->addCustomAction('getCreateItemLink');
-        //      $objectTable->addCustomAction('getCreateAttributLink');
+		//      $objectTable->addCustomAction('getCreateItemLink');
+		//      $objectTable->addCustomAction('getCreateAttributLink');
 
-        $objectTable->addIntroButton('addadsense', 'adsense.php?op=mod', _AM_SOBJECT_ADSENSES_CREATE);
-        /*
+		$objectTable->addIntroButton('addadsense', 'adsense.php?op=mod', _AM_SOBJECT_ADSENSES_CREATE);
+		/*
                 $criteria_upcoming = new \CriteriaCompo();
                 $criteria_upcoming->add(new \Criteria('start_date', time(), '>'));
                 $objectTable->addFilter(_AM_SOBJECT_FILTER_UPCOMING, array(
@@ -139,16 +139,16 @@ 
                                             'criteria' => $criteria_last30days
                 ));
         */
-        $objectTable->addQuickSearch(['title', 'summary', 'description']);
-        $objectTable->addCustomAction('getCloneLink');
+		$objectTable->addQuickSearch(['title', 'summary', 'description']);
+		$objectTable->addCustomAction('getCloneLink');
 
-        $objectTable->render();
+		$objectTable->render();
 
-        echo '<br>';
-        smart_close_collapsable('createdadsenses');
-        echo '<br>';
+		echo '<br>';
+		smart_close_collapsable('createdadsenses');
+		echo '<br>';
 
-        break;
+		break;
 }
 
 //smart_modFooter();

Switch Indentation +48 added lines, -48 removed lines

@@ -57,65 +57,65 @@ 
 }
 
 switch ($op) {
-    case 'mod':
+    	case 'mod':
 
-        $adsenseid = isset($_GET['adsenseid']) ? (int)$_GET['adsenseid'] : 0;
+        	$adsenseid = isset($_GET['adsenseid']) ? (int)$_GET['adsenseid'] : 0;
 
-        smart_xoops_cp_header();
-        $adminObject->displayNavigation(basename(__FILE__));
+        	smart_xoops_cp_header();
+        	$adminObject->displayNavigation(basename(__FILE__));
 
-        editclass(true, $adsenseid);
-        break;
+        	editclass(true, $adsenseid);
+        	break;
 
-    case 'clone':
+    	case 'clone':
 
-        $adsenseid = isset($_GET['adsenseid']) ? (int)$_GET['adsenseid'] : 0;
+        	$adsenseid = isset($_GET['adsenseid']) ? (int)$_GET['adsenseid'] : 0;
 
-        smart_xoops_cp_header();
-        $adminObject->displayNavigation(basename(__FILE__));
+        	smart_xoops_cp_header();
+        	$adminObject->displayNavigation(basename(__FILE__));
 
-        editclass(true, $adsenseid, true);
-        break;
+        	editclass(true, $adsenseid, true);
+        	break;
 
-    case 'addadsense':
-        if (@require_once SMARTOBJECT_ROOT_PATH . 'include/captcha/captcha.php') {
-            $xoopsCaptcha = XoopsCaptcha::getInstance();
-            if (!$xoopsCaptcha->verify()) {
-                redirect_header('javascript:history.go(-1);', 3, $xoopsCaptcha->getMessage());
-            }
-        }
-        require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
-        $controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectAdsenseHandler);
-        $controller->storeFromDefaultForm(_AM_SOBJECT_ADSENSES_CREATED, _AM_SOBJECT_ADSENSES_MODIFIED);
-        break;
+    	case 'addadsense':
+        	if (@require_once SMARTOBJECT_ROOT_PATH . 'include/captcha/captcha.php') {
+            	$xoopsCaptcha = XoopsCaptcha::getInstance();
+            	if (!$xoopsCaptcha->verify()) {
+                	redirect_header('javascript:history.go(-1);', 3, $xoopsCaptcha->getMessage());
+            	}
+        	}
+        	require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
+        	$controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectAdsenseHandler);
+        	$controller->storeFromDefaultForm(_AM_SOBJECT_ADSENSES_CREATED, _AM_SOBJECT_ADSENSES_MODIFIED);
+        	break;
 
-    case 'del':
+    	case 'del':
 
-        require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
-        $controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectAdsenseHandler);
-        $controller->handleObjectDeletion();
+        	require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
+        	$controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectAdsenseHandler);
+        	$controller->handleObjectDeletion();
 
-        break;
+        	break;
 
-    default:
+    	default:
 
-        smart_xoops_cp_header();
-        $adminObject->displayNavigation(basename(__FILE__));
+        	smart_xoops_cp_header();
+        	$adminObject->displayNavigation(basename(__FILE__));
 
-        //smart_adminMenu(3, _AM_SOBJECT_ADSENSES);
+        	//smart_adminMenu(3, _AM_SOBJECT_ADSENSES);
 
-        smart_collapsableBar('createdadsenses', _AM_SOBJECT_ADSENSES, _AM_SOBJECT_ADSENSES_DSC);
+        	smart_collapsableBar('createdadsenses', _AM_SOBJECT_ADSENSES, _AM_SOBJECT_ADSENSES_DSC);
 
-        require_once SMARTOBJECT_ROOT_PATH . 'class/smartobjecttable.php';
-        $objectTable = new XoopsModules\Smartobject\SmartObjectTable($smartobjectAdsenseHandler);
-        $objectTable->addColumn(new XoopsModules\Smartobject\SmartObjectColumn('description', 'left'));
-        $objectTable->addColumn(new XoopsModules\Smartobject\SmartObjectColumn(_AM_SOBJECT_ADSENSE_TAG, 'center', 200, 'getXoopsCode'));
+        	require_once SMARTOBJECT_ROOT_PATH . 'class/smartobjecttable.php';
+        	$objectTable = new XoopsModules\Smartobject\SmartObjectTable($smartobjectAdsenseHandler);
+        	$objectTable->addColumn(new XoopsModules\Smartobject\SmartObjectColumn('description', 'left'));
+        	$objectTable->addColumn(new XoopsModules\Smartobject\SmartObjectColumn(_AM_SOBJECT_ADSENSE_TAG, 'center', 200, 'getXoopsCode'));
 
-        //      $objectTable->addCustomAction('getCreateItemLink');
-        //      $objectTable->addCustomAction('getCreateAttributLink');
+        	//      $objectTable->addCustomAction('getCreateItemLink');
+        	//      $objectTable->addCustomAction('getCreateAttributLink');
 
-        $objectTable->addIntroButton('addadsense', 'adsense.php?op=mod', _AM_SOBJECT_ADSENSES_CREATE);
-        /*
+        	$objectTable->addIntroButton('addadsense', 'adsense.php?op=mod', _AM_SOBJECT_ADSENSES_CREATE);
+        	/*
                 $criteria_upcoming = new \CriteriaCompo();
                 $criteria_upcoming->add(new \Criteria('start_date', time(), '>'));
                 $objectTable->addFilter(_AM_SOBJECT_FILTER_UPCOMING, array(
@@ -139,16 +139,16 @@ 
                                             'criteria' => $criteria_last30days
                 ));
         */
-        $objectTable->addQuickSearch(['title', 'summary', 'description']);
-        $objectTable->addCustomAction('getCloneLink');
+        	$objectTable->addQuickSearch(['title', 'summary', 'description']);
+        	$objectTable->addCustomAction('getCloneLink');
 
-        $objectTable->render();
+        	$objectTable->render();
 
-        echo '<br>';
-        smart_close_collapsable('createdadsenses');
-        echo '<br>';
+        	echo '<br>';
+        	smart_close_collapsable('createdadsenses');
+        	echo '<br>';
 
-        break;
+        	break;
 }
 
 //smart_modFooter();

Spacing +10 added lines, -10 removed lines

@@ -40,9 +40,9 @@ 
     }
 }
 
-require_once __DIR__ . '/admin_header.php';
-require_once SMARTOBJECT_ROOT_PATH . 'class/smartobjecttable.php';
-require_once SMARTOBJECT_ROOT_PATH . 'class/adsense.php';
+require_once __DIR__.'/admin_header.php';
+require_once SMARTOBJECT_ROOT_PATH.'class/smartobjecttable.php';
+require_once SMARTOBJECT_ROOT_PATH.'class/adsense.php';
 $smartobjectAdsenseHandler = xoops_getModuleHandler('adsense');
 smart_loadLanguageFile('smartobject', 'adsense');
 $adminObject = \Xmf\Module\Admin::getInstance();
@@ -59,7 +59,7 @@ 
 switch ($op) {
     case 'mod':
 
-        $adsenseid = isset($_GET['adsenseid']) ? (int)$_GET['adsenseid'] : 0;
+        $adsenseid = isset($_GET['adsenseid']) ? (int) $_GET['adsenseid'] : 0;
 
         smart_xoops_cp_header();
         $adminObject->displayNavigation(basename(__FILE__));
@@ -69,7 +69,7 @@ 
 
     case 'clone':
 
-        $adsenseid = isset($_GET['adsenseid']) ? (int)$_GET['adsenseid'] : 0;
+        $adsenseid = isset($_GET['adsenseid']) ? (int) $_GET['adsenseid'] : 0;
 
         smart_xoops_cp_header();
         $adminObject->displayNavigation(basename(__FILE__));
@@ -78,20 +78,20 @@ 
         break;
 
     case 'addadsense':
-        if (@require_once SMARTOBJECT_ROOT_PATH . 'include/captcha/captcha.php') {
+        if (@require_once SMARTOBJECT_ROOT_PATH.'include/captcha/captcha.php') {
             $xoopsCaptcha = XoopsCaptcha::getInstance();
             if (!$xoopsCaptcha->verify()) {
                 redirect_header('javascript:history.go(-1);', 3, $xoopsCaptcha->getMessage());
             }
         }
-        require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
+        require_once XOOPS_ROOT_PATH.'/modules/smartobject/class/smartobjectcontroller.php';
         $controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectAdsenseHandler);
         $controller->storeFromDefaultForm(_AM_SOBJECT_ADSENSES_CREATED, _AM_SOBJECT_ADSENSES_MODIFIED);
         break;
 
     case 'del':
 
-        require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
+        require_once XOOPS_ROOT_PATH.'/modules/smartobject/class/smartobjectcontroller.php';
         $controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectAdsenseHandler);
         $controller->handleObjectDeletion();
 
@@ -106,7 +106,7 @@ 
 
         smart_collapsableBar('createdadsenses', _AM_SOBJECT_ADSENSES, _AM_SOBJECT_ADSENSES_DSC);
 
-        require_once SMARTOBJECT_ROOT_PATH . 'class/smartobjecttable.php';
+        require_once SMARTOBJECT_ROOT_PATH.'class/smartobjecttable.php';
         $objectTable = new XoopsModules\Smartobject\SmartObjectTable($smartobjectAdsenseHandler);
         $objectTable->addColumn(new XoopsModules\Smartobject\SmartObjectColumn('description', 'left'));
         $objectTable->addColumn(new XoopsModules\Smartobject\SmartObjectColumn(_AM_SOBJECT_ADSENSE_TAG, 'center', 200, 'getXoopsCode'));
@@ -153,4 +153,4 @@ 
 
 //smart_modFooter();
 //xoops_cp_footer();
-require_once __DIR__ . '/admin_footer.php';
+require_once __DIR__.'/admin_footer.php';

admin/about.php

Spacing +2 added lines, -2 removed lines

@@ -17,11 +17,11 @@
  * @author       XOOPS Development Team
  */
 
-require_once __DIR__ . '/admin_header.php';
+require_once __DIR__.'/admin_header.php';
 xoops_cp_header();
 
 $adminObject->displayNavigation(basename(__FILE__));
 $adminObject::setPaypal('xoopsfoundation@gmail.com');
 $adminObject->displayAbout(false);
 
-require_once __DIR__ . '/admin_footer.php';
+require_once __DIR__.'/admin_footer.php';

admin/menu.php

Indentation +49 added lines, -49 removed lines

@@ -18,9 +18,9 @@ 
 $adminmenu = [];
 
 $adminmenu[] = [
-    'title' => _MI_SOBJECT_HOME,
-    'link'  => 'admin/index.php',
-    'icon'  => $pathIcon32 . '/home.png',
+	'title' => _MI_SOBJECT_HOME,
+	'link'  => 'admin/index.php',
+	'icon'  => $pathIcon32 . '/home.png',
 ];
 
 //$adminmenu[] = [
@@ -30,76 +30,76 @@ 
 //];
 
 $adminmenu[] = [
-    'title' => _MI_SOBJECT_SENT_LINKS,
-    'link'  => 'admin/link.php',
-    'icon'  => $pathIcon32 . '/addlink.png',
+	'title' => _MI_SOBJECT_SENT_LINKS,
+	'link'  => 'admin/link.php',
+	'icon'  => $pathIcon32 . '/addlink.png',
 ];
 
 $adminmenu[] = [
-    'title' => _MI_SOBJECT_TAGS,
-    'link'  => 'admin/customtag.php',
-    'icon'  => $pathIcon32 . '/identity.png',
+	'title' => _MI_SOBJECT_TAGS,
+	'link'  => 'admin/customtag.php',
+	'icon'  => $pathIcon32 . '/identity.png',
 ];
 
 $adminmenu[] = [
-    'title' => _MI_SOBJECT_ADSENSES,
-    'link'  => 'admin/adsense.php',
-    'icon'  => $pathIcon32 . '/alert.png',
+	'title' => _MI_SOBJECT_ADSENSES,
+	'link'  => 'admin/adsense.php',
+	'icon'  => $pathIcon32 . '/alert.png',
 ];
 
 $adminmenu[] = [
-    'title' => _MI_SOBJECT_RATINGS,
-    'link'  => 'admin/rating.php',
-    'icon'  => $pathIcon32 . '/stats.png',
+	'title' => _MI_SOBJECT_RATINGS,
+	'link'  => 'admin/rating.php',
+	'icon'  => $pathIcon32 . '/stats.png',
 ];
 
 $adminmenu[] = [
-    'title' => _MI_SOBJECT_ABOUT,
-    'link'  => 'admin/about.php',
-    'icon'  => $pathIcon32 . '/about.png',
+	'title' => _MI_SOBJECT_ABOUT,
+	'link'  => 'admin/about.php',
+	'icon'  => $pathIcon32 . '/about.png',
 ];
 
 //---------------------------------
 
 if (!defined('SMARTOBJECT_ROOT_PATH')) {
-    require_once XOOPS_ROOT_PATH . '/modules/smartobject/include/functions.php';
+	require_once XOOPS_ROOT_PATH . '/modules/smartobject/include/functions.php';
 }
 
 $smartobjectConfig = smart_getModuleConfig('smartobject');
 
 if (isset($smartobjectConfig['enable_currencyman']) && true === $smartobjectConfig['enable_currencyman']) {
-    $adminmenu[] = [
-        'title' => _MI_SOBJECT_CURRENCIES,
-        'link'  => 'admin/currency.php',
-        'icon'  => $pathIcon32 . '/cash_stack.png',
-    ];
+	$adminmenu[] = [
+		'title' => _MI_SOBJECT_CURRENCIES,
+		'link'  => 'admin/currency.php',
+		'icon'  => $pathIcon32 . '/cash_stack.png',
+	];
 }
 
 global $xoopsModule;
 if (isset($xoopsModule)) {
-    $i = -1;
-
-    // --- for XCL ---
-    //  $headermenu[$i]['link'] = '../../system/admin.php?fct=preferences&op=showmod&mod=' . $xoopsModule->getVar('mid');
-    $mid = $xoopsModule->getVar('mid');
-    if (defined('XOOPS_CUBE_LEGACY')) {
-        $link_pref = XOOPS_URL . '/modules/legacy/admin/index.php?action=PreferenceEdit&confmod_id=' . $mid;
-    } else {
-        $link_pref = XOOPS_URL . '/modules/system/admin.php?fct=preferences&op=showmod&mod=' . $mid;
-    }
-    $headermenu[$i]['link'] = $link_pref;
-    // -----
-
-    // --- for XCL ---
-    //  $headermenu[$i]['link'] = XOOPS_URL . "/modules/system/admin.php?fct=modulesadmin&op=update&module=" . $xoopsModule->getVar('dirname');
-    $dirname = $xoopsModule->getVar('dirname');
-    if (defined('XOOPS_CUBE_LEGACY')) {
-        $link_module = XOOPS_URL . '/modules/legacy/admin/index.php?action=ModuleUpdate&dirname=' . $dirname;
-    } else {
-        $link_module = XOOPS_URL . '/modules/system/admin.php?fct=modulesadmin&op=update&module=' . $dirname;
-    }
-    $headermenu[$i]['link'] = $link_module;
-    // -----
-
-    ++$i;
+	$i = -1;
+
+	// --- for XCL ---
+	//  $headermenu[$i]['link'] = '../../system/admin.php?fct=preferences&op=showmod&mod=' . $xoopsModule->getVar('mid');
+	$mid = $xoopsModule->getVar('mid');
+	if (defined('XOOPS_CUBE_LEGACY')) {
+		$link_pref = XOOPS_URL . '/modules/legacy/admin/index.php?action=PreferenceEdit&confmod_id=' . $mid;
+	} else {
+		$link_pref = XOOPS_URL . '/modules/system/admin.php?fct=preferences&op=showmod&mod=' . $mid;
+	}
+	$headermenu[$i]['link'] = $link_pref;
+	// -----
+
+	// --- for XCL ---
+	//  $headermenu[$i]['link'] = XOOPS_URL . "/modules/system/admin.php?fct=modulesadmin&op=update&module=" . $xoopsModule->getVar('dirname');
+	$dirname = $xoopsModule->getVar('dirname');
+	if (defined('XOOPS_CUBE_LEGACY')) {
+		$link_module = XOOPS_URL . '/modules/legacy/admin/index.php?action=ModuleUpdate&dirname=' . $dirname;
+	} else {
+		$link_module = XOOPS_URL . '/modules/system/admin.php?fct=modulesadmin&op=update&module=' . $dirname;
+	}
+	$headermenu[$i]['link'] = $link_module;
+	// -----
+
+	++$i;
 }

Spacing +12 added lines, -12 removed lines

@@ -20,7 +20,7 @@ 
 $adminmenu[] = [
     'title' => _MI_SOBJECT_HOME,
     'link'  => 'admin/index.php',
-    'icon'  => $pathIcon32 . '/home.png',
+    'icon'  => $pathIcon32.'/home.png',
 ];
 
 //$adminmenu[] = [
@@ -32,37 +32,37 @@ 
 $adminmenu[] = [
     'title' => _MI_SOBJECT_SENT_LINKS,
     'link'  => 'admin/link.php',
-    'icon'  => $pathIcon32 . '/addlink.png',
+    'icon'  => $pathIcon32.'/addlink.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_SOBJECT_TAGS,
     'link'  => 'admin/customtag.php',
-    'icon'  => $pathIcon32 . '/identity.png',
+    'icon'  => $pathIcon32.'/identity.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_SOBJECT_ADSENSES,
     'link'  => 'admin/adsense.php',
-    'icon'  => $pathIcon32 . '/alert.png',
+    'icon'  => $pathIcon32.'/alert.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_SOBJECT_RATINGS,
     'link'  => 'admin/rating.php',
-    'icon'  => $pathIcon32 . '/stats.png',
+    'icon'  => $pathIcon32.'/stats.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_SOBJECT_ABOUT,
     'link'  => 'admin/about.php',
-    'icon'  => $pathIcon32 . '/about.png',
+    'icon'  => $pathIcon32.'/about.png',
 ];
 
 //---------------------------------
 
 if (!defined('SMARTOBJECT_ROOT_PATH')) {
-    require_once XOOPS_ROOT_PATH . '/modules/smartobject/include/functions.php';
+    require_once XOOPS_ROOT_PATH.'/modules/smartobject/include/functions.php';
 }
 
 $smartobjectConfig = smart_getModuleConfig('smartobject');
@@ -71,7 +71,7 @@ 
     $adminmenu[] = [
         'title' => _MI_SOBJECT_CURRENCIES,
         'link'  => 'admin/currency.php',
-        'icon'  => $pathIcon32 . '/cash_stack.png',
+        'icon'  => $pathIcon32.'/cash_stack.png',
     ];
 }
 
@@ -83,9 +83,9 @@ 
     //  $headermenu[$i]['link'] = '../../system/admin.php?fct=preferences&op=showmod&mod=' . $xoopsModule->getVar('mid');
     $mid = $xoopsModule->getVar('mid');
     if (defined('XOOPS_CUBE_LEGACY')) {
-        $link_pref = XOOPS_URL . '/modules/legacy/admin/index.php?action=PreferenceEdit&confmod_id=' . $mid;
+        $link_pref = XOOPS_URL.'/modules/legacy/admin/index.php?action=PreferenceEdit&confmod_id='.$mid;
     } else {
-        $link_pref = XOOPS_URL . '/modules/system/admin.php?fct=preferences&op=showmod&mod=' . $mid;
+        $link_pref = XOOPS_URL.'/modules/system/admin.php?fct=preferences&op=showmod&mod='.$mid;
     }
     $headermenu[$i]['link'] = $link_pref;
     // -----
@@ -94,9 +94,9 @@ 
     //  $headermenu[$i]['link'] = XOOPS_URL . "/modules/system/admin.php?fct=modulesadmin&op=update&module=" . $xoopsModule->getVar('dirname');
     $dirname = $xoopsModule->getVar('dirname');
     if (defined('XOOPS_CUBE_LEGACY')) {
-        $link_module = XOOPS_URL . '/modules/legacy/admin/index.php?action=ModuleUpdate&dirname=' . $dirname;
+        $link_module = XOOPS_URL.'/modules/legacy/admin/index.php?action=ModuleUpdate&dirname='.$dirname;
     } else {
-        $link_module = XOOPS_URL . '/modules/system/admin.php?fct=modulesadmin&op=update&module=' . $dirname;
+        $link_module = XOOPS_URL.'/modules/system/admin.php?fct=modulesadmin&op=update&module='.$dirname;
     }
     $headermenu[$i]['link'] = $link_module;
     // -----

sendlink.php

Indentation +76 added lines, -76 removed lines

@@ -28,82 +28,82 @@
 $op = isset($_POST['op']) ? $_POST['op'] : '';
 
 switch ($op) {
-    case 'sendlink':
-
-        require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
-        $controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectLinkHandler);
-
-        $linkObj = $controller->storeSmartObject();
-        if ($linkObj->hasError()) {
-            /**
-             * @todo inform user and propose to close the window if a problem occured when saving the link
-             */
-        }
-
-        $xoopsMailer = xoops_getMailer();
-        $xoopsMailer->useMail();
-        $xoopsMailer->setTemplateDir('language/' . $xoopsConfig['language'] . '/mail_template');
-
-        $xoopsMailer->setTemplate('sendlink.tpl');
-        $xoopsMailer->assign('X_SITENAME', $xoopsConfig['sitename']);
-        $xoopsMailer->assign('TO_NAME', $linkObj->getVar('to_name'));
-        $xoopsMailer->assign('FROM_NAME', $linkObj->getVar('from_name'));
-        $xoopsMailer->assign('SITEURL', XOOPS_URL . '/');
-        $xoopsMailer->assign('ADMINMAIL', $xoopsConfig['adminmail']);
-        $xoopsMailer->assign('MESSAGE', $_POST['body']);
-        $xoopsMailer->setToEmails($linkObj->getVar('to_email'));
-        $xoopsMailer->setFromEmail($linkObj->getVar('from_email'));
-        $xoopsMailer->setFromName($xoopsConfig['sitename']);
-        $xoopsMailer->setSubject(sprintf(_CO_SOBJECT_SUBJECT_DEFAULT, $myts->oopsStripSlashesGPC($xoopsConfig['sitename'])));
-
-        if (!$xoopsMailer->send(true)) {
-            $xoopsTpl->assign('send_error', sprintf(_CO_SOBJECT_SEND_ERROR, $xoopsConfig['adminmail']) . '<br>' . $xoopsMailer->getErrors(true));
-        } else {
-            $xoopsTpl->assign('send_success', _CO_SOBJECT_SEND_SUCCESS);
-        }
-
-        break;
-
-    default:
-        if (isset($_GET['mid'])) {
-            $mid = $_GET['mid'];
-        } else {
-            /**
-             * @todo close the window if no mid is passed as GET
-             */
-        }
-
-        $hModule = xoops_getHandler('module');
-        $module  = $hModule->get($mid);
-        $linkObj->setVar('mid', $module->getVar('mid'));
-        $linkObj->setVar('mid_name', $module->getVar('name'));
-
-        if (isset($_GET['link'])) {
-            $link = $_GET['link'];
-        } else {
-            /**
-             * @todo close the window if no link is passed as GET
-             */
-        }
-        $linkObj->setVar('link', $link);
-
-        if (is_object($xoopsUser)) {
-            $linkObj->setVar('from_uid', $xoopsUser->getVar('uid'));
-            $linkObj->setVar('from_name', '' !== $xoopsUser->getVar('name') ? $xoopsUser->getVar('name') : $xoopsUser->getVar('uname'));
-            $linkObj->setVar('from_email', $xoopsUser->getVar('email'));
-        }
-
-        $linkObj->setVar('subject', sprintf(_CO_SOBJECT_SUBJECT_DEFAULT, $xoopsConfig['sitename']));
-        $linkObj->setVar('body', sprintf(_CO_SOBJECT_BODY_DEFAULT, $xoopsConfig['sitename'], $link));
-        $linkObj->setVar('date', time());
-        $linkObj->hideFieldFromForm(['from_uid', 'to_uid', 'link', 'mid', 'mid_name']);
-
-        $form = $linkObj->getForm(_CO_SOBJECT_SEND_LINK_FORM, 'sendlink', false, _SEND, 'javascript:window.close();');
-
-        $form->assign($xoopsTpl);
-
-        $xoopsTpl->assign('showform', true);
-        break;
+	case 'sendlink':
+
+		require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
+		$controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectLinkHandler);
+
+		$linkObj = $controller->storeSmartObject();
+		if ($linkObj->hasError()) {
+			/**
+			 * @todo inform user and propose to close the window if a problem occured when saving the link
+			 */
+		}
+
+		$xoopsMailer = xoops_getMailer();
+		$xoopsMailer->useMail();
+		$xoopsMailer->setTemplateDir('language/' . $xoopsConfig['language'] . '/mail_template');
+
+		$xoopsMailer->setTemplate('sendlink.tpl');
+		$xoopsMailer->assign('X_SITENAME', $xoopsConfig['sitename']);
+		$xoopsMailer->assign('TO_NAME', $linkObj->getVar('to_name'));
+		$xoopsMailer->assign('FROM_NAME', $linkObj->getVar('from_name'));
+		$xoopsMailer->assign('SITEURL', XOOPS_URL . '/');
+		$xoopsMailer->assign('ADMINMAIL', $xoopsConfig['adminmail']);
+		$xoopsMailer->assign('MESSAGE', $_POST['body']);
+		$xoopsMailer->setToEmails($linkObj->getVar('to_email'));
+		$xoopsMailer->setFromEmail($linkObj->getVar('from_email'));
+		$xoopsMailer->setFromName($xoopsConfig['sitename']);
+		$xoopsMailer->setSubject(sprintf(_CO_SOBJECT_SUBJECT_DEFAULT, $myts->oopsStripSlashesGPC($xoopsConfig['sitename'])));
+
+		if (!$xoopsMailer->send(true)) {
+			$xoopsTpl->assign('send_error', sprintf(_CO_SOBJECT_SEND_ERROR, $xoopsConfig['adminmail']) . '<br>' . $xoopsMailer->getErrors(true));
+		} else {
+			$xoopsTpl->assign('send_success', _CO_SOBJECT_SEND_SUCCESS);
+		}
+
+		break;
+
+	default:
+		if (isset($_GET['mid'])) {
+			$mid = $_GET['mid'];
+		} else {
+			/**
+			 * @todo close the window if no mid is passed as GET
+			 */
+		}
+
+		$hModule = xoops_getHandler('module');
+		$module  = $hModule->get($mid);
+		$linkObj->setVar('mid', $module->getVar('mid'));
+		$linkObj->setVar('mid_name', $module->getVar('name'));
+
+		if (isset($_GET['link'])) {
+			$link = $_GET['link'];
+		} else {
+			/**
+			 * @todo close the window if no link is passed as GET
+			 */
+		}
+		$linkObj->setVar('link', $link);
+
+		if (is_object($xoopsUser)) {
+			$linkObj->setVar('from_uid', $xoopsUser->getVar('uid'));
+			$linkObj->setVar('from_name', '' !== $xoopsUser->getVar('name') ? $xoopsUser->getVar('name') : $xoopsUser->getVar('uname'));
+			$linkObj->setVar('from_email', $xoopsUser->getVar('email'));
+		}
+
+		$linkObj->setVar('subject', sprintf(_CO_SOBJECT_SUBJECT_DEFAULT, $xoopsConfig['sitename']));
+		$linkObj->setVar('body', sprintf(_CO_SOBJECT_BODY_DEFAULT, $xoopsConfig['sitename'], $link));
+		$linkObj->setVar('date', time());
+		$linkObj->hideFieldFromForm(['from_uid', 'to_uid', 'link', 'mid', 'mid_name']);
+
+		$form = $linkObj->getForm(_CO_SOBJECT_SEND_LINK_FORM, 'sendlink', false, _SEND, 'javascript:window.close();');
+
+		$form->assign($xoopsTpl);
+
+		$xoopsTpl->assign('showform', true);
+		break;
 }
 
 $xoopsTpl->display('db:smartobject_sendlink.tpl');

Spacing +9 added lines, -9 removed lines

@@ -9,17 +9,17 @@ 
 
 use XoopsModules\Smartobject\SmartObjectController;
 
-require_once __DIR__ . '/header.php';
-require_once SMARTOBJECT_ROOT_PATH . 'class/smartloader.php';
-require_once SMARTOBJECT_ROOT_PATH . 'class/smartobjectlink.php';
-require_once XOOPS_ROOT_PATH . '/class/template.php';
+require_once __DIR__.'/header.php';
+require_once SMARTOBJECT_ROOT_PATH.'class/smartloader.php';
+require_once SMARTOBJECT_ROOT_PATH.'class/smartobjectlink.php';
+require_once XOOPS_ROOT_PATH.'/class/template.php';
 
 $xoopsTpl                = new \XoopsTpl();
 $myts                    = \MyTextSanitizer::getInstance();
 $xoopsConfig['sitename'] = $myts->displayTarea($xoopsConfig['sitename']);
 
 xoops_header(false);
-echo smart_get_css_link(SMARTOBJECT_URL . 'assets/css/module.css');
+echo smart_get_css_link(SMARTOBJECT_URL.'assets/css/module.css');
 echo '</head><body>';
 
 $smartobjectLinkHandler = xoops_getModuleHandler('link', 'smartobject');
@@ -30,7 +30,7 @@ 
 switch ($op) {
     case 'sendlink':
 
-        require_once XOOPS_ROOT_PATH . '/modules/smartobject/class/smartobjectcontroller.php';
+        require_once XOOPS_ROOT_PATH.'/modules/smartobject/class/smartobjectcontroller.php';
         $controller = new XoopsModules\Smartobject\SmartObjectController($smartobjectLinkHandler);
 
         $linkObj = $controller->storeSmartObject();
@@ -42,13 +42,13 @@ 
 
         $xoopsMailer = xoops_getMailer();
         $xoopsMailer->useMail();
-        $xoopsMailer->setTemplateDir('language/' . $xoopsConfig['language'] . '/mail_template');
+        $xoopsMailer->setTemplateDir('language/'.$xoopsConfig['language'].'/mail_template');
 
         $xoopsMailer->setTemplate('sendlink.tpl');
         $xoopsMailer->assign('X_SITENAME', $xoopsConfig['sitename']);
         $xoopsMailer->assign('TO_NAME', $linkObj->getVar('to_name'));
         $xoopsMailer->assign('FROM_NAME', $linkObj->getVar('from_name'));
-        $xoopsMailer->assign('SITEURL', XOOPS_URL . '/');
+        $xoopsMailer->assign('SITEURL', XOOPS_URL.'/');
         $xoopsMailer->assign('ADMINMAIL', $xoopsConfig['adminmail']);
         $xoopsMailer->assign('MESSAGE', $_POST['body']);
         $xoopsMailer->setToEmails($linkObj->getVar('to_email'));
@@ -57,7 +57,7 @@ 
         $xoopsMailer->setSubject(sprintf(_CO_SOBJECT_SUBJECT_DEFAULT, $myts->oopsStripSlashesGPC($xoopsConfig['sitename'])));
 
         if (!$xoopsMailer->send(true)) {
-            $xoopsTpl->assign('send_error', sprintf(_CO_SOBJECT_SEND_ERROR, $xoopsConfig['adminmail']) . '<br>' . $xoopsMailer->getErrors(true));
+            $xoopsTpl->assign('send_error', sprintf(_CO_SOBJECT_SEND_ERROR, $xoopsConfig['adminmail']).'<br>'.$xoopsMailer->getErrors(true));
         } else {
             $xoopsTpl->assign('send_success', _CO_SOBJECT_SEND_SUCCESS);
         }

preloads/autoloader.php

Indentation +19 added lines, -19 removed lines

@@ -4,29 +4,29 @@
  * @see http://www.php-fig.org/psr/psr-4/examples/
  */
 spl_autoload_register(function ($class) {
-    // project-specific namespace prefix
-    $prefix = 'XoopsModules\\' . ucfirst(basename(dirname(__DIR__)));
+	// project-specific namespace prefix
+	$prefix = 'XoopsModules\\' . ucfirst(basename(dirname(__DIR__)));
 
-    // base directory for the namespace prefix
-    $base_dir = __DIR__ . '/../class/';
+	// base directory for the namespace prefix
+	$base_dir = __DIR__ . '/../class/';
 
-    // does the class use the namespace prefix?
-    $len      = strlen($prefix);
+	// does the class use the namespace prefix?
+	$len      = strlen($prefix);
 
-    if (0 !== strncmp($prefix, $class, $len)) {
-        return;
-    }
+	if (0 !== strncmp($prefix, $class, $len)) {
+		return;
+	}
 
-    // get the relative class name
-    $relative_class = substr($class, $len);
+	// get the relative class name
+	$relative_class = substr($class, $len);
 
-    // replace the namespace prefix with the base directory, replace namespace
-    // separators with directory separators in the relative class name, append
-    // with .php
-    $file           = $base_dir . str_replace('\\', '/', $relative_class) . '.php';
+	// replace the namespace prefix with the base directory, replace namespace
+	// separators with directory separators in the relative class name, append
+	// with .php
+	$file           = $base_dir . str_replace('\\', '/', $relative_class) . '.php';
 
-    // if the file exists, require it
-    if (file_exists($file)) {
-        require $file;
-    }
+	// if the file exists, require it
+	if (file_exists($file)) {
+		require $file;
+	}
 });

Spacing +4 added lines, -4 removed lines

@@ -3,12 +3,12 @@ 
 /**
  * @see http://www.php-fig.org/psr/psr-4/examples/
  */
-spl_autoload_register(function ($class) {
+spl_autoload_register(function($class) {
     // project-specific namespace prefix
-    $prefix = 'XoopsModules\\' . ucfirst(basename(dirname(__DIR__)));
+    $prefix = 'XoopsModules\\'.ucfirst(basename(dirname(__DIR__)));
 
     // base directory for the namespace prefix
-    $base_dir = __DIR__ . '/../class/';
+    $base_dir = __DIR__.'/../class/';
 
     // does the class use the namespace prefix?
     $len      = strlen($prefix);
@@ -23,7 +23,7 @@ 
     // replace the namespace prefix with the base directory, replace namespace
     // separators with directory separators in the relative class name, append
     // with .php
-    $file           = $base_dir . str_replace('\\', '/', $relative_class) . '.php';
+    $file = $base_dir.str_replace('\\', '/', $relative_class).'.php';
 
     // if the file exists, require it
     if (file_exists($file)) {

preloads/core.php

Indentation +8 added lines, -8 removed lines

@@ -22,12 +22,12 @@
  */
 class SmartobjectCorePreload extends \XoopsPreloadItem
 {
-    // to add PSR-4 autoloader
-    /**
-     * @param $args
-     */
-    public static function eventCoreIncludeCommonEnd($args)
-    {
-        include __DIR__ . '/autoloader.php';
-    }
+	// to add PSR-4 autoloader
+	/**
+	 * @param $args
+	 */
+	public static function eventCoreIncludeCommonEnd($args)
+	{
+		include __DIR__ . '/autoloader.php';
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -28,6 +28,6 @@
      */
     public static function eventCoreIncludeCommonEnd($args)
     {
-        include __DIR__ . '/autoloader.php';
+        include __DIR__.'/autoloader.php';
     }
 }

class/SmartobjectRating.php

Indentation +119 added lines, -119 removed lines

@@ -30,123 +30,123 @@
  */
 class SmartobjectRating extends Smartobject\BaseSmartObject
 {
-    public $_modulePlugin = false;
-
-    /**
-     * SmartobjectRating constructor.
-     */
-    public function __construct()
-    {
-        $this->quickInitVar('ratingid', XOBJ_DTYPE_INT, true);
-        $this->quickInitVar('dirname', XOBJ_DTYPE_TXTBOX, true, _CO_SOBJECT_RATING_DIRNAME);
-        $this->quickInitVar('item', XOBJ_DTYPE_TXTBOX, true, _CO_SOBJECT_RATING_ITEM);
-        $this->quickInitVar('itemid', XOBJ_DTYPE_INT, true, _CO_SOBJECT_RATING_ITEMID);
-        $this->quickInitVar('uid', XOBJ_DTYPE_INT, true, _CO_SOBJECT_RATING_UID);
-        $this->quickInitVar('date', XOBJ_DTYPE_LTIME, true, _CO_SOBJECT_RATING_DATE);
-        $this->quickInitVar('rate', XOBJ_DTYPE_INT, true, _CO_SOBJECT_RATING_RATE);
-
-        $this->initNonPersistableVar('name', XOBJ_DTYPE_TXTBOX, 'user', _CO_SOBJECT_RATING_NAME);
-
-        $this->setControl('dirname', [
-            'handler'  => 'rating',
-            'method'   => 'getModuleList',
-            'onSelect' => 'submit'
-        ]);
-
-        $this->setControl('item', [
-            'object' => &$this,
-            'method' => 'getItemList'
-        ]);
-
-        $this->setControl('uid', 'user');
-
-        $this->setControl('rate', [
-            'handler' => 'rating',
-            'method'  => 'getRateList'
-        ]);
-    }
-
-    /**
-     * @param  string $key
-     * @param  string $format
-     * @return mixed
-     */
-    public function getVar($key, $format = 's')
-    {
-        if ('s' === $format && in_array($key, ['name', 'dirname'])) {
-            //            return call_user_func(array($this, $key));
-            return $this->{$key}();
-        }
-
-        return parent::getVar($key, $format);
-    }
-
-    /**
-     * @return string
-     */
-    public function name()
-    {
-        $ret = smart_getLinkedUnameFromId($this->getVar('uid', 'e'), true, []);
-
-        return $ret;
-    }
-
-    /**
-     * @return mixed
-     */
-    public function dirname()
-    {
-        global $smartobjectRatingHandler;
-        $moduleArray = $smartobjectRatingHandler->getModuleList();
-
-        return $moduleArray[$this->getVar('dirname', 'n')];
-    }
-
-    /**
-     * @return mixed
-     */
-    public function getItemList()
-    {
-        $plugin = $this->getModulePlugin();
-
-        return $plugin->getItemList();
-    }
-
-    /**
-     * @return string
-     */
-    public function getItemValue()
-    {
-        $moduleUrl      = XOOPS_URL . '/modules/' . $this->getVar('dirname', 'n') . '/';
-        $plugin         = $this->getModulePlugin();
-        $pluginItemInfo = $plugin->getItemInfo($this->getVar('item'));
-        if (!$pluginItemInfo) {
-            return '';
-        }
-        $itemPath = sprintf($pluginItemInfo['url'], $this->getVar('itemid'));
-        $ret      = '<a href="' . $moduleUrl . $itemPath . '">' . $pluginItemInfo['caption'] . '</a>';
-
-        return $ret;
-    }
-
-    /**
-     * @return mixed
-     */
-    public function getRateValue()
-    {
-        return $this->getVar('rate');
-    }
-
-    /**
-     * @return bool
-     */
-    public function getModulePlugin()
-    {
-        if (!$this->_modulePlugin) {
-            global $smartobjectRatingHandler;
-            $this->_modulePlugin = $smartobjectRatingHandler->pluginsObject->getPlugin($this->getVar('dirname', 'n'));
-        }
-
-        return $this->_modulePlugin;
-    }
+	public $_modulePlugin = false;
+
+	/**
+	 * SmartobjectRating constructor.
+	 */
+	public function __construct()
+	{
+		$this->quickInitVar('ratingid', XOBJ_DTYPE_INT, true);
+		$this->quickInitVar('dirname', XOBJ_DTYPE_TXTBOX, true, _CO_SOBJECT_RATING_DIRNAME);
+		$this->quickInitVar('item', XOBJ_DTYPE_TXTBOX, true, _CO_SOBJECT_RATING_ITEM);
+		$this->quickInitVar('itemid', XOBJ_DTYPE_INT, true, _CO_SOBJECT_RATING_ITEMID);
+		$this->quickInitVar('uid', XOBJ_DTYPE_INT, true, _CO_SOBJECT_RATING_UID);
+		$this->quickInitVar('date', XOBJ_DTYPE_LTIME, true, _CO_SOBJECT_RATING_DATE);
+		$this->quickInitVar('rate', XOBJ_DTYPE_INT, true, _CO_SOBJECT_RATING_RATE);
+
+		$this->initNonPersistableVar('name', XOBJ_DTYPE_TXTBOX, 'user', _CO_SOBJECT_RATING_NAME);
+
+		$this->setControl('dirname', [
+			'handler'  => 'rating',
+			'method'   => 'getModuleList',
+			'onSelect' => 'submit'
+		]);
+
+		$this->setControl('item', [
+			'object' => &$this,
+			'method' => 'getItemList'
+		]);
+
+		$this->setControl('uid', 'user');
+
+		$this->setControl('rate', [
+			'handler' => 'rating',
+			'method'  => 'getRateList'
+		]);
+	}
+
+	/**
+	 * @param  string $key
+	 * @param  string $format
+	 * @return mixed
+	 */
+	public function getVar($key, $format = 's')
+	{
+		if ('s' === $format && in_array($key, ['name', 'dirname'])) {
+			//            return call_user_func(array($this, $key));
+			return $this->{$key}();
+		}
+
+		return parent::getVar($key, $format);
+	}
+
+	/**
+	 * @return string
+	 */
+	public function name()
+	{
+		$ret = smart_getLinkedUnameFromId($this->getVar('uid', 'e'), true, []);
+
+		return $ret;
+	}
+
+	/**
+	 * @return mixed
+	 */
+	public function dirname()
+	{
+		global $smartobjectRatingHandler;
+		$moduleArray = $smartobjectRatingHandler->getModuleList();
+
+		return $moduleArray[$this->getVar('dirname', 'n')];
+	}
+
+	/**
+	 * @return mixed
+	 */
+	public function getItemList()
+	{
+		$plugin = $this->getModulePlugin();
+
+		return $plugin->getItemList();
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getItemValue()
+	{
+		$moduleUrl      = XOOPS_URL . '/modules/' . $this->getVar('dirname', 'n') . '/';
+		$plugin         = $this->getModulePlugin();
+		$pluginItemInfo = $plugin->getItemInfo($this->getVar('item'));
+		if (!$pluginItemInfo) {
+			return '';
+		}
+		$itemPath = sprintf($pluginItemInfo['url'], $this->getVar('itemid'));
+		$ret      = '<a href="' . $moduleUrl . $itemPath . '">' . $pluginItemInfo['caption'] . '</a>';
+
+		return $ret;
+	}
+
+	/**
+	 * @return mixed
+	 */
+	public function getRateValue()
+	{
+		return $this->getVar('rate');
+	}
+
+	/**
+	 * @return bool
+	 */
+	public function getModulePlugin()
+	{
+		if (!$this->_modulePlugin) {
+			global $smartobjectRatingHandler;
+			$this->_modulePlugin = $smartobjectRatingHandler->pluginsObject->getPlugin($this->getVar('dirname', 'n'));
+		}
+
+		return $this->_modulePlugin;
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -117,14 +117,14 @@
      */
     public function getItemValue()
     {
-        $moduleUrl      = XOOPS_URL . '/modules/' . $this->getVar('dirname', 'n') . '/';
+        $moduleUrl      = XOOPS_URL.'/modules/'.$this->getVar('dirname', 'n').'/';
         $plugin         = $this->getModulePlugin();
         $pluginItemInfo = $plugin->getItemInfo($this->getVar('item'));
         if (!$pluginItemInfo) {
             return '';
         }
         $itemPath = sprintf($pluginItemInfo['url'], $this->getVar('itemid'));
-        $ret      = '<a href="' . $moduleUrl . $itemPath . '">' . $pluginItemInfo['caption'] . '</a>';
+        $ret      = '<a href="'.$moduleUrl.$itemPath.'">'.$pluginItemInfo['caption'].'</a>';
 
         return $ret;
     }

class/SmartObjectTree.php

Indentation +13 added lines, -13 removed lines

@@ -30,17 +30,17 @@
  */
 class SmartObjectTree extends \XoopsObjectTree
 {
-    public function _initialize()
-    {
-        foreach (array_keys($this->_objects) as $i) {
-            $key1                         = $this->_objects[$i]->getVar($this->_myId);
-            $this->tree[$key1]['obj']     = $this->_objects[$i];
-            $key2                         = $this->_objects[$i]->getVar($this->_parentId, 'e');
-            $this->tree[$key1]['parent']  = $key2;
-            $this->tree[$key2]['child'][] = $key1;
-            if (isset($this->_rootId)) {
-                $this->tree[$key1]['root'] = $this->_objects[$i]->getVar($this->_rootId);
-            }
-        }
-    }
+	public function _initialize()
+	{
+		foreach (array_keys($this->_objects) as $i) {
+			$key1                         = $this->_objects[$i]->getVar($this->_myId);
+			$this->tree[$key1]['obj']     = $this->_objects[$i];
+			$key2                         = $this->_objects[$i]->getVar($this->_parentId, 'e');
+			$this->tree[$key1]['parent']  = $key2;
+			$this->tree[$key2]['child'][] = $key1;
+			if (isset($this->_rootId)) {
+				$this->tree[$key1]['root'] = $this->_objects[$i]->getVar($this->_rootId);
+			}
+		}
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 
 // defined('XOOPS_ROOT_PATH') || die('Restricted access');
 
-require_once XOOPS_ROOT_PATH . '/class/tree.php';
+require_once XOOPS_ROOT_PATH.'/class/tree.php';
 
 /**
  * Class smartobjecttree

class/SmartPersistableMlObjectHandler.php

Indentation +49 added lines, -49 removed lines

@@ -21,60 +21,60 @@
  */
 class SmartPersistableMlObjectHandler extends Smartobject\SmartPersistableObjectHandler
 {
-    /**
-     * @param  null|\CriteriaElement  $criteria
-     * @param  bool $id_as_key
-     * @param  bool $as_object
-     * @param  bool $debug
-     * @param  bool $language
-     * @return array
-     */
-    public function getObjects(
-        \CriteriaElement $criteria = null,
-        $id_as_key = false,
-        $as_object = true,
-        $debug = false,
-        $language = false
-    ) {
-        // Create the first part of the SQL query to join the "_text" table
-        $sql = 'SELECT * FROM ' . $this->table . ' AS ' . $this->_itemname . ' INNER JOIN ' . $this->table . '_text AS ' . $this->_itemname . '_text ON ' . $this->_itemname . '.' . $this->keyName . '=' . $this->_itemname . '_text.' . $this->keyName;
+	/**
+	 * @param  null|\CriteriaElement  $criteria
+	 * @param  bool $id_as_key
+	 * @param  bool $as_object
+	 * @param  bool $debug
+	 * @param  bool $language
+	 * @return array
+	 */
+	public function getObjects(
+		\CriteriaElement $criteria = null,
+		$id_as_key = false,
+		$as_object = true,
+		$debug = false,
+		$language = false
+	) {
+		// Create the first part of the SQL query to join the "_text" table
+		$sql = 'SELECT * FROM ' . $this->table . ' AS ' . $this->_itemname . ' INNER JOIN ' . $this->table . '_text AS ' . $this->_itemname . '_text ON ' . $this->_itemname . '.' . $this->keyName . '=' . $this->_itemname . '_text.' . $this->keyName;
 
-        if ($language) {
-            // If a language was specified, then let's create a WHERE clause to only return the objects associated with this language
+		if ($language) {
+			// If a language was specified, then let's create a WHERE clause to only return the objects associated with this language
 
-            // if no criteria was previously created, let's create it
-            if (!$criteria) {
-                $criteria = new \CriteriaCompo();
-            }
-            $criteria->add(new \Criteria('language', $language));
+			// if no criteria was previously created, let's create it
+			if (!$criteria) {
+				$criteria = new \CriteriaCompo();
+			}
+			$criteria->add(new \Criteria('language', $language));
 
-            return parent::getObjects($criteria, $id_as_key, $as_object, $debug, $sql);
-        }
+			return parent::getObjects($criteria, $id_as_key, $as_object, $debug, $sql);
+		}
 
-        return parent::getObjects($criteria, $id_as_key, $as_object, $debug, $sql);
-    }
+		return parent::getObjects($criteria, $id_as_key, $as_object, $debug, $sql);
+	}
 
-    /**
-     * @param  mixed $id
-     * @param  bool  $language
-     * @param  bool  $as_object
-     * @param  bool  $debug
-     * @return mixed
-     */
-    public function &get($id, $language = false, $as_object = true, $debug = false)
-    {
-        if (!$language) {
-            return parent::get($id, $as_object, $debug);
-        } else {
-            $criteria = new \CriteriaCompo();
-            $criteria->add(new \Criteria('language', $language));
+	/**
+	 * @param  mixed $id
+	 * @param  bool  $language
+	 * @param  bool  $as_object
+	 * @param  bool  $debug
+	 * @return mixed
+	 */
+	public function &get($id, $language = false, $as_object = true, $debug = false)
+	{
+		if (!$language) {
+			return parent::get($id, $as_object, $debug);
+		} else {
+			$criteria = new \CriteriaCompo();
+			$criteria->add(new \Criteria('language', $language));
 
-            return parent::get($id, $as_object, $debug, $criteria);
-        }
-    }
+			return parent::get($id, $as_object, $debug, $criteria);
+		}
+	}
 
-    public function changeTableNameForML()
-    {
-        $this->table = $this->db->prefix($this->_moduleName . '_' . $this->_itemname . '_text');
-    }
+	public function changeTableNameForML()
+	{
+		$this->table = $this->db->prefix($this->_moduleName . '_' . $this->_itemname . '_text');
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
         $language = false
     ) {
         // Create the first part of the SQL query to join the "_text" table
-        $sql = 'SELECT * FROM ' . $this->table . ' AS ' . $this->_itemname . ' INNER JOIN ' . $this->table . '_text AS ' . $this->_itemname . '_text ON ' . $this->_itemname . '.' . $this->keyName . '=' . $this->_itemname . '_text.' . $this->keyName;
+        $sql = 'SELECT * FROM '.$this->table.' AS '.$this->_itemname.' INNER JOIN '.$this->table.'_text AS '.$this->_itemname.'_text ON '.$this->_itemname.'.'.$this->keyName.'='.$this->_itemname.'_text.'.$this->keyName;
 
         if ($language) {
             // If a language was specified, then let's create a WHERE clause to only return the objects associated with this language
@@ -75,6 +75,6 @@ 
 
     public function changeTableNameForML()
     {
-        $this->table = $this->db->prefix($this->_moduleName . '_' . $this->_itemname . '_text');
+        $this->table = $this->db->prefix($this->_moduleName.'_'.$this->_itemname.'_text');
     }
 }