ggoffy / wggithub

admin/menu.php

Indentation +30 added lines, -30 removed lines

@@ -27,52 +27,52 @@
 $sysPathIcon32 = $moduleInfo->getInfo('sysicons32');
 
 $adminmenu[] = [
-	'title' => _MI_WGGITHUB_ADMENU1,
-	'link' => 'admin/index.php',
-	'icon' => $sysPathIcon32.'/dashboard.png',
+    'title' => _MI_WGGITHUB_ADMENU1,
+    'link' => 'admin/index.php',
+    'icon' => $sysPathIcon32.'/dashboard.png',
 ];
 $adminmenu[] = [
-	'title' => _MI_WGGITHUB_ADMENU2,
-	'link' => 'admin/settings.php',
-	'icon' => 'assets/icons/32/blank.gif',
+    'title' => _MI_WGGITHUB_ADMENU2,
+    'link' => 'admin/settings.php',
+    'icon' => 'assets/icons/32/blank.gif',
 ];
 $adminmenu[] = [
-	'title' => _MI_WGGITHUB_ADMENU3,
-	'link' => 'admin/directories.php',
-	'icon' => 'assets/icons/32/blank.gif',
+    'title' => _MI_WGGITHUB_ADMENU3,
+    'link' => 'admin/directories.php',
+    'icon' => 'assets/icons/32/blank.gif',
 ];
 $adminmenu[] = [
-	'title' => _MI_WGGITHUB_ADMENU4,
-	'link' => 'admin/logs.php',
-	'icon' => 'assets/icons/32/blank.gif',
+    'title' => _MI_WGGITHUB_ADMENU4,
+    'link' => 'admin/logs.php',
+    'icon' => 'assets/icons/32/blank.gif',
 ];
 $adminmenu[] = [
-	'title' => _MI_WGGITHUB_ADMENU5,
-	'link' => 'admin/repositories.php',
-	'icon' => 'assets/icons/32/blank.gif',
+    'title' => _MI_WGGITHUB_ADMENU5,
+    'link' => 'admin/repositories.php',
+    'icon' => 'assets/icons/32/blank.gif',
 ];
 $adminmenu[] = [
-	'title' => _MI_WGGITHUB_ADMENU6,
-	'link' => 'admin/readmes.php',
-	'icon' => 'assets/icons/32/addlink.png',
+    'title' => _MI_WGGITHUB_ADMENU6,
+    'link' => 'admin/readmes.php',
+    'icon' => 'assets/icons/32/addlink.png',
 ];
 $adminmenu[] = [
-	'title' => _MI_WGGITHUB_ADMENU7,
-	'link' => 'admin/releases.php',
-	'icon' => 'assets/icons/32/blank.gif',
+    'title' => _MI_WGGITHUB_ADMENU7,
+    'link' => 'admin/releases.php',
+    'icon' => 'assets/icons/32/blank.gif',
 ];
 $adminmenu[] = [
-	'title' => _MI_WGGITHUB_ADMENU8,
-	'link' => 'admin/permissions.php',
-	'icon' => $sysPathIcon32.'/permissions.png',
+    'title' => _MI_WGGITHUB_ADMENU8,
+    'link' => 'admin/permissions.php',
+    'icon' => $sysPathIcon32.'/permissions.png',
 ];
 $adminmenu[] = [
-	'title' => _MI_WGGITHUB_ADMENU9,
-	'link' => 'admin/feedback.php',
-	'icon' => $sysPathIcon32.'/mail_foward.png',
+    'title' => _MI_WGGITHUB_ADMENU9,
+    'link' => 'admin/feedback.php',
+    'icon' => $sysPathIcon32.'/mail_foward.png',
 ];
 $adminmenu[] = [
-	'title' => _MI_WGGITHUB_ABOUT,
-	'link' => 'admin/about.php',
-	'icon' => $sysPathIcon32.'/about.png',
+    'title' => _MI_WGGITHUB_ABOUT,
+    'link' => 'admin/about.php',
+    'icon' => $sysPathIcon32.'/about.png',
 ];

admin/github-api.php

Spacing +3 added lines, -3 removed lines

@@ -30,7 +30,7 @@ 
 use XoopsModules\Wggithub;
 use XoopsModules\Wggithub\Github;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 // It recovered the value of argument op in URL$
 
 
@@ -41,8 +41,8 @@ 
 
 $count = 0;
 foreach ($data as $key => $repo) {
-    echo "<br>key:" . $key . ' repo:' . $repo['name'];
+    echo "<br>key:".$key.' repo:'.$repo['name'];
     $count++;
-    if ($count>5) {break;}
+    if ($count > 5) {break; }
 }
 echo "<br>OK 2";

admin/logs.php

Indentation +101 added lines, -101 removed lines

