Changeset 3902:861a3d1a15f3 for plugins/simpleMenu/index.php

Timestamp:

10/22/18 14:46:15 (7 years ago)

Author:

franck <carnet.franck.paul@…>

Branch:

default

Message:

Use numeric fields (with min/max) for sortering items (categories, …), fix ordering of simple menu items when drag'n'drop is disabled

File:

• plugins/simpleMenu/index.php (modified) (3 diffs)

plugins/simpleMenu/index.php

@@ -321 +321 @@
                 }
                 $menu = $newmenu;
+
+                $core->auth->user_prefs->addWorkspace('accessibility');
+                if ($core->auth->user_prefs->accessibility->nodragdrop) {
+                    # Order menu items
+                    $order = [];
+                    if (empty($_POST['im_order']) && !empty($_POST['order'])) {
+                        $order = $_POST['order'];
+                        asort($order);
+                        $order = array_keys($order);
+                    } elseif (!empty($_POST['im_order'])) {
+                        $order = $_POST['im_order'];
+                        if (substr($order, -1) == ',') {
+                            $order = substr($order, 0, strlen($order) - 1);
+                        }
+                        $order = explode(',', $order);
+                    }
+                    if (!empty($order)) {
+                        $newmenu = [];
+                        foreach ($order as $i => $k) {
+                            $newmenu[] = [
+                                'label' => $menu[$k]['label'],
+                                'descr' => $menu[$k]['descr'],
+                                'url'   => $menu[$k]['url']];
+                        }
+                        $menu = $newmenu;
+                    }
+                }
+
                 // Save menu in blog settings
                 $core->blog->settings->system->put('simpleMenu', $menu);
@@ -332 +360 @@
             }
         }
-
-        # Order menu items
-        $order = [];
-        if (empty($_POST['im_order']) && !empty($_POST['order'])) {
-            $order = $_POST['order'];
-            asort($order);
-            $order = array_keys($order);
-        } elseif (!empty($_POST['im_order'])) {
-            $order = $_POST['im_order'];
-            if (substr($order, -1) == ',') {
-                $order = substr($order, 0, strlen($order) - 1);
-            }
-            $order = explode(',', $order);
-        }
-
-        if (!empty($_POST['updateaction']) && !empty($order)) {
-            try {
-                $newmenu = [];
-                foreach ($order as $i => $k) {
-                    $newmenu[] = [
-                        'label' => $menu[$k]['label'],
-                        'descr' => $menu[$k]['descr'],
-                        'url'   => $menu[$k]['url']];
-                }
-                $menu = $newmenu;
-                // Save menu in blog settings
-                $core->blog->settings->system->put('simpleMenu', $menu);
-                $core->blog->triggerBlog();
-
-                // All done successfully, return to menu items list
-                dcPage::addSuccessNotice(__('Menu items have been successfully updated.'));
-                http::redirect($p_url);
-            } catch (Exception $e) {
-                $core->error->add($e->getMessage());
-            }
-        }
-
     }
 }
@@ -573 +564 @@
             form::number(['order[' . $i . ']'], [
                 'min'        => 1,
+                'max'        => count($menu),
                 'default'    => $count,
                 'class'      => 'position',