Inspection of "- added field repo_readme" - ggoffy/wggithub - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 55161e...62ffb0 )

by Goffy

created 2020-11-27 21:32 UTC

Status

Indentation +109 added lines, -109 removed lines patch added patch discarded remove patch

@@ -31,116 +31,116 @@
 block discarded – undo
 // Request repo_id
 $repoId = Request::getInt('repo_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_repositories.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_REPOSITORY, 'repositories.php?op=new', 'add');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		$repositoriesCount = $repositoriesHandler->getCountRepositories();
-		$repositoriesAll = $repositoriesHandler->getAllRepositories($start, $limit);
-		$GLOBALS['xoopsTpl']->assign('repositories_count', $repositoriesCount);
-		$GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
-		$GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
-		// Table view repositories
-		if ($repositoriesCount > 0) {
-			foreach (\array_keys($repositoriesAll) as $i) {
-				$repository = $repositoriesAll[$i]->getValuesRepositories();
-				$GLOBALS['xoopsTpl']->append('repositories_list', $repository);
-				unset($repository);
-			}
-			// Display Navigation
-			if ($repositoriesCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($repositoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-			}
-		} else {
-			$GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_REPOSITORIES);
-		}
-		break;
-	case 'new':
-		$templateMain = 'wggithub_admin_repositories.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_REPOSITORIES_LIST, 'repositories.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Form Create
-		$repositoriesObj = $repositoriesHandler->create();
-		$form = $repositoriesObj->getFormRepositories();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'save':
-		// Security Check
-		if (!$GLOBALS['xoopsSecurity']->check()) {
-			\redirect_header('repositories.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-		}
-		if ($repoId > 0) {
-			$repositoriesObj = $repositoriesHandler->get($repoId);
-		} else {
-			$repositoriesObj = $repositoriesHandler->create();
-		}
-		// Set Vars
-		$repositoriesObj->setVar('repo_nodeid', Request::getString('repo_nodeid', ''));
-		$repositoriesObj->setVar('repo_user', Request::getString('repo_user', ''));
-		$repositoriesObj->setVar('repo_name', Request::getString('repo_name', ''));
-		$repositoriesObj->setVar('repo_fullname', Request::getString('repo_fullname', ''));
-		$repositoryCreatedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_createdat'));
-		$repositoriesObj->setVar('repo_createdat', $repositoryCreatedatObj->getTimestamp());
-		$repositoryUpdatedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_updatedat'));
-		$repositoriesObj->setVar('repo_updatedat', $repositoryUpdatedatObj->getTimestamp());
-		$repositoriesObj->setVar('repo_htmlurl', Request::getString('repo_htmlurl', ''));
+    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_repositories.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_REPOSITORY, 'repositories.php?op=new', 'add');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        $repositoriesCount = $repositoriesHandler->getCountRepositories();
+        $repositoriesAll = $repositoriesHandler->getAllRepositories($start, $limit);
+        $GLOBALS['xoopsTpl']->assign('repositories_count', $repositoriesCount);
+        $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
+        $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
+        // Table view repositories
+        if ($repositoriesCount > 0) {
+            foreach (\array_keys($repositoriesAll) as $i) {
+                $repository = $repositoriesAll[$i]->getValuesRepositories();
+                $GLOBALS['xoopsTpl']->append('repositories_list', $repository);
+                unset($repository);
+            }
+            // Display Navigation
+            if ($repositoriesCount > $limit) {
+                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($repositoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+            }
+        } else {
+            $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_REPOSITORIES);
+        }
+        break;
+    case 'new':
+        $templateMain = 'wggithub_admin_repositories.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_REPOSITORIES_LIST, 'repositories.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Form Create
+        $repositoriesObj = $repositoriesHandler->create();
+        $form = $repositoriesObj->getFormRepositories();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'save':
+        // Security Check
+        if (!$GLOBALS['xoopsSecurity']->check()) {
+            \redirect_header('repositories.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+        }
+        if ($repoId > 0) {
+            $repositoriesObj = $repositoriesHandler->get($repoId);
+        } else {
+            $repositoriesObj = $repositoriesHandler->create();
+        }
+        // Set Vars
+        $repositoriesObj->setVar('repo_nodeid', Request::getString('repo_nodeid', ''));
+        $repositoriesObj->setVar('repo_user', Request::getString('repo_user', ''));
+        $repositoriesObj->setVar('repo_name', Request::getString('repo_name', ''));
+        $repositoriesObj->setVar('repo_fullname', Request::getString('repo_fullname', ''));
+        $repositoryCreatedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_createdat'));
+        $repositoriesObj->setVar('repo_createdat', $repositoryCreatedatObj->getTimestamp());
+        $repositoryUpdatedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_updatedat'));
+        $repositoriesObj->setVar('repo_updatedat', $repositoryUpdatedatObj->getTimestamp());
+        $repositoriesObj->setVar('repo_htmlurl', Request::getString('repo_htmlurl', ''));
         $repositoriesObj->setVar('repo_prerelease', Request::getInt('repo_prerelease', 0));
         $repositoriesObj->setVar('repo_release', Request::getInt('repo_release', 0));
-		$repositoriesObj->setVar('repo_status', Request::getInt('repo_status', 0));
-		$repositoryDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_datecreated'));
-		$repositoriesObj->setVar('repo_datecreated', $repositoryDatecreatedObj->getTimestamp());
-		$repositoriesObj->setVar('repo_submitter', Request::getInt('repo_submitter', 0));
-		// Insert Data
-		if ($repositoriesHandler->insert($repositoriesObj)) {
-			\redirect_header('repositories.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
-		}
-		// Get Form
-		$GLOBALS['xoopsTpl']->assign('error', $repositoriesObj->getHtmlErrors());
-		$form = $repositoriesObj->getFormRepositories();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'edit':
-		$templateMain = 'wggithub_admin_repositories.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_REPOSITORY, 'repositories.php?op=new', 'add');
-		$adminObject->addItemButton(_AM_WGGITHUB_REPOSITORIES_LIST, 'repositories.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Get Form
-		$repositoriesObj = $repositoriesHandler->get($repoId);
-		$form = $repositoriesObj->getFormRepositories();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'delete':
-		$templateMain = 'wggithub_admin_repositories.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
-		$repositoriesObj = $repositoriesHandler->get($repoId);
-		$repoName = $repositoriesObj->getVar('repo_name');
-		if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
-			if (!$GLOBALS['xoopsSecurity']->check()) {
-				\redirect_header('repositories.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
-			}
-			if ($repositoriesHandler->delete($repositoriesObj)) {
-				\redirect_header('repositories.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
-			} else {
-				$GLOBALS['xoopsTpl']->assign('error', $repositoriesObj->getHtmlErrors());
-			}
-		} else {
-			$xoopsconfirm = new Common\XoopsConfirm(
-				['ok' => 1, 'repo_id' => $repoId, 'op' => 'delete'],
-				$_SERVER['REQUEST_URI'],
-				\sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $repositoriesObj->getVar('repo_name')));
-			$form = $xoopsconfirm->getFormXoopsConfirm();
-			$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		}
-		break;
+        $repositoriesObj->setVar('repo_status', Request::getInt('repo_status', 0));
+        $repositoryDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_datecreated'));
+        $repositoriesObj->setVar('repo_datecreated', $repositoryDatecreatedObj->getTimestamp());
+        $repositoriesObj->setVar('repo_submitter', Request::getInt('repo_submitter', 0));
+        // Insert Data
+        if ($repositoriesHandler->insert($repositoriesObj)) {
+            \redirect_header('repositories.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
+        }
+        // Get Form
+        $GLOBALS['xoopsTpl']->assign('error', $repositoriesObj->getHtmlErrors());
+        $form = $repositoriesObj->getFormRepositories();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'edit':
+        $templateMain = 'wggithub_admin_repositories.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_REPOSITORY, 'repositories.php?op=new', 'add');
+        $adminObject->addItemButton(_AM_WGGITHUB_REPOSITORIES_LIST, 'repositories.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Get Form
+        $repositoriesObj = $repositoriesHandler->get($repoId);
+        $form = $repositoriesObj->getFormRepositories();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'delete':
+        $templateMain = 'wggithub_admin_repositories.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
+        $repositoriesObj = $repositoriesHandler->get($repoId);
+        $repoName = $repositoriesObj->getVar('repo_name');
+        if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
+            if (!$GLOBALS['xoopsSecurity']->check()) {
+                \redirect_header('repositories.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
+            }
+            if ($repositoriesHandler->delete($repositoriesObj)) {
+                \redirect_header('repositories.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
+            } else {
+                $GLOBALS['xoopsTpl']->assign('error', $repositoriesObj->getHtmlErrors());
+            }
+        } else {
+            $xoopsconfirm = new Common\XoopsConfirm(
+                ['ok' => 1, 'repo_id' => $repoId, 'op' => 'delete'],
+                $_SERVER['REQUEST_URI'],
+                \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $repositoriesObj->getVar('repo_name')));
+            $form = $xoopsconfirm->getFormXoopsConfirm();
+            $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        }
+        break;
 }
 require __DIR__ . '/footer.php';

Please login to merge, or discard this patch.

class/Repositories.php 1 patch

Indentation +128 added lines, -128 removed lines patch added patch discarded remove patch

@@ -32,85 +32,85 @@  discard block
 block discarded – undo
  */
 class Repositories extends \XoopsObject
 {
-	/**
-	 * Constructor
-	 *
-	 * @param null
-	 */
-	public function __construct()
-	{
-		$this->initVar('repo_id', XOBJ_DTYPE_INT);
-		$this->initVar('repo_nodeid', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('repo_user', XOBJ_DTYPE_TXTBOX);
+    /**
+     * Constructor
+     *
+     * @param null
+     */
+    public function __construct()
+    {
+        $this->initVar('repo_id', XOBJ_DTYPE_INT);
+        $this->initVar('repo_nodeid', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('repo_user', XOBJ_DTYPE_TXTBOX);
         $this->initVar('repo_name', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('repo_fullname', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('repo_createdat', XOBJ_DTYPE_INT);
-		$this->initVar('repo_updatedat', XOBJ_DTYPE_INT);
-		$this->initVar('repo_htmlurl', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('repo_fullname', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('repo_createdat', XOBJ_DTYPE_INT);
+        $this->initVar('repo_updatedat', XOBJ_DTYPE_INT);
+        $this->initVar('repo_htmlurl', XOBJ_DTYPE_TXTBOX);
         $this->initVar('repo_readme', XOBJ_DTYPE_TXTBOX);
         $this->initVar('repo_prerelease', XOBJ_DTYPE_TXTBOX);
         $this->initVar('repo_release', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('repo_status', XOBJ_DTYPE_INT);
-		$this->initVar('repo_datecreated', XOBJ_DTYPE_INT);
-		$this->initVar('repo_submitter', XOBJ_DTYPE_INT);
-	}
+        $this->initVar('repo_status', XOBJ_DTYPE_INT);
+        $this->initVar('repo_datecreated', XOBJ_DTYPE_INT);
+        $this->initVar('repo_submitter', XOBJ_DTYPE_INT);
+    }
 
-	/**
-	 * @static function &getInstance
-	 *
-	 * @param null
-	 */
-	public static function getInstance()
-	{
-		static $instance = false;
-		if (!$instance) {
-			$instance = new self();
-		}
-	}
+    /**
+     * @static function &getInstance
+     *
+     * @param null
+     */
+    public static function getInstance()
+    {
+        static $instance = false;
+        if (!$instance) {
+            $instance = new self();
+        }
+    }
 
-	/**
-	 * The new inserted $Id
-	 * @return inserted id
-	 */
-	public function getNewInsertedIdRepositories()
-	{
-		$newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
-		return $newInsertedId;
-	}
+    /**
+     * The new inserted $Id
+     * @return inserted id
+     */
+    public function getNewInsertedIdRepositories()
+    {
+        $newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
+        return $newInsertedId;
+    }
 
-	/**
-	 * @public function getForm
-	 * @param bool $action
-	 * @return \XoopsThemeForm
-	 */
-	public function getFormRepositories($action = false)
-	{
-		$helper = \XoopsModules\Wggithub\Helper::getInstance();
-		if (!$action) {
-			$action = $_SERVER['REQUEST_URI'];
-		}
-		// Title
-		$title = $this->isNew() ? \sprintf(_AM_WGGITHUB_REPOSITORY_ADD) : \sprintf(_AM_WGGITHUB_REPOSITORY_EDIT);
-		// Get Theme Form
-		\xoops_load('XoopsFormLoader');
-		$form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
-		$form->setExtra('enctype="multipart/form-data"');
-		// Form Text repoNodeid
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NODEID, 'repo_nodeid', 50, 255, $this->getVar('repo_nodeid')));
+    /**
+     * @public function getForm
+     * @param bool $action
+     * @return \XoopsThemeForm
+     */
+    public function getFormRepositories($action = false)
+    {
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
+        if (!$action) {
+            $action = $_SERVER['REQUEST_URI'];
+        }
+        // Title
+        $title = $this->isNew() ? \sprintf(_AM_WGGITHUB_REPOSITORY_ADD) : \sprintf(_AM_WGGITHUB_REPOSITORY_EDIT);
+        // Get Theme Form
+        \xoops_load('XoopsFormLoader');
+        $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
+        $form->setExtra('enctype="multipart/form-data"');
+        // Form Text repoNodeid
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NODEID, 'repo_nodeid', 50, 255, $this->getVar('repo_nodeid')));
         // Form Text repoUser
         $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_USER, 'repo_user', 50, 255, $this->getVar('repo_user')), true);
-		// Form Text repoName
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NAME, 'repo_name', 50, 255, $this->getVar('repo_name')), true);
-		// Form Text repoFullname
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_FULLNAME, 'repo_fullname', 50, 255, $this->getVar('repo_fullname')));
-		// Form Text Date Select repoCreatedat
-		$repoCreatedat = $this->isNew() ?: $this->getVar('repo_createdat');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_CREATEDAT, 'repo_createdat', '', $repoCreatedat));
-		// Form Text Date Select repoUpdatedat
-		$repoUpdatedat = $this->isNew() ?: $this->getVar('repo_updatedat');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_UPDATEDAT, 'repo_updatedat', '', $repoUpdatedat));
-		// Form Text repoHtmlurl
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_HTMLURL, 'repo_htmlurl', 50, 255, $this->getVar('repo_htmlurl')));
+        // Form Text repoName
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NAME, 'repo_name', 50, 255, $this->getVar('repo_name')), true);
+        // Form Text repoFullname
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_FULLNAME, 'repo_fullname', 50, 255, $this->getVar('repo_fullname')));
+        // Form Text Date Select repoCreatedat
+        $repoCreatedat = $this->isNew() ?: $this->getVar('repo_createdat');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_CREATEDAT, 'repo_createdat', '', $repoCreatedat));
+        // Form Text Date Select repoUpdatedat
+        $repoUpdatedat = $this->isNew() ?: $this->getVar('repo_updatedat');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_UPDATEDAT, 'repo_updatedat', '', $repoUpdatedat));
+        // Form Text repoHtmlurl
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_HTMLURL, 'repo_htmlurl', 50, 255, $this->getVar('repo_htmlurl')));
         // Form Text repoReadme
         $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_README, 'repo_readme', 50, 255, $this->getVar('repo_readme')));
         // Form Text repoPrerelease
