Changeset 1488

Timestamp:

12/22/07 19:00:47 (8 months ago)

Author:

rho

Message:

fixed page's parent reassign

• when updating a page's parent that has childs, childs wasn't updated
  to lose parent
  

Signed-off: Rolando Espinoza La fuente <rho@prosoftpeople.com>

Files:

• devel/mod/pages/lib.php (modified) (1 diff)
• devel/mod/pages/lib/pages.inc.php (modified) (2 diffs)

devel/mod/pages/lib.php

@@ -359 +359 @@
-                $parent = $li[0];
-                $childs = isset($li[1]) ? $li[1] : null;
+
+                // workaround
+                if (!$parent) {
+                    // this never should happen
+                    trigger_error(__FUNCTION__.": invalid menu element", E_USER_NOTICE);
+                    continue;
+                }
-
-                // set owner

devel/mod/pages/lib/pages.inc.php

@@ -102 +102 @@
-                        $rs = true;
-                    } else {
+                        $oldparent = get_field('pages', 'parent', 'ident', $page->ident);
-                        $rs = update_record('pages', $page);
-                        if ($rs) {
-                            $messages[] = __gettext('Page updated successful');
+
+                            // check parent menu changes
+                            // If it's top menu, update childs 
+                            if ($page->parent != $oldparent && $oldparent == 0) {
+                                // set childs' parent to 0 (top menu)
+                                execute_sql("UPDATE {$CFG->prefix}pages SET parent=0 WHERE parent={$page->ident} AND owner={$owner}", false);
+                            }
-                        } else {
-                            $messages[] = __gettext('Error updating the page. Please try again.');
@@ -137 +145 @@
-                pages_header_redirect(pages_url($owner, 'pages::'));
-            } else {
-
+, E_USER_NOTICE
-            }
-        }