Thanks Larry and Ede,

Seems that I missed the point.
I'll have a check with the new code.

Michaël

Le 02/04/2012 19:03, edgar.sol...@web.de a écrit :
> Larry and all,
>
> please check snapshot revision 2787.
>
> ..ede
>
>
> -------- Original Message --------
> Subject: [Jump-pilot-svn-notify] SF.net SVN: jump-pilot:[2787] core/trunk/src
> Date: Mon, 02 Apr 2012 17:01:43 +0000
> From: e...@users.sourceforge.net
> Reply-To: jump-pilot-svn-not...@lists.sourceforge.net
> To: jump-pilot-svn-not...@lists.sourceforge.net
>
> Revision: 2787
>            http://jump-pilot.svn.sourceforge.net/jump-pilot/?rev=2787&view=rev
> Author:   edso
> Date:     2012-04-02 17:01:43 +0000 (Mon, 02 Apr 2012)
> Log Message:
> -----------
> retry bugfix #3510778 EZ Button does not respect layer editability
> - revert commit 2781
> - FeatureInstaller.java: wrap MenuListener in the JumpMenuListener wrapper so 
> that EasyPanel can determine which menu items had EnableChecks [courtesy of 
> Larry Becker]
>
> Modified Paths:
> --------------
>      
> core/trunk/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
>      core/trunk/src/org/openjump/core/ui/plugin/view/EasyPanel.java
>
> Modified: 
> core/trunk/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
> ===================================================================
> --- 
> core/trunk/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
>   2012-04-02 13:14:37 UTC (rev 2786)
> +++ 
> core/trunk/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
>   2012-04-02 17:01:43 UTC (rev 2787)
> @@ -651,8 +651,9 @@
>     public void addMenuItemShownListener(final JMenuItem menuItem,
>         final MenuItemShownListener menuItemShownListener) {
>       JMenu menu = (JMenu) ((JPopupMenu) menuItem.getParent()).getInvoker();
> -    menu.addMenuListener(new MenuItemShownMenuListener(menuItem,
> -        menuItemShownListener));
> +    // wraps MenuListener in the JumpMenuListener wrapper class so that
> +    // EasyPanel can determine which menu items had EnableChecks [Larry 
> Becker]
> +    menu.addMenuListener(new JumpMenuListener(menuItemShownListener, 
> menuItem));
>     }
>
>     /**
>
> Modified: core/trunk/src/org/openjump/core/ui/plugin/view/EasyPanel.java
> ===================================================================
> --- core/trunk/src/org/openjump/core/ui/plugin/view/EasyPanel.java    
> 2012-04-02 13:14:37 UTC (rev 2786)
> +++ core/trunk/src/org/openjump/core/ui/plugin/view/EasyPanel.java    
> 2012-04-02 17:01:43 UTC (rev 2787)
> @@ -320,7 +320,7 @@
>                                                       
> ((FeatureInstaller.JumpMenuListener) menuListeners[i]).menuSelected(null);    
>                                                   
>                                               }
>                                       }
> -                             } //else {
> +                             } else {
>                                       JPopupMenu popupMenu = ((JPopupMenu) 
> jMenuItem.getParent());                    
>                                       PopupMenuListener[] listeners = 
> popupMenu.getListeners(PopupMenuListener.class);
>                                       for (int i=0; i<listeners.length; i++) {
> @@ -329,7 +329,7 @@
>                                                               
> .popupMenuWillBecomeVisible(new PopupMenuEvent(popupMenu));
>                                               }
>                                       }
> -                             //}
> +                             }
>                               SwingUtilities.invokeLater(new Runnable() {
>                                       public void run() {
>                                               if (jMenuItem.isEnabled())
>
>
>
> ------------------------------------------------------------------------------
> This SF email is sponsosred by:
> Try Windows Azure free for 90 days Click Here
> http://p.sf.net/sfu/sfd2d-msazure
> _______________________________________________
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
>


------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to