@@ -125,72 +125,72 @@  discard block
 block discarded – undo
         $repoStatusSelect->addOption(Constants::STATUS_UPDATED, _AM_WGGITHUB_STATUS_UPDATED);
         $form->addElement($repoStatusSelect);
         // Form Text Date Select repoDatecreated
-		$repoDatecreated = $this->isNew() ?: $this->getVar('repo_datecreated');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_DATECREATED, 'repo_datecreated', '', $repoDatecreated));
-		// Form Select User repoSubmitter
-		$form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_REPOSITORY_SUBMITTER, 'repo_submitter', false, $this->getVar('repo_submitter')));
-		// To Save
-		$form->addElement(new \XoopsFormHidden('op', 'save'));
-		$form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
-		return $form;
-	}
+        $repoDatecreated = $this->isNew() ?: $this->getVar('repo_datecreated');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_DATECREATED, 'repo_datecreated', '', $repoDatecreated));
+        // Form Select User repoSubmitter
+        $form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_REPOSITORY_SUBMITTER, 'repo_submitter', false, $this->getVar('repo_submitter')));
+        // To Save
+        $form->addElement(new \XoopsFormHidden('op', 'save'));
+        $form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
+        return $form;
+    }
 
-	/**
-	 * Get Values
-	 * @param null $keys
-	 * @param null $format
-	 * @param null $maxDepth
-	 * @return array
-	 */
-	public function getValuesRepositories($keys = null, $format = null, $maxDepth = null)
-	{
-		$ret = $this->getValues($keys, $format, $maxDepth);
-		$ret['id']          = $this->getVar('repo_id');
-		$ret['nodeid']      = $this->getVar('repo_nodeid');
+    /**
+     * Get Values
+     * @param null $keys
+     * @param null $format
+     * @param null $maxDepth
+     * @return array
+     */
+    public function getValuesRepositories($keys = null, $format = null, $maxDepth = null)
+    {
+        $ret = $this->getValues($keys, $format, $maxDepth);
+        $ret['id']          = $this->getVar('repo_id');
+        $ret['nodeid']      = $this->getVar('repo_nodeid');
         $ret['user']        = $this->getVar('repo_user');
-		$ret['name']        = $this->getVar('repo_name');
-		$ret['fullname']    = $this->getVar('repo_fullname');
-		$ret['createdat']   = \formatTimestamp($this->getVar('repo_createdat'), 'm');
-		$ret['updatedat']   = \formatTimestamp($this->getVar('repo_updatedat'), 'm');
-		$ret['htmlurl']     = $this->getVar('repo_htmlurl');
+        $ret['name']        = $this->getVar('repo_name');
+        $ret['fullname']    = $this->getVar('repo_fullname');
+        $ret['createdat']   = \formatTimestamp($this->getVar('repo_createdat'), 'm');
+        $ret['updatedat']   = \formatTimestamp($this->getVar('repo_updatedat'), 'm');
+        $ret['htmlurl']     = $this->getVar('repo_htmlurl');
         $ret['readme']      = $this->getVar('repo_readme');
         $ret['prerelease']  = $this->getVar('repo_prerelease');
         $ret['release']     = $this->getVar('repo_release');
-		$status             = $this->getVar('repo_status');
-		$ret['status']      = $status;
-		switch ($status) {
-			case Constants::STATUS_NONE:
-			default:
-				$status_text = \_AM_WGGITHUB_STATUS_NONE;
-				break;
+        $status             = $this->getVar('repo_status');
+        $ret['status']      = $status;
+        switch ($status) {
+            case Constants::STATUS_NONE:
+            default:
+                $status_text = \_AM_WGGITHUB_STATUS_NONE;
+                break;
             case Constants::STATUS_NEW:
                 $status_text = \_AM_WGGITHUB_STATUS_NEW;
                 break;
             case Constants::STATUS_UPTODATE:
-				$status_text = \_AM_WGGITHUB_STATUS_UPTODATE;
-				break;
-			case Constants::STATUS_UPDATED:
-				$status_text = \_AM_WGGITHUB_STATUS_UPDATED;
-				break;
-		}
-		$ret['status_text'] = $status_text;
-		$ret['datecreated'] = \formatTimestamp($this->getVar('repo_datecreated'), 'm');
-		$ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('repo_submitter'));
-		return $ret;
-	}
+                $status_text = \_AM_WGGITHUB_STATUS_UPTODATE;
+                break;
+            case Constants::STATUS_UPDATED:
+                $status_text = \_AM_WGGITHUB_STATUS_UPDATED;
+                break;
+        }
+        $ret['status_text'] = $status_text;
+        $ret['datecreated'] = \formatTimestamp($this->getVar('repo_datecreated'), 'm');
+        $ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('repo_submitter'));
+        return $ret;
+    }
 