@@ -31,107 +31,107 @@
 // Request log_id
 $logId = Request::getInt('log_id');
 switch ($op) {
-	case 'list':
-	default:
-		// Define Stylesheet
-		$GLOBALS['xoTheme']->addStylesheet($style, null);
-		$start = Request::getInt('start', 0);
-		$limit = Request::getInt('limit', $helper->getConfig('adminpager'));
-		$templateMain = 'wggithub_admin_logs.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_LOG, 'logs.php?op=new', 'add');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		$logsCount = $logsHandler->getCountLogs();
-		$logsAll = $logsHandler->getAllLogs($start, $limit);
-		$GLOBALS['xoopsTpl']->assign('logs_count', $logsCount);
-		$GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
-		$GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
-		// Table view logs
-		if ($logsCount > 0) {
-			foreach (\array_keys($logsAll) as $i) {
-				$log = $logsAll[$i]->getValuesLogs();
-				$GLOBALS['xoopsTpl']->append('logs_list', $log);
-				unset($log);
-			}
-			// Display Navigation
-			if ($logsCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-			}
-		} else {
-			$GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_LOGS);
-		}
-		break;
-	case 'new':
-		$templateMain = 'wggithub_admin_logs.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_LOGS_LIST, 'logs.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Form Create
-		$logsObj = $logsHandler->create();
-		$form = $logsObj->getFormLogs();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'save':
-		// Security Check
-		if (!$GLOBALS['xoopsSecurity']->check()) {
-			\redirect_header('logs.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-		}
-		if ($logId > 0) {
-			$logsObj = $logsHandler->get($logId);
-		} else {
-			$logsObj = $logsHandler->create();
-		}
-		// Set Vars
+    case 'list':
+    default:
+        // Define Stylesheet
+        $GLOBALS['xoTheme']->addStylesheet($style, null);
+        $start = Request::getInt('start', 0);
+        $limit = Request::getInt('limit', $helper->getConfig('adminpager'));
+        $templateMain = 'wggithub_admin_logs.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_LOG, 'logs.php?op=new', 'add');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        $logsCount = $logsHandler->getCountLogs();
+        $logsAll = $logsHandler->getAllLogs($start, $limit);
+        $GLOBALS['xoopsTpl']->assign('logs_count', $logsCount);
+        $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
+        $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
+        // Table view logs
+        if ($logsCount > 0) {
+            foreach (\array_keys($logsAll) as $i) {
+                $log = $logsAll[$i]->getValuesLogs();
+                $GLOBALS['xoopsTpl']->append('logs_list', $log);
+                unset($log);
+            }
+            // Display Navigation
+            if ($logsCount > $limit) {
+                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+            }
+        } else {
+            $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_LOGS);
+        }
+        break;
+    case 'new':
+        $templateMain = 'wggithub_admin_logs.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_LOGS_LIST, 'logs.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Form Create
+        $logsObj = $logsHandler->create();
+        $form = $logsObj->getFormLogs();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'save':
+        // Security Check
+        if (!$GLOBALS['xoopsSecurity']->check()) {
+            \redirect_header('logs.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+        }
+        if ($logId > 0) {
+            $logsObj = $logsHandler->get($logId);
+        } else {
+            $logsObj = $logsHandler->create();
+        }
+        // Set Vars
         $logsObj->setVar('log_type', Request::getInt('log_type', 0));
-		$logsObj->setVar('log_details', Request::getString('log_details', ''));
-		$logsObj->setVar('log_result', Request::getString('log_result', ''));
-		$logDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('log_datecreated'));
-		$logsObj->setVar('log_datecreated', $logDatecreatedObj->getTimestamp());
-		$logsObj->setVar('log_submitter', Request::getInt('log_submitter', 0));
-		// Insert Data
-		if ($logsHandler->insert($logsObj)) {
-			\redirect_header('logs.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
-		}
-		// Get Form
-		$GLOBALS['xoopsTpl']->assign('error', $logsObj->getHtmlErrors());
-		$form = $logsObj->getFormLogs();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'edit':
-		$templateMain = 'wggithub_admin_logs.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_LOG, 'logs.php?op=new', 'add');
-		$adminObject->addItemButton(_AM_WGGITHUB_LOGS_LIST, 'logs.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Get Form
-		$logsObj = $logsHandler->get($logId);
-		$form = $logsObj->getFormLogs();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'delete':
-		$templateMain = 'wggithub_admin_logs.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
-		$logsObj = $logsHandler->get($logId);
-		$reqRequest = $logsObj->getVar('log_details');
-		if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
-			if (!$GLOBALS['xoopsSecurity']->check()) {
-				\redirect_header('logs.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
-			}
-			if ($logsHandler->delete($logsObj)) {
-				\redirect_header('logs.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
-			} else {
-				$GLOBALS['xoopsTpl']->assign('error', $logsObj->getHtmlErrors());
-			}
-		} else {
-			$xoopsconfirm = new Common\XoopsConfirm(
-				['ok' => 1, 'log_id' => $logId, 'op' => 'delete'],
-				$_SERVER['REQUEST_URI'],
-				\sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $logsObj->getVar('log_details')));
-			$form = $xoopsconfirm->getFormXoopsConfirm();
-			$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		}
-		break;
+        $logsObj->setVar('log_details', Request::getString('log_details', ''));
+        $logsObj->setVar('log_result', Request::getString('log_result', ''));
+        $logDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('log_datecreated'));
+        $logsObj->setVar('log_datecreated', $logDatecreatedObj->getTimestamp());
+        $logsObj->setVar('log_submitter', Request::getInt('log_submitter', 0));
+        // Insert Data
+        if ($logsHandler->insert($logsObj)) {
+            \redirect_header('logs.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
+        }
+        // Get Form
+        $GLOBALS['xoopsTpl']->assign('error', $logsObj->getHtmlErrors());
+        $form = $logsObj->getFormLogs();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'edit':
+        $templateMain = 'wggithub_admin_logs.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_LOG, 'logs.php?op=new', 'add');
+        $adminObject->addItemButton(_AM_WGGITHUB_LOGS_LIST, 'logs.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Get Form
+        $logsObj = $logsHandler->get($logId);
+        $form = $logsObj->getFormLogs();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'delete':
+        $templateMain = 'wggithub_admin_logs.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
+        $logsObj = $logsHandler->get($logId);
+        $reqRequest = $logsObj->getVar('log_details');
+        if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
+            if (!$GLOBALS['xoopsSecurity']->check()) {
+                \redirect_header('logs.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
+            }
+            if ($logsHandler->delete($logsObj)) {
+                \redirect_header('logs.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
+            } else {
+                $GLOBALS['xoopsTpl']->assign('error', $logsObj->getHtmlErrors());
+            }
+        } else {
+            $xoopsconfirm = new Common\XoopsConfirm(
+                ['ok' => 1, 'log_id' => $logId, 'op' => 'delete'],
+                $_SERVER['REQUEST_URI'],
+                \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $logsObj->getVar('log_details')));
+            $form = $xoopsconfirm->getFormXoopsConfirm();
+            $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        }
+        break;
 }
 require __DIR__ . '/footer.php';

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
 use XoopsModules\Wggithub\Constants;
 use XoopsModules\Wggithub\Common;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 // It recovered the value of argument op in URL$
 $op = Request::getCmd('op', 'list');
 // Request log_id
@@ -55,8 +55,8 @@ 
 			}
 			// Display Navigation
 			if ($logsCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+				include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+				$pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit='.$limit);
 				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
 			}
 		} else {
@@ -134,4 +134,4 @@ 
 		}
 		break;
 }
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

admin/index.php

Spacing +12 added lines, -12 removed lines

@@ -23,8 +23,8 @@ 
 
 use XoopsModules\Wggithub\Common;
 
-include_once \dirname(__DIR__) . '/preloads/autoloader.php';
-require __DIR__ . '/header.php';
+include_once \dirname(__DIR__).'/preloads/autoloader.php';
+require __DIR__.'/header.php';
 
 // Template Index
 $templateMain = 'wggithub_admin_index.tpl';
@@ -40,12 +40,12 @@ 
 // InfoBox Statistics
 $adminObject->addInfoBox(_AM_WGGITHUB_STATISTICS);
 // Info elements
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_SETTINGS . '</label>', $countSettings));
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_DIRECTORIES . '</label>', $countDirectories));
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_LOGS . '</label>', $countLogs));
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_REPOSITORIES . '</label>', $countRepositories));
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_READMES . '</label>', $countReadmes));
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_RELEASES . '</label>', $countReleases));
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_SETTINGS.'</label>', $countSettings));
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_DIRECTORIES.'</label>', $countDirectories));
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_LOGS.'</label>', $countLogs));
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_REPOSITORIES.'</label>', $countRepositories));
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_READMES.'</label>', $countReadmes));
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_RELEASES.'</label>', $countReleases));
 
 // Upload Folders
 $configurator = new Common\Configurator();
