juliushaertl / apporder

templates/admin.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
     <h2><?php p($l->t('App Order')) ?></h2>
     <p><?php p($l->t('Set default order for all users')) ?></p>
     <ul id="appsorter">
-    <?php foreach($_['nav'] as $entry) { ?>
+    <?php foreach ($_['nav'] as $entry) { ?>
         <li>
             <img class="app-icon svg" alt="" src="<?php print_unescaped($entry['icon']); ?>">
             <a href="<?php print_unescaped($entry['href']); ?>">

controller/settingscontroller.php

Spacing +12 added lines, -12 removed lines

@@ -8,21 +8,21 @@ 
 use \OCP\INavigationManager;
 use \OCA\AppOrder\Service\ConfigService;
 
-class SettingsController extends Controller{
+class SettingsController extends Controller {
 
     private $userId;
     private $l10n;
     private $appConfig;
     private $navigationManager;
 
-    public function __construct($appName, IRequest $request, ConfigService $appConfig, INavigationManager $navigationManager, $userId){
+    public function __construct($appName, IRequest $request, ConfigService $appConfig, INavigationManager $navigationManager, $userId) {
         parent::__construct($appName, $request);
         $this->userId = $userId;
         $this->appConfig = $appConfig;
         $this->navigationManager = $navigationManager;
     }
 
-    public function adminIndex(){
+    public function adminIndex() {
         $navigation = $this->navigationManager->getAll();
         $order = json_decode($this->appConfig->getAppValue('order'));
         $nav = $this->matchOrder($navigation, $order);
@@ -47,12 +47,12 @@ 
     private function matchOrder($nav, $order) {
         $nav_tmp = array();
         $result = array();
-        foreach($nav as $app)
+        foreach ($nav as $app)
             $nav_tmp[$app['href']] = $app;
-        foreach($order as $app)
+        foreach ($order as $app)
             $result[$app] = $nav_tmp[$app];
-        foreach($nav as $app)
-            if(!array_key_exists($app['href'], $result))
+        foreach ($nav as $app)
+            if (!array_key_exists($app['href'], $result))
                 $result[$app['href']] = $app;
         return $result;
     }
@@ -60,7 +60,7 @@ 
     /**
      * @NoAdminRequired
      */
-    public function getOrder(){
+    public function getOrder() {
         $order = $this->getAppOrder();
         return array('status' => 'success', 'order' => $order);
     }
@@ -68,7 +68,7 @@ 
     /**
      * @NoAdminRequired
      */
-    public function savePersonal($order){
+    public function savePersonal($order) {
         $this->appConfig->setUserValue('order', $this->userId, $order);
         $response = array(
             'status' => 'success',
@@ -78,9 +78,9 @@ 
         return $response;
     }
 
-    public function saveDefaultOrder($order){
-        if (!is_null($order)){
-            $this->appConfig->setAppValue('order',$order);
+    public function saveDefaultOrder($order) {
+        if (!is_null($order)) {
+            $this->appConfig->setAppValue('order', $order);
         }
         return array('status' => 'success', 'order' => $order);
     }

Braces +14 added lines, -10 removed lines

@@ -37,23 +37,27 @@
     private function getAppOrder() {
         $order_user = $this->appConfig->getUserValue('order', $this->userId);
         $order_default = $this->appConfig->getAppValue('order');
-        if ($order_user !== null && $order_user !== "")
-            $order = $order_user;
-        else
-            $order = $order_default;
+        if ($order_user !== null && $order_user !== "") {
+                    $order = $order_user;
+        } else {
+                    $order = $order_default;
+        }
         return $order;
     }
 
     private function matchOrder($nav, $order) {
         $nav_tmp = array();
         $result = array();
-        foreach($nav as $app)
-            $nav_tmp[$app['href']] = $app;
-        foreach($order as $app)
-            $result[$app] = $nav_tmp[$app];
-        foreach($nav as $app)
-            if(!array_key_exists($app['href'], $result))
+        foreach($nav as $app) {
+                    $nav_tmp[$app['href']] = $app;
+        }
+        foreach($order as $app) {
+                    $result[$app] = $nav_tmp[$app];
+        }
+        foreach($nav as $app) {
+                    if(!array_key_exists($app['href'], $result))
                 $result[$app['href']] = $app;
+        }
         return $result;
     }
 

appinfo/application.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 
 class Application extends App {
 
-    public function __construct(array $urlParams=array()){
+    public function __construct(array $urlParams = array()) {
         parent::__construct('apporder', $urlParams);
         $container = $this->getContainer();
         $container->registerService('ConfigService', function($c) {

service/configservice.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
     private $config;
     private $appName;
 
-    public function __construct(IConfig $config, $appName){
+    public function __construct(IConfig $config, $appName) {
         $this->config = $config;
         $this->appName = $appName;
     }