Since Perl 5.20 a POD =item cannot start with single
number, as is the case in syncToCdn POD explanation of
return codes. Using pod2* reports syntax errors like
line 228: Expected text after =item, not a number
Wrapped the numbers with POD's zero-effect markup Z<>
to fix the issue.
apparently, $session->isAdminOn is meaningless in wG8. this is a poor situation that it exists but is busted. have to do this same thing another way.
This reverts commit 1e22418757.
WebGUI.Admin.prototype.getHelperHandler was using gotoAsset(), which calls tree.goto(), which mangles the URL into being a request for
the data needed to draw the Tree view of that asset, or tries to. that works fine when navigating to an asset, but foo?func=edit is
not an asset, and the mangled URL is garbled. all of the urls (okay, the one attached to the 'edit' and 'view' entries) aren't
trying to direct the admin to an asset but instead are trying to direct it to an admin screen, which, again, cannot be shown in the
data table in Tree view. so edit and view were fixed by changing that to instead do a showView(), which flops over to the View mode
and then loads the URL without trying to mangle it.
at this point, nothing is blowing up in Doug's JS admin when testing the asset helper menu items in View or Tree mode, though
there are still some problems.
change the Lock AssetHelper to both refresh and display a message indicating that the asset is locked. the refresh is needed for tree view which displays a little icon.
Heh. Actually, the save button is a YUI styled button, and it is perfectly rendered. It also happens to have a background color exactly matching the dialog's background's. Since these buttons are usually unstyled HTML buttons, I swapped the YUI one out for that.
I also got distracted with the form rendering half way down the page and creeping down further as the hoverhelp tips come and go so I opened http://www.webgui.org/community/webgui-8/issues/12433 and then temporarily vanquished them in this form.
WebGUI.Admin.prototype.addPasteHandler: guessing but fix use of 'this' vs 'self'
currentTabName() to avoid code duplication for figuring out which context we're running in (Tree/View)
WebGUI.Admin.prototype.pasteAsset: handle pasting in the Tree view by flopping over to the View view first
WebGUI.Admin.Tree.prototype.goto: better error handling/debugging
added this FIXME to WebGUI.Admin.prototype.gotoAsset:
a lot of Tree view operations fail after this point. where the View version just directly goes to the URL, the Tree view tries to modify the URL to pass the extjs-st