-	/**
-	 * Returns an array representation of the object
-	 *
-	 * @return array
-	 */
-	public function toArrayRepositories()
-	{
-		$ret = [];
-		$vars = $this->getVars();
-		foreach (\array_keys($vars) as $var) {
-			$ret[$var] = $this->getVar('"{$var}"');
-		}
-		return $ret;
-	}
+    /**
+     * Returns an array representation of the object
+     *
+     * @return array
+     */
+    public function toArrayRepositories()
+    {
+        $ret = [];
+        $vars = $this->getVars();
+        foreach (\array_keys($vars) as $var) {
+            $ret[$var] = $this->getVar('"{$var}"');
+        }
+        return $ret;
+    }
 }

Please login to merge, or discard this patch.

		@@ -31,116 +31,116 @@
		block discarded – undo
31	31	// Request repo_id
32	32	$repoId = Request::getInt('repo_id');
33	33	switch ($op) {
34		- case 'list':
35		- default:
36		- // Define Stylesheet
37		- $GLOBALS['xoTheme']->addStylesheet($style, null);
38		- $start = Request::getInt('start', 0);
39		- $limit = Request::getInt('limit', $helper->getConfig('adminpager'));
40		- $templateMain = 'wggithub_admin_repositories.tpl';
41		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
42		- $adminObject->addItemButton(_AM_WGGITHUB_ADD_REPOSITORY, 'repositories.php?op=new', 'add');
43		- $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
44		- $repositoriesCount = $repositoriesHandler->getCountRepositories();
45		- $repositoriesAll = $repositoriesHandler->getAllRepositories($start, $limit);
46		- $GLOBALS['xoopsTpl']->assign('repositories_count', $repositoriesCount);
47		- $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
48		- $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
49		- // Table view repositories
50		- if ($repositoriesCount > 0) {
51		- foreach (\array_keys($repositoriesAll) as $i) {
52		- $repository = $repositoriesAll[$i]->getValuesRepositories();
53		- $GLOBALS['xoopsTpl']->append('repositories_list', $repository);
54		- unset($repository);
55		- }
56		- // Display Navigation
57		- if ($repositoriesCount > $limit) {
58		- include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
59		- $pagenav = new \XoopsPageNav($repositoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
60		- $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
61		- }
62		- } else {
63		- $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_REPOSITORIES);
64		- }
65		- break;
66		- case 'new':
67		- $templateMain = 'wggithub_admin_repositories.tpl';
68		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
69		- $adminObject->addItemButton(_AM_WGGITHUB_REPOSITORIES_LIST, 'repositories.php', 'list');
70		- $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
71		- // Form Create
72		- $repositoriesObj = $repositoriesHandler->create();
73		- $form = $repositoriesObj->getFormRepositories();
74		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
75		- break;
76		- case 'save':
77		- // Security Check
78		- if (!$GLOBALS['xoopsSecurity']->check()) {
79		- \redirect_header('repositories.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
80		- }
81		- if ($repoId > 0) {
82		- $repositoriesObj = $repositoriesHandler->get($repoId);
83		- } else {
84		- $repositoriesObj = $repositoriesHandler->create();
85		- }
86		- // Set Vars
87		- $repositoriesObj->setVar('repo_nodeid', Request::getString('repo_nodeid', ''));
88		- $repositoriesObj->setVar('repo_user', Request::getString('repo_user', ''));
89		- $repositoriesObj->setVar('repo_name', Request::getString('repo_name', ''));
90		- $repositoriesObj->setVar('repo_fullname', Request::getString('repo_fullname', ''));
91		- $repositoryCreatedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_createdat'));
92		- $repositoriesObj->setVar('repo_createdat', $repositoryCreatedatObj->getTimestamp());
93		- $repositoryUpdatedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_updatedat'));
94		- $repositoriesObj->setVar('repo_updatedat', $repositoryUpdatedatObj->getTimestamp());
95		- $repositoriesObj->setVar('repo_htmlurl', Request::getString('repo_htmlurl', ''));
	34	+ case 'list':
	35	+ default:
	36	+ // Define Stylesheet
	37	+ $GLOBALS['xoTheme']->addStylesheet($style, null);
	38	+ $start = Request::getInt('start', 0);
	39	+ $limit = Request::getInt('limit', $helper->getConfig('adminpager'));
	40	+ $templateMain = 'wggithub_admin_repositories.tpl';
	41	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
	42	+ $adminObject->addItemButton(_AM_WGGITHUB_ADD_REPOSITORY, 'repositories.php?op=new', 'add');
	43	+ $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
	44	+ $repositoriesCount = $repositoriesHandler->getCountRepositories();
	45	+ $repositoriesAll = $repositoriesHandler->getAllRepositories($start, $limit);
	46	+ $GLOBALS['xoopsTpl']->assign('repositories_count', $repositoriesCount);
	47	+ $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
	48	+ $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
	49	+ // Table view repositories
	50	+ if ($repositoriesCount > 0) {
	51	+ foreach (\array_keys($repositoriesAll) as $i) {
	52	+ $repository = $repositoriesAll[$i]->getValuesRepositories();
	53	+ $GLOBALS['xoopsTpl']->append('repositories_list', $repository);
	54	+ unset($repository);
	55	+ }
	56	+ // Display Navigation
	57	+ if ($repositoriesCount > $limit) {
	58	+ include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
	59	+ $pagenav = new \XoopsPageNav($repositoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
	60	+ $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
	61	+ }
	62	+ } else {
	63	+ $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_REPOSITORIES);
	64	+ }
	65	+ break;
	66	+ case 'new':
	67	+ $templateMain = 'wggithub_admin_repositories.tpl';
	68	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
	69	+ $adminObject->addItemButton(_AM_WGGITHUB_REPOSITORIES_LIST, 'repositories.php', 'list');
	70	+ $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
	71	+ // Form Create
	72	+ $repositoriesObj = $repositoriesHandler->create();
	73	+ $form = $repositoriesObj->getFormRepositories();
	74	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	75	+ break;
	76	+ case 'save':
	77	+ // Security Check
	78	+ if (!$GLOBALS['xoopsSecurity']->check()) {
	79	+ \redirect_header('repositories.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
	80	+ }
	81	+ if ($repoId > 0) {
	82	+ $repositoriesObj = $repositoriesHandler->get($repoId);
	83	+ } else {
	84	+ $repositoriesObj = $repositoriesHandler->create();
	85	+ }
	86	+ // Set Vars
	87	+ $repositoriesObj->setVar('repo_nodeid', Request::getString('repo_nodeid', ''));
	88	+ $repositoriesObj->setVar('repo_user', Request::getString('repo_user', ''));
	89	+ $repositoriesObj->setVar('repo_name', Request::getString('repo_name', ''));
	90	+ $repositoriesObj->setVar('repo_fullname', Request::getString('repo_fullname', ''));
	91	+ $repositoryCreatedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_createdat'));
	92	+ $repositoriesObj->setVar('repo_createdat', $repositoryCreatedatObj->getTimestamp());
	93	+ $repositoryUpdatedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_updatedat'));
	94	+ $repositoriesObj->setVar('repo_updatedat', $repositoryUpdatedatObj->getTimestamp());
	95	+ $repositoriesObj->setVar('repo_htmlurl', Request::getString('repo_htmlurl', ''));
96	96	$repositoriesObj->setVar('repo_prerelease', Request::getInt('repo_prerelease', 0));
97	97	$repositoriesObj->setVar('repo_release', Request::getInt('repo_release', 0));
98		- $repositoriesObj->setVar('repo_status', Request::getInt('repo_status', 0));
99		- $repositoryDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_datecreated'));
100		- $repositoriesObj->setVar('repo_datecreated', $repositoryDatecreatedObj->getTimestamp());
101		- $repositoriesObj->setVar('repo_submitter', Request::getInt('repo_submitter', 0));
102		- // Insert Data
103		- if ($repositoriesHandler->insert($repositoriesObj)) {
104		- \redirect_header('repositories.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
105		- }
106		- // Get Form
107		- $GLOBALS['xoopsTpl']->assign('error', $repositoriesObj->getHtmlErrors());
108		- $form = $repositoriesObj->getFormRepositories();
109		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
110		- break;
111		- case 'edit':
112		- $templateMain = 'wggithub_admin_repositories.tpl';
113		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
114		- $adminObject->addItemButton(_AM_WGGITHUB_ADD_REPOSITORY, 'repositories.php?op=new', 'add');
115		- $adminObject->addItemButton(_AM_WGGITHUB_REPOSITORIES_LIST, 'repositories.php', 'list');
116		- $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
117		- // Get Form
118		- $repositoriesObj = $repositoriesHandler->get($repoId);
119		- $form = $repositoriesObj->getFormRepositories();
120		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
121		- break;
122		- case 'delete':
123		- $templateMain = 'wggithub_admin_repositories.tpl';
124		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
125		- $repositoriesObj = $repositoriesHandler->get($repoId);
126		- $repoName = $repositoriesObj->getVar('repo_name');
127		- if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
128		- if (!$GLOBALS['xoopsSecurity']->check()) {
129		- \redirect_header('repositories.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
130		- }
131		- if ($repositoriesHandler->delete($repositoriesObj)) {
132		- \redirect_header('repositories.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
133		- } else {
134		- $GLOBALS['xoopsTpl']->assign('error', $repositoriesObj->getHtmlErrors());
135		- }
136		- } else {
137		- $xoopsconfirm = new Common\XoopsConfirm(
138		- ['ok' => 1, 'repo_id' => $repoId, 'op' => 'delete'],
139		- $_SERVER['REQUEST_URI'],
140		- \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $repositoriesObj->getVar('repo_name')));
141		- $form = $xoopsconfirm->getFormXoopsConfirm();
142		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
143		- }
144		- break;
	98	+ $repositoriesObj->setVar('repo_status', Request::getInt('repo_status', 0));
	99	+ $repositoryDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('repo_datecreated'));
	100	+ $repositoriesObj->setVar('repo_datecreated', $repositoryDatecreatedObj->getTimestamp());
	101	+ $repositoriesObj->setVar('repo_submitter', Request::getInt('repo_submitter', 0));
	102	+ // Insert Data
	103	+ if ($repositoriesHandler->insert($repositoriesObj)) {
	104	+ \redirect_header('repositories.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
	105	+ }
	106	+ // Get Form
	107	+ $GLOBALS['xoopsTpl']->assign('error', $repositoriesObj->getHtmlErrors());
	108	+ $form = $repositoriesObj->getFormRepositories();
	109	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	110	+ break;
	111	+ case 'edit':
	112	+ $templateMain = 'wggithub_admin_repositories.tpl';
	113	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
	114	+ $adminObject->addItemButton(_AM_WGGITHUB_ADD_REPOSITORY, 'repositories.php?op=new', 'add');
	115	+ $adminObject->addItemButton(_AM_WGGITHUB_REPOSITORIES_LIST, 'repositories.php', 'list');
	116	+ $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
	117	+ // Get Form
	118	+ $repositoriesObj = $repositoriesHandler->get($repoId);
	119	+ $form = $repositoriesObj->getFormRepositories();
	120	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	121	+ break;
	122	+ case 'delete':
	123	+ $templateMain = 'wggithub_admin_repositories.tpl';
	124	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('repositories.php'));
	125	+ $repositoriesObj = $repositoriesHandler->get($repoId);
	126	+ $repoName = $repositoriesObj->getVar('repo_name');
	127	+ if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
	128	+ if (!$GLOBALS['xoopsSecurity']->check()) {
	129	+ \redirect_header('repositories.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
	130	+ }
	131	+ if ($repositoriesHandler->delete($repositoriesObj)) {
	132	+ \redirect_header('repositories.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
	133	+ } else {
	134	+ $GLOBALS['xoopsTpl']->assign('error', $repositoriesObj->getHtmlErrors());
	135	+ }
	136	+ } else {
	137	+ $xoopsconfirm = new Common\XoopsConfirm(
	138	+ ['ok' => 1, 'repo_id' => $repoId, 'op' => 'delete'],
	139	+ $_SERVER['REQUEST_URI'],
	140	+ \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $repositoriesObj->getVar('repo_name')));
	141	+ $form = $xoopsconfirm->getFormXoopsConfirm();
	142	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	143	+ }
	144	+ break;
145	145	}
146	146	require __DIR__ . '/footer.php';

		@@ -32,85 +32,85 @@ discard block
		block discarded – undo
32	32	*/
33	33	class Repositories extends \XoopsObject
34	34	{
35		- /**
36		- * Constructor
37		- *
38		- * @param null
39		- */
40		- public function __construct()
41		- {
42		- $this->initVar('repo_id', XOBJ_DTYPE_INT);
43		- $this->initVar('repo_nodeid', XOBJ_DTYPE_TXTBOX);
44		- $this->initVar('repo_user', XOBJ_DTYPE_TXTBOX);
	35	+ /**
	36	+ * Constructor
	37	+ *
	38	+ * @param null
	39	+ */
	40	+ public function __construct()
	41	+ {
	42	+ $this->initVar('repo_id', XOBJ_DTYPE_INT);
	43	+ $this->initVar('repo_nodeid', XOBJ_DTYPE_TXTBOX);
	44	+ $this->initVar('repo_user', XOBJ_DTYPE_TXTBOX);
45	45	$this->initVar('repo_name', XOBJ_DTYPE_TXTBOX);
46		- $this->initVar('repo_fullname', XOBJ_DTYPE_TXTBOX);
47		- $this->initVar('repo_createdat', XOBJ_DTYPE_INT);
48		- $this->initVar('repo_updatedat', XOBJ_DTYPE_INT);
49		- $this->initVar('repo_htmlurl', XOBJ_DTYPE_TXTBOX);
	46	+ $this->initVar('repo_fullname', XOBJ_DTYPE_TXTBOX);
	47	+ $this->initVar('repo_createdat', XOBJ_DTYPE_INT);
	48	+ $this->initVar('repo_updatedat', XOBJ_DTYPE_INT);
	49	+ $this->initVar('repo_htmlurl', XOBJ_DTYPE_TXTBOX);
50	50	$this->initVar('repo_readme', XOBJ_DTYPE_TXTBOX);
51	51	$this->initVar('repo_prerelease', XOBJ_DTYPE_TXTBOX);
52	52	$this->initVar('repo_release', XOBJ_DTYPE_TXTBOX);
53		- $this->initVar('repo_status', XOBJ_DTYPE_INT);
54		- $this->initVar('repo_datecreated', XOBJ_DTYPE_INT);
55		- $this->initVar('repo_submitter', XOBJ_DTYPE_INT);
56		- }
	53	+ $this->initVar('repo_status', XOBJ_DTYPE_INT);
	54	+ $this->initVar('repo_datecreated', XOBJ_DTYPE_INT);
	55	+ $this->initVar('repo_submitter', XOBJ_DTYPE_INT);
	56	+ }
57	57
58		- /**
59		- * @static function &getInstance
60		- *
61		- * @param null
62		- */
63		- public static function getInstance()
64		- {
65		- static $instance = false;
66		- if (!$instance) {
67		- $instance = new self();
68		- }
69		- }
	58	+ /**
	59	+ * @static function &getInstance
	60	+ *
	61	+ * @param null
	62	+ */
	63	+ public static function getInstance()
	64	+ {
	65	+ static $instance = false;
	66	+ if (!$instance) {
	67	+ $instance = new self();
	68	+ }
	69	+ }
70	70
71		- /**
72		- * The new inserted $Id
73		- * @return inserted id
74		- */
75		- public function getNewInsertedIdRepositories()
76		- {
77		- $newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
78		- return $newInsertedId;
79		- }
	71	+ /**
	72	+ * The new inserted $Id
	73	+ * @return inserted id
	74	+ */
	75	+ public function getNewInsertedIdRepositories()
	76	+ {
	77	+ $newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
	78	+ return $newInsertedId;
	79	+ }
80	80
81		- /**
82		- * @public function getForm
83		- * @param bool $action
84		- * @return \XoopsThemeForm
85		- */
86		- public function getFormRepositories($action = false)
87		- {
88		- $helper = \XoopsModules\Wggithub\Helper::getInstance();
89		- if (!$action) {
90		- $action = $_SERVER['REQUEST_URI'];
91		- }
92		- // Title
93		- $title = $this->isNew() ? \sprintf(_AM_WGGITHUB_REPOSITORY_ADD) : \sprintf(_AM_WGGITHUB_REPOSITORY_EDIT);
94		- // Get Theme Form
95		- \xoops_load('XoopsFormLoader');
96		- $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
97		- $form->setExtra('enctype="multipart/form-data"');
98		- // Form Text repoNodeid
99		- $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NODEID, 'repo_nodeid', 50, 255, $this->getVar('repo_nodeid')));
	81	+ /**
	82	+ * @public function getForm
	83	+ * @param bool $action
	84	+ * @return \XoopsThemeForm
	85	+ */
	86	+ public function getFormRepositories($action = false)
	87	+ {
	88	+ $helper = \XoopsModules\Wggithub\Helper::getInstance();
	89	+ if (!$action) {
	90	+ $action = $_SERVER['REQUEST_URI'];
	91	+ }
	92	+ // Title
	93	+ $title = $this->isNew() ? \sprintf(_AM_WGGITHUB_REPOSITORY_ADD) : \sprintf(_AM_WGGITHUB_REPOSITORY_EDIT);
	94	+ // Get Theme Form
	95	+ \xoops_load('XoopsFormLoader');
	96	+ $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
	97	+ $form->setExtra('enctype="multipart/form-data"');
	98	+ // Form Text repoNodeid
	99	+ $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NODEID, 'repo_nodeid', 50, 255, $this->getVar('repo_nodeid')));
100	100	// Form Text repoUser
101	101	$form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_USER, 'repo_user', 50, 255, $this->getVar('repo_user')), true);
102		- // Form Text repoName
103		- $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NAME, 'repo_name', 50, 255, $this->getVar('repo_name')), true);
104		- // Form Text repoFullname
105		- $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_FULLNAME, 'repo_fullname', 50, 255, $this->getVar('repo_fullname')));
106		- // Form Text Date Select repoCreatedat
107		- $repoCreatedat = $this->isNew() ?: $this->getVar('repo_createdat');
108		- $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_CREATEDAT, 'repo_createdat', '', $repoCreatedat));
109		- // Form Text Date Select repoUpdatedat
110		- $repoUpdatedat = $this->isNew() ?: $this->getVar('repo_updatedat');
111		- $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_UPDATEDAT, 'repo_updatedat', '', $repoUpdatedat));
112		- // Form Text repoHtmlurl
113		- $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_HTMLURL, 'repo_htmlurl', 50, 255, $this->getVar('repo_htmlurl')));
	102	+ // Form Text repoName
	103	+ $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NAME, 'repo_name', 50, 255, $this->getVar('repo_name')), true);
	104	+ // Form Text repoFullname
	105	+ $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_FULLNAME, 'repo_fullname', 50, 255, $this->getVar('repo_fullname')));
	106	+ // Form Text Date Select repoCreatedat
	107	+ $repoCreatedat = $this->isNew() ?: $this->getVar('repo_createdat');
	108	+ $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_CREATEDAT, 'repo_createdat', '', $repoCreatedat));
	109	+ // Form Text Date Select repoUpdatedat
	110	+ $repoUpdatedat = $this->isNew() ?: $this->getVar('repo_updatedat');
	111	+ $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_UPDATEDAT, 'repo_updatedat', '', $repoUpdatedat));
	112	+ // Form Text repoHtmlurl
	113	+ $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_HTMLURL, 'repo_htmlurl', 50, 255, $this->getVar('repo_htmlurl')));
