diff --git a/ChangeLog b/ChangeLog index 918d618fc..90bb0a4c4 100644 --- a/ChangeLog +++ b/ChangeLog @@ -15,6 +15,7 @@ Improvements: * Display project exports in modal box * Improve accordion component * Improve currencies pages navigation +* Improve link labels pages navigation * Offer the possibility to define version compatibility from plugins Version 1.0.36 (Dec 30, 2016) diff --git a/app/Controller/CurrencyController.php b/app/Controller/CurrencyController.php index 2b2275acd..155e229e1 100644 --- a/app/Controller/CurrencyController.php +++ b/app/Controller/CurrencyController.php @@ -19,9 +19,9 @@ class CurrencyController extends BaseController { $this->response->html($this->helper->layout->config('currency/show', array( 'application_currency' => $this->configModel->get('application_currency'), - 'rates' => $this->currencyModel->getAll(), - 'currencies' => $this->currencyModel->getCurrencies(), - 'title' => t('Settings').' > '.t('Currency rates'), + 'rates' => $this->currencyModel->getAll(), + 'currencies' => $this->currencyModel->getCurrencies(), + 'title' => t('Settings') . ' > ' . t('Currency rates'), ))); } @@ -34,9 +34,9 @@ class CurrencyController extends BaseController */ public function create(array $values = array(), array $errors = array()) { - $this->response->html($this->helper->layout->config('currency/create', array( - 'values' => $values, - 'errors' => $errors, + $this->response->html($this->template->render('currency/create', array( + 'values' => $values, + 'errors' => $errors, 'currencies' => $this->currencyModel->getCurrencies(), ))); } @@ -77,9 +77,9 @@ class CurrencyController extends BaseController $values['application_currency'] = $this->configModel->get('application_currency'); } - $this->response->html($this->helper->layout->config('currency/change', array( - 'values' => $values, - 'errors' => $errors, + $this->response->html($this->template->render('currency/change', array( + 'values' => $values, + 'errors' => $errors, 'currencies' => $this->currencyModel->getCurrencies(), ))); } diff --git a/app/Controller/LinkController.php b/app/Controller/LinkController.php index 477b25a48..2ad8a2b55 100644 --- a/app/Controller/LinkController.php +++ b/app/Controller/LinkController.php @@ -16,11 +16,11 @@ class LinkController extends BaseController /** * Get the current link * - * @access private + * @access protected * @return array * @throws PageNotFoundException */ - private function getLink() + protected function getLink() { $link = $this->linkModel->getById($this->request->getIntegerParam('link_id')); @@ -32,19 +32,31 @@ class LinkController extends BaseController } /** - * List of links + * List of labels + * + * @access public + */ + public function show() + { + $this->response->html($this->helper->layout->config('link/show', array( + 'links' => $this->linkModel->getMergedList(), + 'title' => t('Settings').' > '.t('Link labels'), + ))); + } + + /** + * Add new link label * * @access public * @param array $values * @param array $errors */ - public function index(array $values = array(), array $errors = array()) + public function create(array $values = array(), array $errors = array()) { - $this->response->html($this->helper->layout->config('link/index', array( - 'links' => $this->linkModel->getMergedList(), + $this->response->html($this->template->render('link/create', array( + 'links' => $this->linkModel->getMergedList(), 'values' => $values, 'errors' => $errors, - 'title' => t('Settings').' > '.t('Task\'s links'), ))); } @@ -61,21 +73,22 @@ class LinkController extends BaseController if ($valid) { if ($this->linkModel->create($values['label'], $values['opposite_label']) !== false) { $this->flash->success(t('Link added successfully.')); - return $this->response->redirect($this->helper->url->to('LinkController', 'index')); + $this->response->redirect($this->helper->url->to('LinkController', 'show'), true); + return; } else { $this->flash->failure(t('Unable to create your link.')); } } - return $this->index($values, $errors); + $this->create($values, $errors); } /** * Edit form * * @access public - * @param array $values - * @param array $errors + * @param array $values + * @param array $errors * @throws PageNotFoundException */ public function edit(array $values = array(), array $errors = array()) @@ -83,12 +96,11 @@ class LinkController extends BaseController $link = $this->getLink(); $link['label'] = t($link['label']); - $this->response->html($this->helper->layout->config('link/edit', array( + $this->response->html($this->template->render('link/edit', array( 'values' => $values ?: $link, 'errors' => $errors, 'labels' => $this->linkModel->getList($link['id']), - 'link' => $link, - 'title' => t('Link modification') + 'link' => $link, ))); } @@ -105,13 +117,14 @@ class LinkController extends BaseController if ($valid) { if ($this->linkModel->update($values)) { $this->flash->success(t('Link updated successfully.')); - return $this->response->redirect($this->helper->url->to('LinkController', 'index')); + $this->response->redirect($this->helper->url->to('LinkController', 'show'), true); + return; } else { $this->flash->failure(t('Unable to update your link.')); } } - return $this->edit($values, $errors); + $this->edit($values, $errors); } /** @@ -123,9 +136,8 @@ class LinkController extends BaseController { $link = $this->getLink(); - $this->response->html($this->helper->layout->config('link/remove', array( + $this->response->html($this->template->render('link/remove', array( 'link' => $link, - 'title' => t('Remove a link') ))); } @@ -145,6 +157,6 @@ class LinkController extends BaseController $this->flash->failure(t('Unable to remove this link.')); } - $this->response->redirect($this->helper->url->to('LinkController', 'index')); + $this->response->redirect($this->helper->url->to('LinkController', 'show'), true); } } diff --git a/app/Template/config/sidebar.php b/app/Template/config/sidebar.php index f35efde08..95be963b9 100644 --- a/app/Template/config/sidebar.php +++ b/app/Template/config/sidebar.php @@ -22,9 +22,9 @@ url->link(t('Tags management'), 'TagController', 'index') ?>
  • app->checkMenuSelection('LinkController') ?>> - url->link(t('Link settings'), 'LinkController', 'index') ?> + url->link(t('Link labels'), 'LinkController', 'show') ?>
  • -
  • app->checkMenuSelection('CurrencyController', 'show') ?>> +
  • app->checkMenuSelection('CurrencyController') ?>> url->link(t('Currency rates'), 'CurrencyController', 'show') ?>
  • app->checkMenuSelection('ConfigController', 'integrations') ?>> diff --git a/app/Template/link/create.php b/app/Template/link/create.php index 23990604f..37610a3b8 100644 --- a/app/Template/link/create.php +++ b/app/Template/link/create.php @@ -1,18 +1,12 @@
    - form->csrf() ?> - form->label(t('Label'), 'label') ?> - form->text('label', $values, $errors, array('required')) ?> - + form->text('label', $values, $errors, array('required', 'autofocus')) ?> form->label(t('Opposite label'), 'opposite_label') ?> form->text('opposite_label', $values, $errors) ?> - -
    - -
    + modal->submitButtons() ?>
    diff --git a/app/Template/link/index.php b/app/Template/link/index.php deleted file mode 100644 index b0dcad1e4..000000000 --- a/app/Template/link/index.php +++ /dev/null @@ -1,33 +0,0 @@ - - - - - - - - - - - - - -
    - - - - | - - -
      - modal->medium('edit', t('Edit'), 'LinkController', 'edit', array('link_id' => $link['id'])) ?> - - modal->confirm('trash-o', t('Remove'), 'LinkController', 'confirm', array('link_id' => $link['id'])) ?> -
    -
    - - - - -render('link/create', array('values' => $values, 'errors' => $errors)) ?> diff --git a/app/Template/link/show.php b/app/Template/link/show.php new file mode 100644 index 000000000..6aadd66b8 --- /dev/null +++ b/app/Template/link/show.php @@ -0,0 +1,36 @@ + + + + + + + + + + + + + +
    + + + + | + + +
      + modal->medium('edit', t('Edit'), 'LinkController', 'edit', array('link_id' => $link['id'])) ?> + + modal->confirm('trash-o', t('Remove'), 'LinkController', 'confirm', array('link_id' => $link['id'])) ?> +
    +
    + + +