[F] "top.tree.setLastClickedElement() is undefined" when clicking action-buttons in pop-up or in a single frame

Author: Deesen

manager/actions/mutate_content.dynamic.php

@@ -486,6 +486,10 @@ function decode(s) {
     return s;
 }
 
+function setLastClickedElement(type, id) {
+    localStorage.setItem('MODX_lastClickedElement', '['+type+','+id+']' );
+}
+
 <?php if ($content['type'] == 'reference' || $modx->manager->action == '72') { // Web Link specific ?>
 var lastImageCtrl;
 var lastFileCtrl;
@@ -641,10 +645,10 @@ function SetUrl(url, width, height, alt) {
           <li id="Button6" class="disabled"><a href="#" onclick="duplicatedocument();"><img src="<?php echo $_style["icons_resource_duplicate"] ?>" alt="icons_resource_duplicate" /> <?php echo $_lang['duplicate']?></a></li>
           <li id="Button3" class="disabled"><a href="#" onclick="deletedocument();"><img src="<?php echo $_style["icons_delete_document"] ?>" alt="icons_delete_document" /> <?php echo $_lang['delete']?></a></li>
       <?php } else { ?>
-          <li id="Button6"><a href="#" onclick="top.tree.setLastClickedElement(0,0);duplicatedocument();"><img src="<?php echo $_style["icons_resource_duplicate"] ?>" alt="icons_resource_duplicate" /> <?php echo $_lang['duplicate']?></a></li>
-          <li id="Button3"><a href="#" onclick="top.tree.setLastClickedElement(0,0);deletedocument();"><img src="<?php echo $_style["icons_delete_document"] ?>" alt="icons_delete_document" /> <?php echo $_lang['delete']?></a></li>
+          <li id="Button6"><a href="#" onclick="setLastClickedElement(0,0);duplicatedocument();"><img src="<?php echo $_style["icons_resource_duplicate"] ?>" alt="icons_resource_duplicate" /> <?php echo $_lang['duplicate']?></a></li>
+          <li id="Button3"><a href="#" onclick="setLastClickedElement(0,0);deletedocument();"><img src="<?php echo $_style["icons_delete_document"] ?>" alt="icons_delete_document" /> <?php echo $_lang['delete']?></a></li>
       <?php } ?>
-          <li id="Button5" class="transition"><a href="#" onclick="top.tree.setLastClickedElement(0,0);documentDirty=false;<?php echo $id==0 ? "document.location.href='index.php?a=2';" : "document.location.href='index.php?a=3&amp;r=1&amp;id=$id".htmlspecialchars($add_path)."';"?>"><img alt="icons_cancel" src="<?php echo $_style["icons_cancel"] ?>" /> <?php echo $_lang['cancel']?></a></li>
+          <li id="Button5" class="transition"><a href="#" onclick="setLastClickedElement(0,0);documentDirty=false;<?php echo $id==0 ? "document.location.href='index.php?a=2';" : "document.location.href='index.php?a=3&amp;r=1&amp;id=$id".htmlspecialchars($add_path)."';"?>"><img alt="icons_cancel" src="<?php echo $_style["icons_cancel"] ?>" /> <?php echo $_lang['cancel']?></a></li>
           <li id="Button4"><a href="#" onclick="window.open('<?php echo $modx->makeUrl($id); ?>','previeWin');"><img alt="icons_preview_resource" src="<?php echo $_style["icons_preview_resource"] ?>" /> <?php echo $_lang['preview']?></a></li>
       </ul>
 </div>

manager/frames/menu.php

@@ -213,6 +213,10 @@ function NavToggle(element) {
 		// remove focus from top nav
 		if(element) element.blur();
 	}
+
+	function setLastClickedElement(type, id) {
+		localStorage.setItem('MODX_lastClickedElement', '['+type+','+id+']' );
+	}
 	</script>
 </head>
 

manager/includes/menu.class.inc.php

@@ -58,7 +58,7 @@ function DrawSub($parentid,$level){
 				$ph['href']     = $value[3];
 				$ph['alt']      = $value[4];
 				$ph['target']   = $value[7];
-				$ph['onclick']  = 'top.tree.setLastClickedElement(0,0);'.$value[5];
+				$ph['onclick']  = 'setLastClickedElement(0,0);'.$value[5];
 				$ph['a_class']  = $this->get_a_class($id);
 				$ph['LinkAttr'] = $this->getLinkAttr($id);
 				$ph['itemName'] = $value[2] . $this->getItemName($id);