114	114	// Form Text repoReadme
115	115	$form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_README, 'repo_readme', 50, 255, $this->getVar('repo_readme')));
116	116	// Form Text repoPrerelease
		@@ -125,72 +125,72 @@ discard block
		block discarded – undo
125	125	$repoStatusSelect->addOption(Constants::STATUS_UPDATED, _AM_WGGITHUB_STATUS_UPDATED);
126	126	$form->addElement($repoStatusSelect);
127	127	// Form Text Date Select repoDatecreated
128		- $repoDatecreated = $this->isNew() ?: $this->getVar('repo_datecreated');
129		- $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_DATECREATED, 'repo_datecreated', '', $repoDatecreated));
130		- // Form Select User repoSubmitter
131		- $form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_REPOSITORY_SUBMITTER, 'repo_submitter', false, $this->getVar('repo_submitter')));
132		- // To Save
133		- $form->addElement(new \XoopsFormHidden('op', 'save'));
134		- $form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
135		- return $form;
136		- }
	128	+ $repoDatecreated = $this->isNew() ?: $this->getVar('repo_datecreated');
	129	+ $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_DATECREATED, 'repo_datecreated', '', $repoDatecreated));
	130	+ // Form Select User repoSubmitter
	131	+ $form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_REPOSITORY_SUBMITTER, 'repo_submitter', false, $this->getVar('repo_submitter')));
	132	+ // To Save
	133	+ $form->addElement(new \XoopsFormHidden('op', 'save'));
	134	+ $form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
	135	+ return $form;
	136	+ }
