Bug in componentEditor.js

eric galant <[email protected]>
Newsgroups gmane.comp.cms.infoglue.devel
Message-ID <[email protected]>
Hi,
I use the interception point limits who can see the content notification 
info menu in edit on sight.
There are a problem for the users who don't access the content 
notification info menu.
In fact, when they click (left or right button) on page, the "content 
notification" window opens all the time.

The bug correction in the file componentEditor.js

In the function : showComponentMenu (line 458)
It should have a test on "subscribeContentDivElement" variable if it's 
null or not :

line --501----

if(!editUrl || editUrl == "")
    {
        editDivElement.style.display = "none";
        editInlineDivElement.style.display = "none";
      *  ++ if(subscribeContentDivElement) {
     *   subscribeContentDivElement.style.display = "none";*
       ++}*
    }
    else
    {
        editDivElement.style.display = "block";
        editInlineDivElement.style.display = "block";
        //alert("Registering click to:" + editUrl);
        var anEditUrl = editUrl;
        $(editDivElement).click(function () { edit(anEditUrl); });
        $(editInlineDivElement).click(function () { 
editInlineSimple(selectedRepositoryId); });
      *++ if(subscribeContentDivElement) {*
        var subscriptionUrl = componentEditorUrl + 
"Subscriptions!input.action?interceptionPointCategory=Content&entityName=org.infoglue.cms.entities.content.Content&entityId=" 
+ selectedContentId + "&extraParameters=" + selectedContentId + 
"&returnAddress=" + componentEditorUrl + 
"ViewInlineOperationMessages.action";
        $(subscribeContentDivElement).click(function () { 
openInlineDiv(subscriptionUrl, 700, 750, true); });
      * ++}*
    }

Sorry for my english....

Regards,

Eric Galant





------------------------------------------------------------------------------
Come build with us! The BlackBerry&reg; Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9&#45;12, 2009. Register now&#33;
http://p.sf.net/sfu/devconf
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.