@@ -65,12 +65,12 @@ 
 // Test Data
 if ($helper->getConfig('displaySampleButton')) {
 	\xoops_loadLanguage('admin/modulesadmin', 'system');
-	include_once \dirname(__DIR__) . '/testdata/index.php';
-	$adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
-	$adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
+	include_once \dirname(__DIR__).'/testdata/index.php';
+	$adminObject->addItemButton(\constant('CO_'.$moduleDirNameUpper.'_ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
+	$adminObject->addItemButton(\constant('CO_'.$moduleDirNameUpper.'_SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
 //	$adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_EXPORT_SCHEMA'), '__DIR__ . /../../testdata/index.php?op=exportschema', 'add');
 	$adminObject->displayButton('left');
 }
 $GLOBALS['xoopsTpl']->assign('index', $adminObject->displayIndex());
 // End Test Data
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

include/update.php

Spacing +16 added lines, -16 removed lines

@@ -39,7 +39,7 @@ 
     $ret = wggithub_check_db($module);
 
     //check upload directory
-	include_once __DIR__ . '/install.php';
+	include_once __DIR__.'/install.php';
     $ret = xoops_module_install_wggithub($module);
 
     $errors = $module->getErrors();
@@ -61,7 +61,7 @@ 
 {
     global $xoopsDB;
     $result = $xoopsDB->query(
-        'SELECT t1.tpl_id FROM ' . $xoopsDB->prefix('tplfile') . ' t1, ' . $xoopsDB->prefix('tplfile') . ' t2 WHERE t1.tpl_refid = t2.tpl_refid AND t1.tpl_module = t2.tpl_module AND t1.tpl_tplset=t2.tpl_tplset AND t1.tpl_file = t2.tpl_file AND t1.tpl_type = t2.tpl_type AND t1.tpl_id > t2.tpl_id'
+        'SELECT t1.tpl_id FROM '.$xoopsDB->prefix('tplfile').' t1, '.$xoopsDB->prefix('tplfile').' t2 WHERE t1.tpl_refid = t2.tpl_refid AND t1.tpl_module = t2.tpl_module AND t1.tpl_tplset=t2.tpl_tplset AND t1.tpl_file = t2.tpl_file AND t1.tpl_type = t2.tpl_type AND t1.tpl_id > t2.tpl_id'
     );
     $tplids = [];
     while (false !== (list($tplid) = $xoopsDB->fetchRow($result))) {
@@ -69,7 +69,7 @@ 
     }
     if (\count($tplids) > 0) {
         $tplfileHandler  = \xoops_getHandler('tplfile');
-        $duplicate_files = $tplfileHandler->getObjects(new \Criteria('tpl_id', '(' . \implode(',', $tplids) . ')', 'IN'));
+        $duplicate_files = $tplfileHandler->getObjects(new \Criteria('tpl_id', '('.\implode(',', $tplids).')', 'IN'));
 
         if (\count($duplicate_files) > 0) {
             foreach (\array_keys($duplicate_files) as $i) {
@@ -77,9 +77,9 @@ 
             }
         }
     }
-    $sql = 'SHOW INDEX FROM ' . $xoopsDB->prefix('tplfile') . " WHERE KEY_NAME = 'tpl_refid_module_set_file_type'";
+    $sql = 'SHOW INDEX FROM '.$xoopsDB->prefix('tplfile')." WHERE KEY_NAME = 'tpl_refid_module_set_file_type'";
     if (!$result = $xoopsDB->queryF($sql)) {
-        xoops_error($xoopsDB->error() . '<br>' . $sql);
+        xoops_error($xoopsDB->error().'<br>'.$sql);
 
         return false;
     }
@@ -92,9 +92,9 @@ 
 
         return true;
     }
-    $sql = 'ALTER TABLE ' . $xoopsDB->prefix('tplfile') . ' ADD UNIQUE tpl_refid_module_set_file_type ( tpl_refid, tpl_module, tpl_tplset, tpl_file, tpl_type )';
+    $sql = 'ALTER TABLE '.$xoopsDB->prefix('tplfile').' ADD UNIQUE tpl_refid_module_set_file_type ( tpl_refid, tpl_module, tpl_tplset, tpl_file, tpl_type )';
     if (!$result = $xoopsDB->queryF($sql)) {
-        xoops_error($xoopsDB->error() . '<br>' . $sql);
+        xoops_error($xoopsDB->error().'<br>'.$sql);
         $module->setErrors("'tpl_refid_module_set_file_type' unique index is not added to 'tplfile' table. Warning: do not use XOOPS until you add this unique index.");
 
         return false;
@@ -118,12 +118,12 @@ 
     // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
     $field   = 'repo_release';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }
@@ -132,12 +132,12 @@ 
     // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
     $field   = 'repo_prerelease';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }
@@ -145,12 +145,12 @@ 
     // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
     $field   = 'repo_readme';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }
@@ -159,12 +159,12 @@ 
 // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_directories');
     $field   = 'dir_filterrelease';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `dir_online`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }
@@ -186,7 +186,7 @@ 
               PRIMARY KEY (`log_id`)
                 ) ENGINE=InnoDB;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when creating table '$table'.");
             $ret = false;
         }

index.php

Spacing +17 added lines, -17 removed lines

@@ -28,15 +28,15 @@ 
     Github\GithubClient
 };
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 $GLOBALS['xoopsOption']['template_main'] = 'wggithub_index.tpl';
-include_once \XOOPS_ROOT_PATH . '/header.php';
+include_once \XOOPS_ROOT_PATH.'/header.php';
 
 // Permissions
 $permGlobalView = $permissionsHandler->getPermGlobalView();
 if (!$permGlobalView) {
     $GLOBALS['xoopsTpl']->assign('error', _NOPERM);
-    require __DIR__ . '/footer.php';
+    require __DIR__.'/footer.php';
 }
 $permGlobalRead   = $permissionsHandler->getPermGlobalRead();
 $permReadmeUpdate = $permissionsHandler->getPermReadmeUpdate();
@@ -50,7 +50,7 @@ 
 
 // Define Stylesheet
 $GLOBALS['xoTheme']->addStylesheet($style, null);
-$GLOBALS['xoTheme']->addStylesheet(WGGITHUB_URL . '/assets/css/tabs.css', null);
+$GLOBALS['xoTheme']->addStylesheet(WGGITHUB_URL.'/assets/css/tabs.css', null);
 $keywords = [];
 // 
 $GLOBALS['xoopsTpl']->assign('xoops_icons32_url', XOOPS_ICONS32_URL);
@@ -120,13 +120,13 @@ 
             foreach (\array_keys($directoriesAll) as $i) {
                 $directories[$i] = $directoriesAll[$i]->getValuesDirectories();
                 $dirName = $directoriesAll[$i]->getVar('dir_name');
-                $dirFilterRelease = (bool)$directoriesAll[$i]->getVar('dir_filterrelease');
+                $dirFilterRelease = (bool) $directoriesAll[$i]->getVar('dir_filterrelease');
                 $repos = [];
                 $crRepositories = new \CriteriaCompo();
                 $crRepositories->add(new \Criteria('repo_user', $dirName));
                 $repositoriesCountTotal = $repositoriesHandler->getCount($crRepositories);
                 if ('any' === $filterRelease && $dirFilterRelease) {
-                    $crRepositories->add(new \Criteria('repo_prerelease', 1) );
+                    $crRepositories->add(new \Criteria('repo_prerelease', 1));
                     $crRepositories->add(new \Criteria('repo_release', 1), 'OR');
                 }
                 if ('final' === $filterRelease && $dirFilterRelease) {
@@ -181,9 +181,9 @@ 
                 }
                 $directories[$i]['repos'] = $repos;
                 $directories[$i]['previousRepos'] = $start > 0;
-                $directories[$i]['previousOp'] = '&start=' . ($start - $limit) . '&limit=' . $limit . '&release=' . $filterRelease . '&sortby=' . $filterSortby;
+                $directories[$i]['previousOp'] = '&start='.($start - $limit).'&limit='.$limit.'&release='.$filterRelease.'&sortby='.$filterSortby;
                 $directories[$i]['nextRepos'] = ($repositoriesCount - $start) > $limit;
-                $directories[$i]['nextOp'] = '&start=' . ($start + $limit) . '&limit=' . $limit . '&release=' . $filterRelease . '&sortby=' . $filterSortby;
+                $directories[$i]['nextOp'] = '&start='.($start + $limit).'&limit='.$limit.'&release='.$filterRelease.'&sortby='.$filterSortby;
                 $GLOBALS['xoopsTpl']->assign('start', $start);
                 $GLOBALS['xoopsTpl']->assign('limit', $limit);
                 $GLOBALS['xoopsTpl']->assign('menu', $menu);
@@ -194,8 +194,8 @@ 
             unset($crDirectories, $directories);
             // Display Navigation
             if ($directoriesCount > $limit) {
-                require_once \XOOPS_ROOT_PATH . '/class/pagenav.php';
-                $pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                require_once \XOOPS_ROOT_PATH.'/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit='.$limit);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
             }
             $GLOBALS['xoopsTpl']->assign('lang_thereare', \sprintf(\_MA_WGGITHUB_INDEX_THEREARE, $directoriesCount));
@@ -208,11 +208,11 @@ 
         // Permissions
         if (!$permGlobalRead) {
             $GLOBALS['xoopsTpl']->assign('error', \_NOPERM);
-            require __DIR__ . '/footer.php';
+            require __DIR__.'/footer.php';
         }
         $dirName = Request::getString('dir_name', '');
         $result = executeUpdate($dirName);