137	137
138		- /**
139		- * Get Values
140		- * @param null $keys
141		- * @param null $format
142		- * @param null $maxDepth
143		- * @return array
144		- */
145		- public function getValuesRepositories($keys = null, $format = null, $maxDepth = null)
146		- {
147		- $ret = $this->getValues($keys, $format, $maxDepth);
148		- $ret['id'] = $this->getVar('repo_id');
149		- $ret['nodeid'] = $this->getVar('repo_nodeid');
	138	+ /**
	139	+ * Get Values
	140	+ * @param null $keys
	141	+ * @param null $format
	142	+ * @param null $maxDepth
	143	+ * @return array
	144	+ */
	145	+ public function getValuesRepositories($keys = null, $format = null, $maxDepth = null)
	146	+ {
	147	+ $ret = $this->getValues($keys, $format, $maxDepth);
	148	+ $ret['id'] = $this->getVar('repo_id');
	149	+ $ret['nodeid'] = $this->getVar('repo_nodeid');
150	150	$ret['user'] = $this->getVar('repo_user');
151		- $ret['name'] = $this->getVar('repo_name');
152		- $ret['fullname'] = $this->getVar('repo_fullname');
153		- $ret['createdat'] = \formatTimestamp($this->getVar('repo_createdat'), 'm');
154		- $ret['updatedat'] = \formatTimestamp($this->getVar('repo_updatedat'), 'm');
155		- $ret['htmlurl'] = $this->getVar('repo_htmlurl');
	151	+ $ret['name'] = $this->getVar('repo_name');
	152	+ $ret['fullname'] = $this->getVar('repo_fullname');
	153	+ $ret['createdat'] = \formatTimestamp($this->getVar('repo_createdat'), 'm');
	154	+ $ret['updatedat'] = \formatTimestamp($this->getVar('repo_updatedat'), 'm');
	155	+ $ret['htmlurl'] = $this->getVar('repo_htmlurl');
156	156	$ret['readme'] = $this->getVar('repo_readme');
157	157	$ret['prerelease'] = $this->getVar('repo_prerelease');
158	158	$ret['release'] = $this->getVar('repo_release');
159		- $status = $this->getVar('repo_status');
160		- $ret['status'] = $status;
161		- switch ($status) {
162		- case Constants::STATUS_NONE:
163		- default:
164		- $status_text = \_AM_WGGITHUB_STATUS_NONE;
165		- break;
	159	+ $status = $this->getVar('repo_status');
	160	+ $ret['status'] = $status;
	161	+ switch ($status) {
	162	+ case Constants::STATUS_NONE:
	163	+ default:
	164	+ $status_text = \_AM_WGGITHUB_STATUS_NONE;
	165	+ break;
166	166	case Constants::STATUS_NEW:
167	167	$status_text = \_AM_WGGITHUB_STATUS_NEW;
168	168	break;
169	169	case Constants::STATUS_UPTODATE:
170		- $status_text = \_AM_WGGITHUB_STATUS_UPTODATE;
171		- break;
172		- case Constants::STATUS_UPDATED:
173		- $status_text = \_AM_WGGITHUB_STATUS_UPDATED;
174		- break;
175		- }
176		- $ret['status_text'] = $status_text;
177		- $ret['datecreated'] = \formatTimestamp($this->getVar('repo_datecreated'), 'm');
178		- $ret['submitter'] = \XoopsUser::getUnameFromId($this->getVar('repo_submitter'));
179		- return $ret;
180		- }
	170	+ $status_text = \_AM_WGGITHUB_STATUS_UPTODATE;
	171	+ break;
	172	+ case Constants::STATUS_UPDATED:
	173	+ $status_text = \_AM_WGGITHUB_STATUS_UPDATED;
	174	+ break;
	175	+ }
	176	+ $ret['status_text'] = $status_text;
	177	+ $ret['datecreated'] = \formatTimestamp($this->getVar('repo_datecreated'), 'm');
	178	+ $ret['submitter'] = \XoopsUser::getUnameFromId($this->getVar('repo_submitter'));
	179	+ return $ret;
	180	+ }
181	181
182		- /**
183		- * Returns an array representation of the object
184		- *
185		- * @return array
186		- */
187		- public function toArrayRepositories()
188		- {
189		- $ret = [];
190		- $vars = $this->getVars();
191		- foreach (\array_keys($vars) as $var) {
192		- $ret[$var] = $this->getVar('"{$var}"');
193		- }
194		- return $ret;
195		- }
	182	+ /**
	183	+ * Returns an array representation of the object
	184	+ *
	185	+ * @return array
	186	+ */
	187	+ public function toArrayRepositories()
	188	+ {
	189	+ $ret = [];
	190	+ $vars = $this->getVars();
	191	+ foreach (\array_keys($vars) as $var) {
	192	+ $ret[$var] = $this->getVar('"{$var}"');
	193	+ }
	194	+ return $ret;
	195	+ }
196	196	}

ggoffy / wggithub

Push — master ( 55161e...62ffb0 )

Status

Category

Indentation +109 added lines, -109 removed lines patch added patch discarded remove patch

Indentation +128 added lines, -128 removed lines patch added patch discarded remove patch