-        $redir = 'index.php?op=list_afterupdate&start=' . $start . '&limit=' . $limit . '&release=' . $filterRelease . '&sortby=' . $filterSortby;
+        $redir = 'index.php?op=list_afterupdate&start='.$start.'&limit='.$limit.'&release='.$filterRelease.'&sortby='.$filterSortby;
         if ($result) {
             \redirect_header($redir, 2, \_MA_WGGITHUB_READGH_SUCCESS);
         } else {
@@ -224,13 +224,13 @@ 
         // Permissions
         if (!$permReadmeUpdate) {
             $GLOBALS['xoopsTpl']->assign('error', \_NOPERM);
-            require __DIR__ . '/footer.php';
+            require __DIR__.'/footer.php';
         }
         $repoId = Request::getInt('repo_id', 0);
         $repoUser  = Request::getString('repo_user', 'none');
         $repoName  = Request::getString('repo_name', 'none');
         $res = $helper->getHandler('Readmes')->updateReadmes($repoId, $repoUser, $repoName);
-        $redir = 'index.php?op=list_afterupdate&start=' . $start . '&limit=' . $limit . '&release=' . $filterRelease . '&sortby=' . $filterSortby;
+        $redir = 'index.php?op=list_afterupdate&start='.$start.'&limit='.$limit.'&release='.$filterRelease.'&sortby='.$filterSortby;
         if ($result) {
             \redirect_header($redir, 2, \_MA_WGGITHUB_READGH_SUCCESS);
         } else {
@@ -243,14 +243,14 @@ 
 // Breadcrumbs
 $xoBreadcrumbs[] = ['title' => \_MA_WGGITHUB_INDEX];
 // Keywords
-wggithubMetaKeywords($helper->getConfig('keywords') . ', ' . \implode(',', $keywords));
+wggithubMetaKeywords($helper->getConfig('keywords').', '.\implode(',', $keywords));
 unset($keywords);
 // Description
 wggithubMetaDescription(\_MA_WGGITHUB_INDEX_DESC);
 $GLOBALS['xoopsTpl']->assign('xoops_mpageurl', WGGITHUB_URL.'/index.php');
 $GLOBALS['xoopsTpl']->assign('xoops_icons32_url', XOOPS_ICONS32_URL);
 $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';
 
 
 /**
@@ -258,7 +258,7 @@ 
  * @param string $dirName
  * @return bool
  */
-function executeUpdate($dirName = ''){
+function executeUpdate($dirName = '') {
 
     $githubClient = GithubClient::getInstance();
     $helper = Helper::getInstance();

xoops_version.php

Indentation +224 added lines, -224 removed lines

@@ -25,208 +25,208 @@ 
 $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
 // ------------------- Informations ------------------- //
 $modversion = [
-	'name'                => _MI_WGGITHUB_NAME,
-	'version'             => 1.01,
-	'description'         => _MI_WGGITHUB_DESC,
-	'author'              => 'Goffy - XOOPS Development Team',
-	'author_mail'         => 'goffy@wedega.com',
-	'author_website_url'  => 'https://wedega.com',
-	'author_website_name' => 'XOOPS on Wedega',
-	'credits'             => 'Goffy - XOOPS Development Team',
-	'license'             => 'GPL 2.0 or later',
-	'license_url'         => 'http://www.gnu.org/licenses/gpl-3.0.en.html',
-	'help'                => 'page=help',
-	'release_info'        => 'release_info',
-	'release_file'        => XOOPS_URL . '/modules/wggithub/docs/release_info file',
-	'release_date'        => '2020/11/21',
-	'manual'              => 'link to manual file',
-	'manual_file'         => XOOPS_URL . '/modules/wggithub/docs/install.txt',
-	'min_php'             => '7.2',
-	'min_xoops'           => '2.5.10',
-	'min_admin'           => '1.2',
-	'min_db'              => ['mysql' => '5.6', 'mysqli' => '5.6'],
-	'image'               => 'assets/images/logoModule.png',
-	'dirname'             => \basename(__DIR__),
-	'dirmoduleadmin'      => 'Frameworks/moduleclasses/moduleadmin',
-	'sysicons16'          => '../../Frameworks/moduleclasses/icons/16',
-	'sysicons32'          => '../../Frameworks/moduleclasses/icons/32',
-	'modicons16'          => 'assets/icons/16',
-	'modicons32'          => 'assets/icons/32',
-	'demo_site_url'       => 'https://xoops.org',
-	'demo_site_name'      => 'XOOPS Demo Site',
-	'support_url'         => 'https://xoops.org/modules/newbb',
-	'support_name'        => 'Support Forum',
-	'module_website_url'  => 'www.xoops.org',
-	'module_website_name' => 'XOOPS Project',
-	'release'             => '2017-12-02',
-	'module_status'       => 'Beta 1',
-	'system_menu'         => 1,
-	'hasAdmin'            => 1,
-	'hasMain'             => 1,
-	'adminindex'          => 'admin/index.php',
-	'adminmenu'           => 'admin/menu.php',
-	'onInstall'           => 'include/install.php',
-	'onUninstall'         => 'include/uninstall.php',
-	'onUpdate'            => 'include/update.php',
+    'name'                => _MI_WGGITHUB_NAME,
+    'version'             => 1.01,
+    'description'         => _MI_WGGITHUB_DESC,
+    'author'              => 'Goffy - XOOPS Development Team',
+    'author_mail'         => 'goffy@wedega.com',
+    'author_website_url'  => 'https://wedega.com',
+    'author_website_name' => 'XOOPS on Wedega',
+    'credits'             => 'Goffy - XOOPS Development Team',
+    'license'             => 'GPL 2.0 or later',
+    'license_url'         => 'http://www.gnu.org/licenses/gpl-3.0.en.html',
+    'help'                => 'page=help',
+    'release_info'        => 'release_info',
+    'release_file'        => XOOPS_URL . '/modules/wggithub/docs/release_info file',
+    'release_date'        => '2020/11/21',
+    'manual'              => 'link to manual file',
+    'manual_file'         => XOOPS_URL . '/modules/wggithub/docs/install.txt',
+    'min_php'             => '7.2',
+    'min_xoops'           => '2.5.10',
+    'min_admin'           => '1.2',
+    'min_db'              => ['mysql' => '5.6', 'mysqli' => '5.6'],
+    'image'               => 'assets/images/logoModule.png',
+    'dirname'             => \basename(__DIR__),
+    'dirmoduleadmin'      => 'Frameworks/moduleclasses/moduleadmin',
+    'sysicons16'          => '../../Frameworks/moduleclasses/icons/16',
+    'sysicons32'          => '../../Frameworks/moduleclasses/icons/32',
+    'modicons16'          => 'assets/icons/16',
+    'modicons32'          => 'assets/icons/32',
+    'demo_site_url'       => 'https://xoops.org',
+    'demo_site_name'      => 'XOOPS Demo Site',
+    'support_url'         => 'https://xoops.org/modules/newbb',
+    'support_name'        => 'Support Forum',
+    'module_website_url'  => 'www.xoops.org',
+    'module_website_name' => 'XOOPS Project',
+    'release'             => '2017-12-02',
+    'module_status'       => 'Beta 1',
+    'system_menu'         => 1,
+    'hasAdmin'            => 1,
+    'hasMain'             => 1,
+    'adminindex'          => 'admin/index.php',
+    'adminmenu'           => 'admin/menu.php',
+    'onInstall'           => 'include/install.php',
+    'onUninstall'         => 'include/uninstall.php',
+    'onUpdate'            => 'include/update.php',
 ];
 // ------------------- Templates ------------------- //
 $modversion['templates'] = [
-	// Admin templates
-	['file' => 'wggithub_admin_about.tpl', 'description' => '', 'type' => 'admin'],
-	['file' => 'wggithub_admin_header.tpl', 'description' => '', 'type' => 'admin'],
-	['file' => 'wggithub_admin_index.tpl', 'description' => '', 'type' => 'admin'],
-	['file' => 'wggithub_admin_settings.tpl', 'description' => '', 'type' => 'admin'],
-	['file' => 'wggithub_admin_repositories.tpl', 'description' => '', 'type' => 'admin'],
-	['file' => 'wggithub_admin_directories.tpl', 'description' => '', 'type' => 'admin'],
-	['file' => 'wggithub_admin_logs.tpl', 'description' => '', 'type' => 'admin'],
-	['file' => 'wggithub_admin_readmes.tpl', 'description' => '', 'type' => 'admin'],
+    // Admin templates
+    ['file' => 'wggithub_admin_about.tpl', 'description' => '', 'type' => 'admin'],
+    ['file' => 'wggithub_admin_header.tpl', 'description' => '', 'type' => 'admin'],
+    ['file' => 'wggithub_admin_index.tpl', 'description' => '', 'type' => 'admin'],
+    ['file' => 'wggithub_admin_settings.tpl', 'description' => '', 'type' => 'admin'],
+    ['file' => 'wggithub_admin_repositories.tpl', 'description' => '', 'type' => 'admin'],
+    ['file' => 'wggithub_admin_directories.tpl', 'description' => '', 'type' => 'admin'],
+    ['file' => 'wggithub_admin_logs.tpl', 'description' => '', 'type' => 'admin'],
+    ['file' => 'wggithub_admin_readmes.tpl', 'description' => '', 'type' => 'admin'],
     ['file' => 'wggithub_admin_releases.tpl', 'description' => '', 'type' => 'admin'],
-	['file' => 'wggithub_admin_permissions.tpl', 'description' => '', 'type' => 'admin'],
-	['file' => 'wggithub_admin_footer.tpl', 'description' => '', 'type' => 'admin'],
-	// User templates
-	['file' => 'wggithub_header.tpl', 'description' => ''],
-	['file' => 'wggithub_index.tpl', 'description' => ''],
-	['file' => 'wggithub_repositories.tpl', 'description' => ''],
-	['file' => 'wggithub_repositories_list.tpl', 'description' => ''],
-	['file' => 'wggithub_repositories_item.tpl', 'description' => ''],
-	['file' => 'wggithub_directories.tpl', 'description' => ''],
-	['file' => 'wggithub_directories_list.tpl', 'description' => ''],
-	['file' => 'wggithub_directories_item.tpl', 'description' => ''],
-	['file' => 'wggithub_readmes.tpl', 'description' => ''],
-	['file' => 'wggithub_readmes_list.tpl', 'description' => ''],
-	['file' => 'wggithub_readmes_item.tpl', 'description' => ''],
-	['file' => 'wggithub_breadcrumbs.tpl', 'description' => ''],
-	['file' => 'wggithub_search.tpl', 'description' => ''],
-	['file' => 'wggithub_footer.tpl', 'description' => ''],
+    ['file' => 'wggithub_admin_permissions.tpl', 'description' => '', 'type' => 'admin'],
+    ['file' => 'wggithub_admin_footer.tpl', 'description' => '', 'type' => 'admin'],
+    // User templates
+    ['file' => 'wggithub_header.tpl', 'description' => ''],
+    ['file' => 'wggithub_index.tpl', 'description' => ''],
+    ['file' => 'wggithub_repositories.tpl', 'description' => ''],
+    ['file' => 'wggithub_repositories_list.tpl', 'description' => ''],
+    ['file' => 'wggithub_repositories_item.tpl', 'description' => ''],
+    ['file' => 'wggithub_directories.tpl', 'description' => ''],
+    ['file' => 'wggithub_directories_list.tpl', 'description' => ''],
+    ['file' => 'wggithub_directories_item.tpl', 'description' => ''],
+    ['file' => 'wggithub_readmes.tpl', 'description' => ''],
+    ['file' => 'wggithub_readmes_list.tpl', 'description' => ''],
+    ['file' => 'wggithub_readmes_item.tpl', 'description' => ''],
+    ['file' => 'wggithub_breadcrumbs.tpl', 'description' => ''],
+    ['file' => 'wggithub_search.tpl', 'description' => ''],
+    ['file' => 'wggithub_footer.tpl', 'description' => ''],
 ];
 // ------------------- Mysql ------------------- //
 $modversion['sqlfile']['mysql'] = 'sql/mysql.sql';
 // Tables
 $modversion['tables'] = [
-	'wggithub_settings',
-	'wggithub_repositories',
-	'wggithub_directories',
-	'wggithub_logs',
-	'wggithub_readmes',
+    'wggithub_settings',
+    'wggithub_repositories',
+    'wggithub_directories',
+    'wggithub_logs',
+    'wggithub_readmes',
     'wggithub_releases',
 ];
 // ------------------- Search ------------------- //
 $modversion['hasSearch'] = 1;
 $modversion['search'] = [
-	'file' => 'include/search.inc.php',
-	'func' => 'wggithub_search',
+    'file' => 'include/search.inc.php',
+    'func' => 'wggithub_search',
 ];
 // ------------------- Blocks ------------------- //
 // Repositories last
 $modversion['blocks'][] = [
-	'file'        => 'repositories.php',
-	'name'        => _MI_WGGITHUB_REPOSITORIES_BLOCK_LAST,
-	'description' => _MI_WGGITHUB_REPOSITORIES_BLOCK_LAST_DESC,
-	'show_func'   => 'b_wggithub_repositories_show',
-	'edit_func'   => 'b_wggithub_repositories_edit',
-	'template'    => 'wggithub_block_repositories.tpl',
-	'options'     => 'last|5|25|0',
+    'file'        => 'repositories.php',
+    'name'        => _MI_WGGITHUB_REPOSITORIES_BLOCK_LAST,
+    'description' => _MI_WGGITHUB_REPOSITORIES_BLOCK_LAST_DESC,
+    'show_func'   => 'b_wggithub_repositories_show',
+    'edit_func'   => 'b_wggithub_repositories_edit',
+    'template'    => 'wggithub_block_repositories.tpl',
+    'options'     => 'last|5|25|0',
 ];
 // Repositories new
 $modversion['blocks'][] = [
-	'file'        => 'repositories.php',
-	'name'        => _MI_WGGITHUB_REPOSITORIES_BLOCK_NEW,
-	'description' => _MI_WGGITHUB_REPOSITORIES_BLOCK_NEW_DESC,
-	'show_func'   => 'b_wggithub_repositories_show',
-	'edit_func'   => 'b_wggithub_repositories_edit',
-	'template'    => 'wggithub_block_repositories.tpl',
-	'options'     => 'new|5|25|0',
+    'file'        => 'repositories.php',
+    'name'        => _MI_WGGITHUB_REPOSITORIES_BLOCK_NEW,
+    'description' => _MI_WGGITHUB_REPOSITORIES_BLOCK_NEW_DESC,
+    'show_func'   => 'b_wggithub_repositories_show',
+    'edit_func'   => 'b_wggithub_repositories_edit',
+    'template'    => 'wggithub_block_repositories.tpl',
+    'options'     => 'new|5|25|0',
 ];
 // Repositories hits
 $modversion['blocks'][] = [
-	'file'        => 'repositories.php',
-	'name'        => _MI_WGGITHUB_REPOSITORIES_BLOCK_HITS,
-	'description' => _MI_WGGITHUB_REPOSITORIES_BLOCK_HITS_DESC,
-	'show_func'   => 'b_wggithub_repositories_show',
-	'edit_func'   => 'b_wggithub_repositories_edit',
-	'template'    => 'wggithub_block_repositories.tpl',
-	'options'     => 'hits|5|25|0',
+    'file'        => 'repositories.php',
+    'name'        => _MI_WGGITHUB_REPOSITORIES_BLOCK_HITS,
+    'description' => _MI_WGGITHUB_REPOSITORIES_BLOCK_HITS_DESC,
+    'show_func'   => 'b_wggithub_repositories_show',
+    'edit_func'   => 'b_wggithub_repositories_edit',
+    'template'    => 'wggithub_block_repositories.tpl',
+    'options'     => 'hits|5|25|0',
 ];
 // Repositories top
 $modversion['blocks'][] = [
-	'file'        => 'repositories.php',
-	'name'        => _MI_WGGITHUB_REPOSITORIES_BLOCK_TOP,
-	'description' => _MI_WGGITHUB_REPOSITORIES_BLOCK_TOP_DESC,
-	'show_func'   => 'b_wggithub_repositories_show',
-	'edit_func'   => 'b_wggithub_repositories_edit',
-	'template'    => 'wggithub_block_repositories.tpl',
-	'options'     => 'top|5|25|0',
+    'file'        => 'repositories.php',
+    'name'        => _MI_WGGITHUB_REPOSITORIES_BLOCK_TOP,
+    'description' => _MI_WGGITHUB_REPOSITORIES_BLOCK_TOP_DESC,
+    'show_func'   => 'b_wggithub_repositories_show',
+    'edit_func'   => 'b_wggithub_repositories_edit',
+    'template'    => 'wggithub_block_repositories.tpl',
+    'options'     => 'top|5|25|0',
 ];
 // Repositories random
 $modversion['blocks'][] = [
-	'file'        => 'repositories.php',
-	'name'        => _MI_WGGITHUB_REPOSITORIES_BLOCK_RANDOM,
-	'description' => _MI_WGGITHUB_REPOSITORIES_BLOCK_RANDOM_DESC,
-	'show_func'   => 'b_wggithub_repositories_show',
-	'edit_func'   => 'b_wggithub_repositories_edit',
-	'template'    => 'wggithub_block_repositories.tpl',
-	'options'     => 'random|5|25|0',
+    'file'        => 'repositories.php',
+    'name'        => _MI_WGGITHUB_REPOSITORIES_BLOCK_RANDOM,
+    'description' => _MI_WGGITHUB_REPOSITORIES_BLOCK_RANDOM_DESC,
+    'show_func'   => 'b_wggithub_repositories_show',
+    'edit_func'   => 'b_wggithub_repositories_edit',
+    'template'    => 'wggithub_block_repositories.tpl',
+    'options'     => 'random|5|25|0',
 ];
 // ------------------- Config ------------------- //
 // Editor Admin
 \xoops_load('xoopseditorhandler');
 $editorHandler = XoopsEditorHandler::getInstance();
 $modversion['config'][] = [
-	'name'        => 'editor_admin',
-	'title'       => '_MI_WGGITHUB_EDITOR_ADMIN',
-	'description' => '_MI_WGGITHUB_EDITOR_ADMIN_DESC',
-	'formtype'    => 'select',
-	'valuetype'   => 'text',
-	'default'     => 'dhtml',
-	'options'     => array_flip($editorHandler->getList()),
+    'name'        => 'editor_admin',
+    'title'       => '_MI_WGGITHUB_EDITOR_ADMIN',
+    'description' => '_MI_WGGITHUB_EDITOR_ADMIN_DESC',
+    'formtype'    => 'select',
+    'valuetype'   => 'text',
+    'default'     => 'dhtml',
+    'options'     => array_flip($editorHandler->getList()),
 ];
 // Editor User
 \xoops_load('xoopseditorhandler');
 $editorHandler = XoopsEditorHandler::getInstance();
 $modversion['config'][] = [
-	'name'        => 'editor_user',
-	'title'       => '_MI_WGGITHUB_EDITOR_USER',
-	'description' => '_MI_WGGITHUB_EDITOR_USER_DESC',
-	'formtype'    => 'select',
-	'valuetype'   => 'text',
-	'default'     => 'dhtml',
-	'options'     => array_flip($editorHandler->getList()),
+    'name'        => 'editor_user',
+    'title'       => '_MI_WGGITHUB_EDITOR_USER',
+    'description' => '_MI_WGGITHUB_EDITOR_USER_DESC',
+    'formtype'    => 'select',
+    'valuetype'   => 'text',
+    'default'     => 'dhtml',
+    'options'     => array_flip($editorHandler->getList()),
 ];
 // Editor : max characters admin area
 $modversion['config'][] = [
-	'name'        => 'editor_maxchar',
-	'title'       => '_MI_WGGITHUB_EDITOR_MAXCHAR',
-	'description' => '_MI_WGGITHUB_EDITOR_MAXCHAR_DESC',
-	'formtype'    => 'textbox',
-	'valuetype'   => 'int',
-	'default'     => 50,
+    'name'        => 'editor_maxchar',
+    'title'       => '_MI_WGGITHUB_EDITOR_MAXCHAR',
+    'description' => '_MI_WGGITHUB_EDITOR_MAXCHAR_DESC',
+    'formtype'    => 'textbox',
+    'valuetype'   => 'int',
+    'default'     => 50,
 ];
 // Get groups
 $memberHandler = \xoops_getHandler('member');
 $xoopsGroups  = $memberHandler->getGroupList();
 $groups = [];
 foreach ($xoopsGroups as $key => $group) {
-	$groups[$group]  = $key;
+    $groups[$group]  = $key;
 }
 // General access groups
 $modversion['config'][] = [
-	'name'        => 'groups',
-	'title'       => '_MI_WGGITHUB_GROUPS',
-	'description' => '_MI_WGGITHUB_GROUPS_DESC',
-	'formtype'    => 'select_multi',
-	'valuetype'   => 'array',
-	'default'     => $groups,
-	'options'     => $groups,
+    'name'        => 'groups',
+    'title'       => '_MI_WGGITHUB_GROUPS',
+    'description' => '_MI_WGGITHUB_GROUPS_DESC',
+    'formtype'    => 'select_multi',
+    'valuetype'   => 'array',
+    'default'     => $groups,
+    'options'     => $groups,
 ];
 // Upload groups
 $modversion['config'][] = [
-	'name'        => 'upload_groups',
-	'title'       => '_MI_WGGITHUB_UPLOAD_GROUPS',
-	'description' => '_MI_WGGITHUB_UPLOAD_GROUPS_DESC',
-	'formtype'    => 'select_multi',
-	'valuetype'   => 'array',
-	'default'     => $groups,
-	'options'     => $groups,
+    'name'        => 'upload_groups',
+    'title'       => '_MI_WGGITHUB_UPLOAD_GROUPS',
+    'description' => '_MI_WGGITHUB_UPLOAD_GROUPS_DESC',
+    'formtype'    => 'select_multi',
+    'valuetype'   => 'array',
+    'default'     => $groups,
+    'options'     => $groups,
 ];
 // Get Admin groups
 $crGroups = new \CriteriaCompo();
@@ -235,118 +235,118 @@ 
 $adminXoopsGroups  = $memberHandler->getGroupList($crGroups);
 $adminGroups = [];
 foreach ($adminXoopsGroups as $key => $adminGroup) {
-	$adminGroups[$adminGroup]  = $key;
+    $adminGroups[$adminGroup]  = $key;
 }
 $modversion['config'][] = [
-	'name'        => 'admin_groups',
-	'title'       => '_MI_WGGITHUB_ADMIN_GROUPS',
-	'description' => '_MI_WGGITHUB_ADMIN_GROUPS_DESC',
-	'formtype'    => 'select_multi',
-	'valuetype'   => 'array',
-	'default'     => $adminGroups,
-	'options'     => $adminGroups,
+    'name'        => 'admin_groups',
+    'title'       => '_MI_WGGITHUB_ADMIN_GROUPS',
+    'description' => '_MI_WGGITHUB_ADMIN_GROUPS_DESC',
+    'formtype'    => 'select_multi',
+    'valuetype'   => 'array',
+    'default'     => $adminGroups,
+    'options'     => $adminGroups,
 ];
 unset($crGroups);
 // Keywords
 $modversion['config'][] = [
-	'name'        => 'keywords',
-	'title'       => '_MI_WGGITHUB_KEYWORDS',
-	'description' => '_MI_WGGITHUB_KEYWORDS_DESC',
-	'formtype'    => 'textbox',
-	'valuetype'   => 'text',
-	'default'     => 'wggithub, settings, repositories, directories, requests, readmes',
+    'name'        => 'keywords',
+    'title'       => '_MI_WGGITHUB_KEYWORDS',
+    'description' => '_MI_WGGITHUB_KEYWORDS_DESC',
+    'formtype'    => 'textbox',
+    'valuetype'   => 'text',
+    'default'     => 'wggithub, settings, repositories, directories, requests, readmes',
 ];
 // Admin pager
 $modversion['config'][] = [
-	'name'        => 'adminpager',
-	'title'       => '_MI_WGGITHUB_ADMIN_PAGER',
-	'description' => '_MI_WGGITHUB_ADMIN_PAGER_DESC',
-	'formtype'    => 'textbox',
-	'valuetype'   => 'int',
-	'default'     => 25,
+    'name'        => 'adminpager',
+    'title'       => '_MI_WGGITHUB_ADMIN_PAGER',
+    'description' => '_MI_WGGITHUB_ADMIN_PAGER_DESC',
+    'formtype'    => 'textbox',
+    'valuetype'   => 'int',
+    'default'     => 25,
 ];
 // User pager
 $modversion['config'][] = [
-	'name'        => 'userpager',
-	'title'       => '_MI_WGGITHUB_USER_PAGER',
-	'description' => '_MI_WGGITHUB_USER_PAGER_DESC',
-	'formtype'    => 'textbox',
-	'valuetype'   => 'int',
-	'default'     => 25,
+    'name'        => 'userpager',
+    'title'       => '_MI_WGGITHUB_USER_PAGER',
+    'description' => '_MI_WGGITHUB_USER_PAGER_DESC',
+    'formtype'    => 'textbox',
+    'valuetype'   => 'int',
+    'default'     => 25,
 ];
 // Number column
 $modversion['config'][] = [
-	'name'        => 'numb_col',
-	'title'       => '_MI_WGGITHUB_NUMB_COL',
-	'description' => '_MI_WGGITHUB_NUMB_COL_DESC',
-	'formtype'    => 'select',
-	'valuetype'   => 'int',
-	'default'     => 1,
-	'options'     => [1 => '1', 2 => '2', 3 => '3', 4 => '4'],
+    'name'        => 'numb_col',
+    'title'       => '_MI_WGGITHUB_NUMB_COL',
+    'description' => '_MI_WGGITHUB_NUMB_COL_DESC',
+    'formtype'    => 'select',
+    'valuetype'   => 'int',
+    'default'     => 1,
+    'options'     => [1 => '1', 2 => '2', 3 => '3', 4 => '4'],
 ];
 // Divide by
 $modversion['config'][] = [
-	'name'        => 'divideby',
-	'title'       => '_MI_WGGITHUB_DIVIDEBY',
-	'description' => '_MI_WGGITHUB_DIVIDEBY_DESC',
-	'formtype'    => 'select',
-	'valuetype'   => 'int',
-	'default'     => 1,
-	'options'     => [1 => '1', 2 => '2', 3 => '3', 4 => '4'],
+    'name'        => 'divideby',
+    'title'       => '_MI_WGGITHUB_DIVIDEBY',
+    'description' => '_MI_WGGITHUB_DIVIDEBY_DESC',
+    'formtype'    => 'select',
+    'valuetype'   => 'int',
+    'default'     => 1,
+    'options'     => [1 => '1', 2 => '2', 3 => '3', 4 => '4'],
 ];
 // Table type
 $modversion['config'][] = [
-	'name'        => 'table_type',
-	'title'       => '_MI_WGGITHUB_TABLE_TYPE',
-	'description' => '_MI_WGGITHUB_DIVIDEBY_DESC',
-	'formtype'    => 'select',
-	'valuetype'   => 'int',
-	'default'     => 'bordered',
-	'options'     => ['bordered' => 'bordered', 'striped' => 'striped', 'hover' => 'hover', 'condensed' => 'condensed'],
+    'name'        => 'table_type',
+    'title'       => '_MI_WGGITHUB_TABLE_TYPE',
+    'description' => '_MI_WGGITHUB_DIVIDEBY_DESC',
+    'formtype'    => 'select',
+    'valuetype'   => 'int',
+    'default'     => 'bordered',
+    'options'     => ['bordered' => 'bordered', 'striped' => 'striped', 'hover' => 'hover', 'condensed' => 'condensed'],
 ];
 // Panel by
 $modversion['config'][] = [
-	'name'        => 'panel_type',
-	'title'       => '_MI_WGGITHUB_PANEL_TYPE',
-	'description' => '_MI_WGGITHUB_PANEL_TYPE_DESC',
-	'formtype'    => 'select',
-	'valuetype'   => 'text',
-	'default'     => 'default',
-	'options'     => ['default' => 'default', 'primary' => 'primary', 'success' => 'success', 'info' => 'info', 'warning' => 'warning', 'danger' => 'danger'],
+    'name'        => 'panel_type',
+    'title'       => '_MI_WGGITHUB_PANEL_TYPE',
+    'description' => '_MI_WGGITHUB_PANEL_TYPE_DESC',
+    'formtype'    => 'select',
+    'valuetype'   => 'text',
+    'default'     => 'default',
+    'options'     => ['default' => 'default', 'primary' => 'primary', 'success' => 'success', 'info' => 'info', 'warning' => 'warning', 'danger' => 'danger'],
 ];
 // Advertise
 $modversion['config'][] = [
-	'name'        => 'advertise',
-	'title'       => '_MI_WGGITHUB_ADVERTISE',
-	'description' => '_MI_WGGITHUB_ADVERTISE_DESC',
-	'formtype'    => 'textarea',
-	'valuetype'   => 'text',
-	'default'     => '',
+    'name'        => 'advertise',
+    'title'       => '_MI_WGGITHUB_ADVERTISE',
+    'description' => '_MI_WGGITHUB_ADVERTISE_DESC',
+    'formtype'    => 'textarea',
+    'valuetype'   => 'text',
+    'default'     => '',
 ];
 // Bookmarks
 $modversion['config'][] = [
-	'name'        => 'bookmarks',
-	'title'       => '_MI_WGGITHUB_BOOKMARKS',
-	'description' => '_MI_WGGITHUB_BOOKMARKS_DESC',
-	'formtype'    => 'yesno',
-	'valuetype'   => 'int',
-	'default'     => 0,
+    'name'        => 'bookmarks',
+    'title'       => '_MI_WGGITHUB_BOOKMARKS',
+    'description' => '_MI_WGGITHUB_BOOKMARKS_DESC',
+    'formtype'    => 'yesno',
+    'valuetype'   => 'int',
+    'default'     => 0,
 ];
 // Make Sample button visible?
 $modversion['config'][] = [
-	'name'        => 'displaySampleButton',
-	'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON',
-	'description' => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON_DESC',
-	'formtype'    => 'yesno',
-	'valuetype'   => 'int',
-	'default'     => 1,
+    'name'        => 'displaySampleButton',
+    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON',
+    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON_DESC',
+    'formtype'    => 'yesno',
+    'valuetype'   => 'int',
+    'default'     => 1,
 ];
 // Maintained by
 $modversion['config'][] = [
-	'name'        => 'maintainedby',
-	'title'       => '_MI_WGGITHUB_MAINTAINEDBY',
-	'description' => '_MI_WGGITHUB_MAINTAINEDBY_DESC',
-	'formtype'    => 'textbox',
-	'valuetype'   => 'text',
-	'default'     => 'https://xoops.org/modules/newbb',
+    'name'        => 'maintainedby',
+    'title'       => '_MI_WGGITHUB_MAINTAINEDBY',
+    'description' => '_MI_WGGITHUB_MAINTAINEDBY_DESC',
+    'formtype'    => 'textbox',
+    'valuetype'   => 'text',
+    'default'     => 'https://xoops.org/modules/newbb',
 ];

class/Github/Paginator.php

Indentation +142 added lines, -142 removed lines

@@ -12,147 +12,147 @@
  */
 class Paginator extends Sanity implements \Iterator
 {
-	/** @var Api */
-	private $api;
-
-	/** @var Http\Request */
-	private $firstRequest;
-
-	/** @var Http\Request|NULL */
-	private $request;
-
-	/** @var Http\Response|NULL */
-	private $response;
-
-	/** @var int */
-	private $limit;
-
-	/** @var int */
-	private $counter = 0;
-
-
-	public function __construct(Api $api, Http\Request $request)
-	{
-		$this->api = $api;
-		$this->firstRequest = clone $request;
-	}
-
-
-	/**
-	 * Limits maximum steps of iteration.
-	 *
-	 * @param  int|NULL
-	 * @return self
-	 */
-	public function limit($limit)
-	{
-		$this->limit = $limit === NULL
-			? NULL
-			: (int) $limit;
-
-		return $this;
-	}
-
-
-	/**
-	 * @return void
-	 */
-	public function rewind()
-	{
-		$this->request = $this->firstRequest;
-		$this->response = NULL;
-		$this->counter = 0;
-	}
-
-
-	/**
-	 * @return bool
-	 */
-	public function valid()
-	{
-		return $this->request !== NULL && ($this->limit === NULL || $this->counter < $this->limit);
-	}
-
-
-	/**
-	 * @return Http\Response
-	 */
-	public function current()
-	{
-		$this->load();
-		return $this->response;
-	}
-
-
-	/**
-	 * @return int
-	 */
-	public function key()
-	{
-		return static::parsePage($this->request->getUrl());
-	}
-
-
-	/**
-	 * @return void
-	 */
-	public function next()
-	{
-		$this->load();
-
-		if ($url = static::parseLink($this->response->getHeader('Link'), 'next')) {
-			$this->request = new Http\Request(
-				$this->request->getMethod(),
-				$url,
-				$this->request->getHeaders(),
-				$this->request->getContent()
-			);
-		} else {
-			$this->request = NULL;
-		}
-
-		$this->response = NULL;
-		$this->counter++;
-	}
-
-
-	private function load()
-	{
-		if ($this->response === NULL) {
-			$this->response = $this->api->request($this->request);
-		}
-	}
-
-
-	/**
-	 * @param  string
-	 * @return int
-	 */
-	public static function parsePage($url)
-	{
-		list (, $parametersStr) = explode('?', $url, 2) + ['', ''];
-		parse_str($parametersStr, $parameters);
-
-		return isset($parameters['page'])
-			? max(1, (int) $parameters['page'])
-			: 1;
-	}
-
-
-	/**
-	 * @see  https://developer.github.com/guides/traversing-with-pagination/#navigating-through-the-pages
-	 *
-	 * @param  string
-	 * @param  string
-	 * @return string|NULL
-	 */
-	public static function parseLink($link, $rel)
-	{
-		if (!preg_match('(<([^>]+)>;\s*rel="' . preg_quote($rel) . '")', $link, $match)) {
-			return NULL;
-		}
-
-		return $match[1];
-	}
+    /** @var Api */
+    private $api;
+
+    /** @var Http\Request */
+    private $firstRequest;
+
+    /** @var Http\Request|NULL */
+    private $request;
+
+    /** @var Http\Response|NULL */
+    private $response;
+
+    /** @var int */
+    private $limit;
+
+    /** @var int */
+    private $counter = 0;
+
+
+    public function __construct(Api $api, Http\Request $request)
+    {
+        $this->api = $api;
+        $this->firstRequest = clone $request;
+    }
+
+
+    /**
+     * Limits maximum steps of iteration.
+     *
+     * @param  int|NULL
+     * @return self
+     */
+    public function limit($limit)
+    {
+        $this->limit = $limit === NULL
+            ? NULL
+            : (int) $limit;
+
+        return $this;
+    }
+
+
+    /**
+     * @return void
+     */
+    public function rewind()
+    {
+        $this->request = $this->firstRequest;
+        $this->response = NULL;
+        $this->counter = 0;
+    }
+
+
+    /**
+     * @return bool
+     */
+    public function valid()
+    {
+        return $this->request !== NULL && ($this->limit === NULL || $this->counter < $this->limit);
+    }
+
+
+    /**
+     * @return Http\Response
+     */
+    public function current()
+    {
+        $this->load();
+        return $this->response;
+    }
+
+
+    /**
+     * @return int
+     */
+    public function key()
+    {
+        return static::parsePage($this->request->getUrl());
+    }
+
+
+    /**
+     * @return void
+     */
+    public function next()
+    {
+        $this->load();
+
+        if ($url = static::parseLink($this->response->getHeader('Link'), 'next')) {
+            $this->request = new Http\Request(
+                $this->request->getMethod(),
+                $url,
+                $this->request->getHeaders(),
+                $this->request->getContent()
+            );
+        } else {
+            $this->request = NULL;
+        }
+
+        $this->response = NULL;
+        $this->counter++;
+    }
+
+
+    private function load()
+    {
+        if ($this->response === NULL) {
+            $this->response = $this->api->request($this->request);
+        }
+    }
+
+
+    /**
+     * @param  string
+     * @return int
+     */
+    public static function parsePage($url)
+    {
+        list (, $parametersStr) = explode('?', $url, 2) + ['', ''];
+        parse_str($parametersStr, $parameters);
+
+        return isset($parameters['page'])
+            ? max(1, (int) $parameters['page'])
+            : 1;
+    }
+
+
+    /**
+     * @see  https://developer.github.com/guides/traversing-with-pagination/#navigating-through-the-pages
+     *
+     * @param  string
+     * @param  string
+     * @return string|NULL
+     */
+    public static function parseLink($link, $rel)
+    {
+        if (!preg_match('(<([^>]+)>;\s*rel="' . preg_quote($rel) . '")', $link, $match)) {
+            return NULL;
+        }
+
+        return $match[1];
+    }
 
 }

Spacing +1 added lines, -1 removed lines

@@ -148,7 +148,7 @@
 	 */
 	public static function parseLink($link, $rel)
 	{
-		if (!preg_match('(<([^>]+)>;\s*rel="' . preg_quote($rel) . '")', $link, $match)) {
+		if (!preg_match('(<([^>]+)>;\s*rel="'.preg_quote($rel).'")', $link, $match)) {
 			return NULL;
 		}
 

class/Github/Sanity.php

Indentation +14 added lines, -14 removed lines

@@ -8,21 +8,21 @@
  */
 abstract class Sanity
 {
-	/**
-	 * @throws LogicException
-	 */
-	public function & __get($name)
-	{
-		throw new LogicException('Cannot read an undeclared property ' . get_class($this) . "::\$$name.");
-	}
+    /**
+     * @throws LogicException
+     */
+    public function & __get($name)
+    {
+        throw new LogicException('Cannot read an undeclared property ' . get_class($this) . "::\$$name.");
+    }
 
 
-	/**
-	 * @throws LogicException
-	 */
-	public function __set($name, $value)
-	{
-		throw new LogicException('Cannot write to an undeclared property ' . get_class($this) . "::\$$name.");
-	}
+    /**
+     * @throws LogicException
+     */
+    public function __set($name, $value)
+    {
+        throw new LogicException('Cannot write to an undeclared property ' . get_class($this) . "::\$$name.");
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -13,7 +13,7 @@ 
 	 */
 	public function & __get($name)
 	{
-		throw new LogicException('Cannot read an undeclared property ' . get_class($this) . "::\$$name.");
+		throw new LogicException('Cannot read an undeclared property '.get_class($this)."::\$$name.");
 	}
 
 
@@ -22,7 +22,7 @@ 
 	 */
 	public function __set($name, $value)
 	{
-		throw new LogicException('Cannot write to an undeclared property ' . get_class($this) . "::\$$name.");
+		throw new LogicException('Cannot write to an undeclared property '.get_class($this)."::\$$name.");
 	}
 
 }