Re: [JPP-Devel] New csv driver

2012-04-02 Thread Michaël Michaud

Thanks Arnd,

I escaped accented characters and uploaded the extension again as
csv-driver-0.6.0beta3.jar

Michaël


Hallo Michaël,

here is the german language file for You new csv driver.

Arnd


Hi Arnd,

Hi uploaded the internationalized plugin at the same url
Here attached is the english i18n properties file

Thanks,

Michaël


Le 29/03/2012 11:58, Arnd Kielhorn a écrit :

Hello Michael,

I could make the german translation of the dialog.

Arnd


Hi,

thank You for that nice driver.
Is there already an englsih and german translation for it because when

I

load OJ the csv driver dialog is in french?
Sorry, forgot it. I'll send the internationalized version by the end of
the week (english only).

Michaël

Arnd



Hi,

I uploaded a beta version of a new csv driver : csv-driver-0.6

beta.jar

http://geo.michaelm.free.fr/OpenJUMP/resources/drivers/

This is a complete rewrite of my old txt driver.
It should be more flexible and currently includes
- an auto-mode able to guess the structure of most character

delimited

files (including pirol formatted files)
- a set-options mode where the user has to set the file structure to
help the driver to read the file correctly
- a writer with only a few options
I've not yet implemented anything to read csvt files from OGR, but

tried

to make the driver
flexible enough to make it possible.

At the same address, you'll find two documents to help you understand
the options :
OpenJUMP_CSV.odt




OpenJUMP_CSV_auto.odt




It is quite difficult to make such a driver robust and it is in its
early stage.
All your tests and remarks are welcome.

Michaël








--

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


--
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


Re: [JPP-Devel] New csv driver

2012-04-02 Thread Rahkonen Jukka
Hi,

Finnish translation attached. I have also a few comments.

- All multi word menu texts contain dashes on my computer instead of space, 
like this
"Kommenttirivin-rakenne-(regexp)"
- When saving without giving file extension the result is a bit odd "New.csv, 
txt, xyz"
- "Inverse selection" -button cannot be translated
- Where is the string "queted", English translation "Queted string" used?
- Canceling saving from the optional "Select attributes" menu leads to NPE
java.lang.NullPointerException
 at 
fr.michaelm.jump.drivers.csv.CSVDataSource$1.executeUpdate(CSVDataSource.java:184)
 at 
com.vividsolutions.jump.workbench.datasource.AbstractSaveDatasetAsPlugIn.run(AbstractSaveDatasetAsPlugIn.java:33)
 at 
com.vividsolutions.jump.workbench.ui.task.TaskMonitorManager$TaskWrapper.run(TaskMonitorManager.java:152)
 at java.lang.Thread.run(Unknown Source)
-Jukka Rahkonen-

Michaël Michaud  wrote:
Thanks Arnd,

I escaped accented characters and uploaded the extension again as
csv-driver-0.6.0beta3.jar

Michaël


Hallo Michaël,

here is the german language file for You new csv driver.

Arnd



Hi Arnd,

Hi uploaded the internationalized plugin at the same url
Here attached is the english i18n properties file

Thanks,

Michaël


Le 29/03/2012 11:58, Arnd Kielhorn a écrit :


Hello Michael,

I could make the german translation of the dialog.

Arnd



Hi,


thank You for that nice driver.
Is there already an englsih and german translation for it because when


I


load OJ the csv driver dialog is in french?
Sorry, forgot it. I'll send the internationalized version by the end of
the week (english only).

Michaël


Arnd




Hi,

I uploaded a beta version of a new csv driver : csv-driver-0.6


beta.jar


http://geo.michaelm.free.fr/OpenJUMP/resources/drivers/

This is a complete rewrite of my old txt driver.
It should be more flexible and currently includes
- an auto-mode able to guess the structure of most character


delimited


files (including pirol formatted files)
- a set-options mode where the user has to set the file structure to
help the driver to read the file correctly
- a writer with only a few options
I've not yet implemented anything to read csvt files from OGR, but


tried


to make the driver
flexible enough to make it possible.

At the same address, you'll find two documents to help you understand
the options :
OpenJUMP_CSV.odt






OpenJUMP_CSV_auto.odt






It is quite difficult to make such a driver robust and it is in its
early stage.
All your tests and remarks are welcome.

Michaël










--


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




csv_fi.properties
Description: csv_fi.properties
--
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


Re: [JPP-Devel] Combining polygons

2012-04-02 Thread Rahkonen Jukka
Hi,

I had a try with rev. 2785 and "Combine two polygons" attached to EZ Buttons so 
that the layer was editable at the time of shortcut creation.  For me the EZ 
Button connected tool still combines selected polygons whether the layer is 
editable or not. I tried with a few different ways.
- By toggling the Editable setting of a layer without closing the file in 
between.
- Saving the editable layer on disk and opening it as non-editable
- By creating new layers, adding a few polygons, making then non-editable and 
using the EZ Button connected combine tool.


-Jukka Rahkonen-

Michaël Michaud wrote:
> 
> Hi Jukka, Larry
> > - "Combine two selected polygons tool" is fast but it has a 
> little bug. It allows to combine polygons even if the layer 
> is non-editable.
> >> How do you do that ? Menu item is inactive (grayed) if 
> layer is non editable.
> > Good question indeed. It took awhile to repeat it. This is 
> somehow related to EZ-Buttons shortcut. The shortcut seems to 
> remember, not only the menu selection it is connected with, 
> but also the state of OJ project when the shortcut was 
> created. It can be created by opening some layer to OJ and 
> making is editable so thet combine two polygons is working. 
> Create the EZ-Button shortcut now and add a new layer. Now 
> the EZ-Buttons shortcut combines polygons even the layer is 
> not editable. So this is a feature in the EZ-Buttons.
> I tried to fixed this problem.
> 
> Jukka, could you have a look to check it solves your problem 
> (from svn 2781)
> Larry, could you double check the small change I did to the 
> EasyPanel class
> http://jump-pilot.svn.sourceforge.net/viewvc/jump-pilot/core/t
runk/src/org/openjump/core/ui/plugin/view/EasyPanel.java?r1=1873&r2=> 
2781&sortby=date
> 
> Thanks,
> 
> Michaël
> 
> 
> 
> > - "Combine polygon with neighbouring polygon" is very slow 
> and can give an out-of-memory error. Is it perhaps walking 
> through all the features when searching for the one to use 
> for combining? I would guess that it should be much faster if 
> it correctly selects just the few adjacent polygons for 
> candidates first.
> >> I think the algo of this plugin is not so simple... I 
> think Stefan know it better
> > - Making union of selected features suits me well because I 
> would like to combine about 10-20 polygons together. However, 
> because it creates new layer each time there is some extra 
> work with joining them together later.
> > - "Union selected in place (if result can be simple 
> polygon)" would be the ideal tool for just this case.
> >> Agree, this would be helpful. Not sure I would limit to 
> simple polygons though.
> > In my case I want that the new areas are also continuos. 
> Sometimes multipolygons would be acceptable. Perhaps there 
> could be a check box for cases when features selected for 
> combining are all simple polygons "Allow multipolygons as new 
> features? ". If selection already contains multipolygons then 
> the result can naturally contain them too.
> >
> >> I suppose you don't want to union all adjacent polygons, 
> otherwise, the simplest is to union the whole layer, then to 
> explode the multipolygon.
> > Yes, this is the case.  All the polygons in the layer are 
> adjacent and all together would be a monster polygon with 
> thousands of holes and hundreds of thousands of vertices. I 
> am going to reduce the number of polygons into about 500 and 
> I want all them to have some reasonable amount of vertices.  
> Automatic system by using some grid layer as a helper layer 
> for splitting data into rectangular pieces first to be 
> unioned later tile by tile would not give directly a good 
> result. Data is from archipelago and in some places there is 
> just open sea and in other places masses of islands. I will 
> get the job ready faster by doing the selections manually. 
> "Union selected in place" tool would make it more convenient, though.
> >
> >
> >> Here is a workaround (didn't check it though)
> >> - create a second layer.
> >> - draw 1 linestring per group of building, intersecting 
> all buildings you want to aggregate (and only those one)
> >> - create an auto-increment attribute on these linestring layer
> >> - do a spatial join to transfer linestring id to polygons
> >> - union polygons by attribute
> >> (you will get a multipolygon with all polygons not 
> intersecting a linestring : explode it)
> > Sounds doable, I will try it next week.
> >
> > -Jukka-
> >> Michaël
> >
> >
> >
> > -Jukka Rahkonen-
> >
> >
> >
> >
> >
> > 
> --
> 
> > 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

Re: [JPP-Devel] [ jump-pilot-Bugs-3513864 ] Left click on RasterImageLayer name throws exceptions

2012-04-02 Thread Rahkonen Jukka
Hi,

There is still one little oddness left. For the Sextante raster layers the 
toggling of visibility works by doing left click on the layer name. What looks 
like a check box is not active.

-Jukka Rahkonen-

> -Alkuperäinen viesti-
> Lähettäjä: SourceForge.net [mailto:nore...@sourceforge.net] 
> Lähetetty: 1. huhtikuuta 2012 15:17
> Vastaanottaja: SourceForge.net
> Aihe: [JPP-Devel] [ jump-pilot-Bugs-3513864 ] Left click on 
> RasterImageLayer name throws exceptions
> 
> Bugs item #3513864, was opened at 2012-04-01 01:21
> Message generated for change (Comment added) made by michaudm
> You can respond by visiting: 
> https://sourceforge.net/tracker/?func=detail&atid=679906&aid=3
513864&group_id=118054
> 
> Please note that this message will contain a full copy of the 
> comment thread,
> including the initial issue submission, for this request,
> not just the latest update.
> Category: OpenJUMP - Menu - File 
> Group: None
> >Status: Closed
> >Resolution: Fixed
> Priority: 5
> Private: No
> Submitted By: michael michaud (michaudm)
> Assigned to: Nobody/Anonymous (nobody)
> Summary: Left click on RasterImageLayer name throws exceptions
> 
> Initial Comment:
> Load a RasterImageLayer
> Left click on the layer name
> Following stacktrace is thrown to the console :
> 
> Exception in thread "AWT-EventQueue-0" 
> java.lang.ArrayIndexOutOfBoundsException:
>  0
> at 
> com.vividsolutions.jump.workbench.ui.LayerNameRenderer.getColumnWidth
> (LayerNameRenderer.java:134)
> at 
> com.vividsolutions.jump.workbench.ui.LayerNameRenderer.getCheckBoxBou
> nds(LayerNameRenderer.java:123)
> at 
> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel.handleCheckBo
> xClick(TreeLayerNamePanel.java:519)
> at 
> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel.access$000(Tr
> eeLayerNamePanel.java:94)
> at 
> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel$3.mouseClicke
> d(TreeLayerNamePanel.java:184)
> at 
> java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:25
> 3)
> at 
> java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:25
> 2)
> at java.awt.Component.processMouseEvent(Component.java:6293)
> at 
> javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
> at java.awt.Component.processEvent(Component.java:6055)
> at java.awt.Container.processEvent(Container.java:2039)
> at java.awt.Component.dispatchEventImpl(Component.java:4653)
> at java.awt.Container.dispatchEventImpl(Container.java:2097)
> at java.awt.Component.dispatchEvent(Component.java:4481)
> at 
> java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4575
> )
> at 
> java.awt.LightweightDispatcher.processMouseEvent(Container.java:4245)
> 
> at 
> java.awt.LightweightDispatcher.dispatchEvent(Container.java:4166)
> at java.awt.Container.dispatchEventImpl(Container.java:2083)
> at java.awt.Window.dispatchEventImpl(Window.java:2482)
> at java.awt.Component.dispatchEvent(Component.java:4481)
> at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:648)
> at java.awt.EventQueue.access$000(EventQueue.java:84)
> at java.awt.EventQueue$1.run(EventQueue.java:607)
> at java.awt.EventQueue$1.run(EventQueue.java:605)
> at java.security.AccessController.doPrivileged(Native Method)
> at 
> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
> ntrolContext.java:87)
> at 
> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
> ntrolContext.java:98)
> at java.awt.EventQueue$2.run(EventQueue.java:621)
> at java.awt.EventQueue$2.run(EventQueue.java:619)
> at java.security.AccessController.doPrivileged(Native Method)
> at 
> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
> ntrolContext.java:87)
> at java.awt.EventQueue.dispatchEvent(EventQueue.java:618)
> at 
> java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThre
> ad.java:269)
> at 
> java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.
> java:184)
> at 
> java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThre
> ad.java:174)
> at 
> java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
> 
> at 
> java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
> 
> at 
> java.awt.EventDispatchThread.run(EventDispatchThread.java:122)
> 
> 
> --
> 
> >Comment By: michael michaud (michaudm)
> Date: 2012-04-01 05:17
> 
> Message:
> Fixed in svn 2785
> The problem happened only in project with only RasterImageLayer
> RasterImageLayer name was not displayed as other layerables 
> because it was
> not enumerated in the possible Layerable implementation. The missing
> che

Re: [JPP-Devel] Selectable and Editable menu items won't check on Vista and Win7

2012-04-02 Thread edgar . soldin
On 31.03.2012 16:29, Michaël Michaud wrote:
> Hi,
>> we could disable the icons conditionally for windows laf and win7+ as other 
>> lafs and platforms are not hit.
> Agree if you do it ;-)

done

> I will open a bug report which will stay opened as long as the bug is 
> not fixed in jre.

beat you to it ;)

> I had a quick look in the menus and noticed also
> - a icon + check symbol in category context menu (change visibility)
> - a icon + check symbol for View > Edit toolbox menuitem
> I suppose they are also concerned by the bug, which means there are at 
> least 4 menu-items to fix.
> 

it's worked around generically where the menuitems are generated. i also did 
some housekeeping while at it. here's the commit message

Revision: 2786
  http://jump-pilot.svn.sourceforge.net/jump-pilot/?rev=2786&view=rev
Author:   edso
Date: 2012-04-02 13:14:37 + (Mon, 02 Apr 2012)
Log Message:
---
bugfix:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7122141

http://www.mail-archive.com/jump-pilot-devel@lists.sourceforge.net/msg10601.html

com.vividsolutions.jump.workbench.ui.plugin.FeatureInstaller
- unified spaces/tabs, general reformatting
- privatized internally used behead method
- deprecated legacy methods addLayerViewMenuItem(), addLayerNameViewMenuItem()
- added replacement method for deprecated addMainMenuItem to add plain plugins: 
   addMainMenuPlugin( PlugIn, menuPath, menuItemName, checkBox, icon, 
enableCheck, pos), 
  former changes by michael added methods for JMenuItem and AbstractUiPlugIn 
only
- added addMenuItemIcon() method used by all methods that add menu entries to 
popup/mainmenu
  it contains a workaround for the windows laf missing checkmark ticks for 
checkbox entries on vista and later
- cleaned up addPopupMenuItem() method that duplicated routines
- commented (pending deletion) addMainMenuItemWithJava14Fix() methods
  classes using the method above modified to use addMainMenuItem()

..ede

--
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


Re: [JPP-Devel] Combining polygons

2012-04-02 Thread Larry Becker
Hi Michaël and Jukka,

  It looks to me like one of the many revisions to FeatureInstaller lost an
important change to support EZ Buttons.  The patch below restores it.  The
change to EasyPanel should not be necessary.  Please excuse me if my
superficial look at the code has missed an important detail.

Larry


Index:
C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
===
---
C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
(revision 2786)
+++
C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
(working copy)
@@ -649,12 +649,12 @@
   }

   public void addMenuItemShownListener(final JMenuItem menuItem,
-  final MenuItemShownListener menuItemShownListener) {
-JMenu menu = (JMenu) ((JPopupMenu) menuItem.getParent()).getInvoker();
-menu.addMenuListener(new MenuItemShownMenuListener(menuItem,
-menuItemShownListener));
+  final MenuItemShownListener menuItemShownListener) {
+  JMenu menu = (JMenu) ((JPopupMenu)
menuItem.getParent()).getInvoker();
+  menu.addMenuListener(new JumpMenuListener(
menuItemShownListener,menuItem));
   }

+
   /**
* Convenience method for entries without menupath
*


On Mon, Apr 2, 2012 at 2:41 AM, Rahkonen Jukka wrote:

> Hi,
>
> I had a try with rev. 2785 and "Combine two polygons" attached to EZ
> Buttons so that the layer was editable at the time of shortcut creation.
>  For me the EZ Button connected tool still combines selected polygons
> whether the layer is editable or not. I tried with a few different ways.
> - By toggling the Editable setting of a layer without closing the file in
> between.
> - Saving the editable layer on disk and opening it as non-editable
> - By creating new layers, adding a few polygons, making then non-editable
> and using the EZ Button connected combine tool.
>
>
> -Jukka Rahkonen-
>
> Michaël Michaud wrote:
> >
> > Hi Jukka, Larry
> > > - "Combine two selected polygons tool" is fast but it has a
> > little bug. It allows to combine polygons even if the layer
> > is non-editable.
> > >> How do you do that ? Menu item is inactive (grayed) if
> > layer is non editable.
> > > Good question indeed. It took awhile to repeat it. This is
> > somehow related to EZ-Buttons shortcut. The shortcut seems to
> > remember, not only the menu selection it is connected with,
> > but also the state of OJ project when the shortcut was
> > created. It can be created by opening some layer to OJ and
> > making is editable so thet combine two polygons is working.
> > Create the EZ-Button shortcut now and add a new layer. Now
> > the EZ-Buttons shortcut combines polygons even the layer is
> > not editable. So this is a feature in the EZ-Buttons.
> > I tried to fixed this problem.
> >
> > Jukka, could you have a look to check it solves your problem
> > (from svn 2781)
> > Larry, could you double check the small change I did to the
> > EasyPanel class
> > http://jump-pilot.svn.sourceforge.net/viewvc/jump-pilot/core/t
> runk/src/org/openjump/core/ui/plugin/view/EasyPanel.java?r1=1873&r2=>
> 2781&sortby=date
> >
> > Thanks,
> >
> > Michaël
> >
> >
> >
> > > - "Combine polygon with neighbouring polygon" is very slow
> > and can give an out-of-memory error. Is it perhaps walking
> > through all the features when searching for the one to use
> > for combining? I would guess that it should be much faster if
> > it correctly selects just the few adjacent polygons for
> > candidates first.
> > >> I think the algo of this plugin is not so simple... I
> > think Stefan know it better
> > > - Making union of selected features suits me well because I
> > would like to combine about 10-20 polygons together. However,
> > because it creates new layer each time there is some extra
> > work with joining them together later.
> > > - "Union selected in place (if result can be simple
> > polygon)" would be the ideal tool for just this case.
> > >> Agree, this would be helpful. Not sure I would limit to
> > simple polygons though.
> > > In my case I want that the new areas are also continuos.
> > Sometimes multipolygons would be acceptable. Perhaps there
> > could be a check box for cases when features selected for
> > combining are all simple polygons "Allow multipolygons as new
> > features? ". If selection already contains multipolygons then
> > the result can naturally contain them too.
> > >
> > >> I suppose you don't want to union all adjacent polygons,
> > otherwise, the simplest is to union the whole layer, then to
> > explode the multipolygon.
> > > Yes, this is the case.  All the polygons in the layer are
> > adjacent and all together would be a monster polygon with
> > thousands of holes and hundreds of thousands of vertices. I
> > am going to reduce the number of polygons into about 500 and
> > I want all them to have some re

Re: [JPP-Devel] Combining polygons

2012-04-02 Thread edgar . soldin
Larry,

could you please elaborate in short what this change does on the code level?

..thanks ede

On 02.04.2012 16:53, Larry Becker wrote:
> Hi Michaël and Jukka,
> 
>   It looks to me like one of the many revisions to FeatureInstaller lost an
> important change to support EZ Buttons.  The patch below restores it.  The
> change to EasyPanel should not be necessary.  Please excuse me if my
> superficial look at the code has missed an important detail.
> 
> Larry
> 
> 
> Index:
> C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
> ===
> ---
> C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
> (revision 2786)
> +++
> C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
> (working copy)
> @@ -649,12 +649,12 @@
>}
> 
>public void addMenuItemShownListener(final JMenuItem menuItem,
> -  final MenuItemShownListener menuItemShownListener) {
> -JMenu menu = (JMenu) ((JPopupMenu) menuItem.getParent()).getInvoker();
> -menu.addMenuListener(new MenuItemShownMenuListener(menuItem,
> -menuItemShownListener));
> +  final MenuItemShownListener menuItemShownListener) {
> +  JMenu menu = (JMenu) ((JPopupMenu)
> menuItem.getParent()).getInvoker();
> +  menu.addMenuListener(new JumpMenuListener(
> menuItemShownListener,menuItem));
>}
> 
> +
>/**
> * Convenience method for entries without menupath
> *
> 
> 
> On Mon, Apr 2, 2012 at 2:41 AM, Rahkonen Jukka 
> wrote:
> 
>> Hi,
>>
>> I had a try with rev. 2785 and "Combine two polygons" attached to EZ
>> Buttons so that the layer was editable at the time of shortcut creation.
>>  For me the EZ Button connected tool still combines selected polygons
>> whether the layer is editable or not. I tried with a few different ways.
>> - By toggling the Editable setting of a layer without closing the file in
>> between.
>> - Saving the editable layer on disk and opening it as non-editable
>> - By creating new layers, adding a few polygons, making then non-editable
>> and using the EZ Button connected combine tool.
>>
>>
>> -Jukka Rahkonen-
>>
>> Michaël Michaud wrote:
>>>
>>> Hi Jukka, Larry
 - "Combine two selected polygons tool" is fast but it has a
>>> little bug. It allows to combine polygons even if the layer
>>> is non-editable.
> How do you do that ? Menu item is inactive (grayed) if
>>> layer is non editable.
 Good question indeed. It took awhile to repeat it. This is
>>> somehow related to EZ-Buttons shortcut. The shortcut seems to
>>> remember, not only the menu selection it is connected with,
>>> but also the state of OJ project when the shortcut was
>>> created. It can be created by opening some layer to OJ and
>>> making is editable so thet combine two polygons is working.
>>> Create the EZ-Button shortcut now and add a new layer. Now
>>> the EZ-Buttons shortcut combines polygons even the layer is
>>> not editable. So this is a feature in the EZ-Buttons.
>>> I tried to fixed this problem.
>>>
>>> Jukka, could you have a look to check it solves your problem
>>> (from svn 2781)
>>> Larry, could you double check the small change I did to the
>>> EasyPanel class
>>> http://jump-pilot.svn.sourceforge.net/viewvc/jump-pilot/core/t
>> runk/src/org/openjump/core/ui/plugin/view/EasyPanel.java?r1=1873&r2=>
>> 2781&sortby=date
>>>
>>> Thanks,
>>>
>>> Michaël
>>>
>>>
>>>
 - "Combine polygon with neighbouring polygon" is very slow
>>> and can give an out-of-memory error. Is it perhaps walking
>>> through all the features when searching for the one to use
>>> for combining? I would guess that it should be much faster if
>>> it correctly selects just the few adjacent polygons for
>>> candidates first.
> I think the algo of this plugin is not so simple... I
>>> think Stefan know it better
 - Making union of selected features suits me well because I
>>> would like to combine about 10-20 polygons together. However,
>>> because it creates new layer each time there is some extra
>>> work with joining them together later.
 - "Union selected in place (if result can be simple
>>> polygon)" would be the ideal tool for just this case.
> Agree, this would be helpful. Not sure I would limit to
>>> simple polygons though.
 In my case I want that the new areas are also continuos.
>>> Sometimes multipolygons would be acceptable. Perhaps there
>>> could be a check box for cases when features selected for
>>> combining are all simple polygons "Allow multipolygons as new
>>> features? ". If selection already contains multipolygons then
>>> the result can naturally contain them too.

> I suppose you don't want to union all adjacent polygons,
>>> otherwise, the simplest is to union the whole layer, then to
>>> explode the multipolygon.
 Yes, this is the case.  All the polygons in the la

Re: [JPP-Devel] Combining polygons

2012-04-02 Thread Larry Becker
Sure.  It wraps MenuListener in the JumpMenuListener wrapper class so that
EasyPanel can determine which menu items had EnableChecks.  It has been a
few years since I wrote this so I'm not sure of all the hows and whys that
it was done this way.  8-)

Larry

On Mon, Apr 2, 2012 at 10:02 AM,  wrote:

> Larry,
>
> could you please elaborate in short what this change does on the code
> level?
>
> ..thanks ede
>
> On 02.04.2012 16:53, Larry Becker wrote:
> > Hi Michaël and Jukka,
> >
> >   It looks to me like one of the many revisions to FeatureInstaller lost
> an
> > important change to support EZ Buttons.  The patch below restores it.
>  The
> > change to EasyPanel should not be necessary.  Please excuse me if my
> > superficial look at the code has missed an important detail.
> >
> > Larry
> >
> >
> > Index:
> >
> C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
> > ===
> > ---
> >
> C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
> > (revision 2786)
> > +++
> >
> C:/eclipse3/workspace/OpenJumpSvn/src/com/vividsolutions/jump/workbench/ui/plugin/FeatureInstaller.java
> > (working copy)
> > @@ -649,12 +649,12 @@
> >}
> >
> >public void addMenuItemShownListener(final JMenuItem menuItem,
> > -  final MenuItemShownListener menuItemShownListener) {
> > -JMenu menu = (JMenu) ((JPopupMenu)
> menuItem.getParent()).getInvoker();
> > -menu.addMenuListener(new MenuItemShownMenuListener(menuItem,
> > -menuItemShownListener));
> > +  final MenuItemShownListener menuItemShownListener) {
> > +  JMenu menu = (JMenu) ((JPopupMenu)
> > menuItem.getParent()).getInvoker();
> > +  menu.addMenuListener(new JumpMenuListener(
> > menuItemShownListener,menuItem));
> >}
> >
> > +
> >/**
> > * Convenience method for entries without menupath
> > *
> >
> >
> > On Mon, Apr 2, 2012 at 2:41 AM, Rahkonen Jukka <
> jukka.rahko...@mmmtike.fi>wrote:
> >
> >> Hi,
> >>
> >> I had a try with rev. 2785 and "Combine two polygons" attached to EZ
> >> Buttons so that the layer was editable at the time of shortcut creation.
> >>  For me the EZ Button connected tool still combines selected polygons
> >> whether the layer is editable or not. I tried with a few different ways.
> >> - By toggling the Editable setting of a layer without closing the file
> in
> >> between.
> >> - Saving the editable layer on disk and opening it as non-editable
> >> - By creating new layers, adding a few polygons, making then
> non-editable
> >> and using the EZ Button connected combine tool.
> >>
> >>
> >> -Jukka Rahkonen-
> >>
> >> Michaël Michaud wrote:
> >>>
> >>> Hi Jukka, Larry
>  - "Combine two selected polygons tool" is fast but it has a
> >>> little bug. It allows to combine polygons even if the layer
> >>> is non-editable.
> > How do you do that ? Menu item is inactive (grayed) if
> >>> layer is non editable.
>  Good question indeed. It took awhile to repeat it. This is
> >>> somehow related to EZ-Buttons shortcut. The shortcut seems to
> >>> remember, not only the menu selection it is connected with,
> >>> but also the state of OJ project when the shortcut was
> >>> created. It can be created by opening some layer to OJ and
> >>> making is editable so thet combine two polygons is working.
> >>> Create the EZ-Button shortcut now and add a new layer. Now
> >>> the EZ-Buttons shortcut combines polygons even the layer is
> >>> not editable. So this is a feature in the EZ-Buttons.
> >>> I tried to fixed this problem.
> >>>
> >>> Jukka, could you have a look to check it solves your problem
> >>> (from svn 2781)
> >>> Larry, could you double check the small change I did to the
> >>> EasyPanel class
> >>> http://jump-pilot.svn.sourceforge.net/viewvc/jump-pilot/core/t
> >> runk/src/org/openjump/core/ui/plugin/view/EasyPanel.java?r1=1873&r2=>
> >> 2781&sortby=date
> >>>
> >>> Thanks,
> >>>
> >>> Michaël
> >>>
> >>>
> >>>
>  - "Combine polygon with neighbouring polygon" is very slow
> >>> and can give an out-of-memory error. Is it perhaps walking
> >>> through all the features when searching for the one to use
> >>> for combining? I would guess that it should be much faster if
> >>> it correctly selects just the few adjacent polygons for
> >>> candidates first.
> > I think the algo of this plugin is not so simple... I
> >>> think Stefan know it better
>  - Making union of selected features suits me well because I
> >>> would like to combine about 10-20 polygons together. However,
> >>> because it creates new layer each time there is some extra
> >>> work with joining them together later.
>  - "Union selected in place (if result can be simple
> >>> polygon)" would be the ideal tool for just this case.
> > Agree, this would be helpful. Not sure I would limit to
> >>> simple polygons though.
>  In my case I want

Re: [JPP-Devel] New csv driver

2012-04-02 Thread Arnd Kielhorn
I am sorry for that but I made a mistake in my translation.
Attached You'll find the corrected german language file.

Arnd

> Thanks Arnd,
> 
> I escaped accented characters and uploaded the extension again as
> csv-driver-0.6.0beta3.jar
> 
> Michaël
> 
> > Hallo Michaël,
> >
> > here is the german language file for You new csv driver.
> >
> > Arnd
> >
> >> Hi Arnd,
> >>
> >> Hi uploaded the internationalized plugin at the same url
> >> Here attached is the english i18n properties file
> >>
> >> Thanks,
> >>
> >> Michaël
> >>
> >>
> >> Le 29/03/2012 11:58, Arnd Kielhorn a écrit :
> >>> Hello Michael,
> >>>
> >>> I could make the german translation of the dialog.
> >>>
> >>> Arnd
> >>>
>  Hi,
> > thank You for that nice driver.
> > Is there already an englsih and german translation for it because
> when
> >> I
>  load OJ the csv driver dialog is in french?
>  Sorry, forgot it. I'll send the internationalized version by the end
> of
>  the week (english only).
> 
>  Michaël
> > Arnd
> >
> >
> >> Hi,
> >>
> >> I uploaded a beta version of a new csv driver : csv-driver-0.6
> >> beta.jar
> >> http://geo.michaelm.free.fr/OpenJUMP/resources/drivers/
> >>
> >> This is a complete rewrite of my old txt driver.
> >> It should be more flexible and currently includes
> >> - an auto-mode able to guess the structure of most character
> >> delimited
> >> files (including pirol formatted files)
> >> - a set-options mode where the user has to set the file structure
> to
> >> help the driver to read the file correctly
> >> - a writer with only a few options
> >> I've not yet implemented anything to read csvt files from OGR, but
>  tried
> >> to make the driver
> >> flexible enough to make it possible.
> >>
> >> At the same address, you'll find two documents to help you
> understand
> >> the options :
> >> OpenJUMP_CSV.odt
> >>
> >>
> 
> >> OpenJUMP_CSV_auto.odt
> >>
> >>
> 
> >> It is quite difficult to make such a driver robust and it is in its
> >> early stage.
> >> All your tests and remarks are welcome.
> >>
> >> Michaël
> >>
> >>
> >>
> >>
> >>
> 
> >>
> --
>  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
> 

-- 
Empfehlen Sie GMX DSL Ihren Freunden und Bekannten und wir
belohnen Sie mit bis zu 50,- Euro! https://freundschaftswerbung.gmx.de


csv_de.properties
Description: Binary data
--
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


Re: [JPP-Devel] Combining polygons

2012-04-02 Thread edgar . soldin
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 +
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 + (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; ihttp://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


Re: [JPP-Devel] New csv driver

2012-04-02 Thread Michaël Michaud

Hi Jukka,

Thanks for the tests,

Finnish translation attached. I have also a few comments.

I'll add finnish translation, thanks,
- All multi word menu texts contain dashes on my computer instead of 
space, like this

"Kommenttirivin-rakenne-(regexp)"
Did not investigate into that yet. This is changed into dashes by the IO 
framework I used and I
think it has been done to safely use these labels as keys. But I'm not 
sure why labels containing

whitespaces would be a problem.
- When saving without giving file extension the result is a bit odd 
"New.csv, txt, xyz"
Oh, not nice. I must check if it does the same thing for other 
multi-extension

- "Inverse selection" -button cannot be translated

OK, will add it

- Where is the string "queted", English translation "Queted string" used?
Unused. Finally, the parser should know how to parse quoted or unquoted 
fields.
It should write quotes only for records and fields containing a quote or 
a separator, but

this is a place where more tests would be useful.

- Canceling saving from the optional "Select attributes" menu leads to NPE

Oh, very bad.
I hope I'll be able to have a look into all your points by the end of 
the week.
(I also had a look on csvt file from OGR, but I will first consolidate 
current options)


Thanks,

Michaël

java.lang.NullPointerException
 at 
fr.michaelm.jump.drivers.csv.CSVDataSource$1.executeUpdate(CSVDataSource.java:184)
 at 
com.vividsolutions.jump.workbench.datasource.AbstractSaveDatasetAsPlugIn.run(AbstractSaveDatasetAsPlugIn.java:33)
 at 
com.vividsolutions.jump.workbench.ui.task.TaskMonitorManager$TaskWrapper.run(TaskMonitorManager.java:152)

 at java.lang.Thread.run(Unknown Source)
-Jukka Rahkonen-


Michaël Michaud  wrote:
Thanks Arnd,

I escaped accented characters and uploaded the extension again as
csv-driver-0.6.0beta3.jar

Michaël


Hallo Michaël,

here is the german language file for You new csv driver.

Arnd


Hi Arnd,

Hi uploaded the internationalized plugin at the same url
Here attached is the english i18n properties file

Thanks,

Michaël


Le 29/03/2012 11:58, Arnd Kielhorn a écrit :

Hello Michael,

I could make the german translation of the dialog.

Arnd


Hi,

thank You for that nice driver.
Is there already an englsih and german translation for it because when

I

load OJ the csv driver dialog is in french?
Sorry, forgot it. I'll send the internationalized version by the end of
the week (english only).

Michaël

Arnd



Hi,

I uploaded a beta version of a new csv driver : csv-driver-0.6

beta.jar

http://geo.michaelm.free.fr/OpenJUMP/resources/drivers/

This is a complete rewrite of my old txt driver.
It should be more flexible and currently includes
- an auto-mode able to guess the structure of most character

delimited

files (including pirol formatted files)
- a set-options mode where the user has to set the file structure to
help the driver to read the file correctly
- a writer with only a few options
I've not yet implemented anything to read csvt files from OGR, but

tried

to make the driver
flexible enough to make it possible.

At the same address, you'll find two documents to help you understand
the options :
OpenJUMP_CSV.odt




OpenJUMP_CSV_auto.odt





It is quite difficult to make such a driver robust and it is in its
early stage.
All your tests and remarks are welcome.

Michaël







--

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




--
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


--

Re: [JPP-Devel] Selectable and Editable menu items won't check on Vista and Win7

2012-04-02 Thread Michaël Michaud
Hi,

Nice work Ede, thanks a lot.

Michaël
> On 31.03.2012 16:29, Michaël Michaud wrote:
>> Hi,
>>> we could disable the icons conditionally for windows laf and win7+ as other 
>>> lafs and platforms are not hit.
>> Agree if you do it ;-)
> done
>
>> I will open a bug report which will stay opened as long as the bug is
>> not fixed in jre.
> beat you to it ;)
>
>> I had a quick look in the menus and noticed also
>> - a icon + check symbol in category context menu (change visibility)
>> - a icon + check symbol for View>  Edit toolbox menuitem
>> I suppose they are also concerned by the bug, which means there are at
>> least 4 menu-items to fix.
>>
> it's worked around generically where the menuitems are generated. i also did 
> some housekeeping while at it. here's the commit message
>
> Revision: 2786
>http://jump-pilot.svn.sourceforge.net/jump-pilot/?rev=2786&view=rev
> Author:   edso
> Date: 2012-04-02 13:14:37 + (Mon, 02 Apr 2012)
> Log Message:
> ---
> bugfix:
>   http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7122141
>   
> http://www.mail-archive.com/jump-pilot-devel@lists.sourceforge.net/msg10601.html
>
> com.vividsolutions.jump.workbench.ui.plugin.FeatureInstaller
> - unified spaces/tabs, general reformatting
> - privatized internally used behead method
> - deprecated legacy methods addLayerViewMenuItem(), addLayerNameViewMenuItem()
> - added replacement method for deprecated addMainMenuItem to add plain 
> plugins:
> addMainMenuPlugin( PlugIn, menuPath, menuItemName, checkBox, icon, 
> enableCheck, pos),
>former changes by michael added methods for JMenuItem and AbstractUiPlugIn 
> only
> - added addMenuItemIcon() method used by all methods that add menu entries to 
> popup/mainmenu
>it contains a workaround for the windows laf missing checkmark ticks for 
> checkbox entries on vista and later
> - cleaned up addPopupMenuItem() method that duplicated routines
> - commented (pending deletion) addMainMenuItemWithJava14Fix() methods
>classes using the method above modified to use addMainMenuItem()
>
> ..ede
>
> --
> 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


Re: [JPP-Devel] Combining polygons

2012-04-02 Thread Michaël Michaud
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 +
> 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 + (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 @@ -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


Re: [JPP-Devel] [ jump-pilot-Bugs-3513864 ] Left click on RasterImageLayer name throws exceptions

2012-04-02 Thread Michaël Michaud
Hi,
> There is still one little oddness left. For the Sextante raster layers the 
> toggling of visibility works by doing left click on the layer name. What 
> looks like a check box is not active.
Checkbox is slightly shifted on the left because of the missing color 
square, but user click is active where the checkbox should be (on the 
right of the actual checkbox) not where it really is.
Is this what you mean ?
I'll try to add a symbol for Sextante raster, so that all check boxes 
are vertically aligned.

Michaël
> -Jukka Rahkonen-
>
>> -Alkuperäinen viesti-
>> Lähettäjä: SourceForge.net [mailto:nore...@sourceforge.net]
>> Lähetetty: 1. huhtikuuta 2012 15:17
>> Vastaanottaja: SourceForge.net
>> Aihe: [JPP-Devel] [ jump-pilot-Bugs-3513864 ] Left click on
>> RasterImageLayer name throws exceptions
>>
>> Bugs item #3513864, was opened at 2012-04-01 01:21
>> Message generated for change (Comment added) made by michaudm
>> You can respond by visiting:
>> https://sourceforge.net/tracker/?func=detail&atid=679906&aid=3
> 513864&group_id=118054
>> Please note that this message will contain a full copy of the
>> comment thread,
>> including the initial issue submission, for this request,
>> not just the latest update.
>> Category: OpenJUMP - Menu - File
>> Group: None
>>> Status: Closed
>>> Resolution: Fixed
>> Priority: 5
>> Private: No
>> Submitted By: michael michaud (michaudm)
>> Assigned to: Nobody/Anonymous (nobody)
>> Summary: Left click on RasterImageLayer name throws exceptions
>>
>> Initial Comment:
>> Load a RasterImageLayer
>> Left click on the layer name
>> Following stacktrace is thrown to the console :
>>
>> Exception in thread "AWT-EventQueue-0"
>> java.lang.ArrayIndexOutOfBoundsException:
>>   0
>>  at
>> com.vividsolutions.jump.workbench.ui.LayerNameRenderer.getColumnWidth
>> (LayerNameRenderer.java:134)
>>  at
>> com.vividsolutions.jump.workbench.ui.LayerNameRenderer.getCheckBoxBou
>> nds(LayerNameRenderer.java:123)
>>  at
>> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel.handleCheckBo
>> xClick(TreeLayerNamePanel.java:519)
>>  at
>> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel.access$000(Tr
>> eeLayerNamePanel.java:94)
>>  at
>> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel$3.mouseClicke
>> d(TreeLayerNamePanel.java:184)
>>  at
>> java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:25
>> 3)
>>  at
>> java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:25
>> 2)
>>  at java.awt.Component.processMouseEvent(Component.java:6293)
>>  at
>> javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
>>  at java.awt.Component.processEvent(Component.java:6055)
>>  at java.awt.Container.processEvent(Container.java:2039)
>>  at java.awt.Component.dispatchEventImpl(Component.java:4653)
>>  at java.awt.Container.dispatchEventImpl(Container.java:2097)
>>  at java.awt.Component.dispatchEvent(Component.java:4481)
>>  at
>> java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4575
>> )
>>  at
>> java.awt.LightweightDispatcher.processMouseEvent(Container.java:4245)
>>
>>  at
>> java.awt.LightweightDispatcher.dispatchEvent(Container.java:4166)
>>  at java.awt.Container.dispatchEventImpl(Container.java:2083)
>>  at java.awt.Window.dispatchEventImpl(Window.java:2482)
>>  at java.awt.Component.dispatchEvent(Component.java:4481)
>>  at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:648)
>>  at java.awt.EventQueue.access$000(EventQueue.java:84)
>>  at java.awt.EventQueue$1.run(EventQueue.java:607)
>>  at java.awt.EventQueue$1.run(EventQueue.java:605)
>>  at java.security.AccessController.doPrivileged(Native Method)
>>  at
>> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
>> ntrolContext.java:87)
>>  at
>> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
>> ntrolContext.java:98)
>>  at java.awt.EventQueue$2.run(EventQueue.java:621)
>>  at java.awt.EventQueue$2.run(EventQueue.java:619)
>>  at java.security.AccessController.doPrivileged(Native Method)
>>  at
>> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
>> ntrolContext.java:87)
>>  at java.awt.EventQueue.dispatchEvent(EventQueue.java:618)
>>  at
>> java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThre
>> ad.java:269)
>>  at
>> java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.
>> java:184)
>>  at
>> java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThre
>> ad.java:174)
>>  at
>> java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
>>
>>  at
>> java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
>>
>>  at
>> java.awt.EventDispatchT

Re: [JPP-Devel] Selectable and Editable menu items won't check on Vista and Win7

2012-04-02 Thread Edgar Soldin
actually the observed light blue square background for selected menuitems seems 
to be standard vista+ theme. other native windows applications have the same 
"issue". you just have to get used to it. so it's mainly an issue of personal 
preference, but still it was a fun issue to work around.

..ede

On 02.04.2012 20:07, Michaël Michaud wrote:
> Hi,
> 
> Nice work Ede, thanks a lot.
> 
> Michaël
>> On 31.03.2012 16:29, Michaël Michaud wrote:
>>> Hi,
 we could disable the icons conditionally for windows laf and win7+ as 
 other lafs and platforms are not hit.
>>> Agree if you do it ;-)
>> done
>>
>>> I will open a bug report which will stay opened as long as the bug is
>>> not fixed in jre.
>> beat you to it ;)
>>
>>> I had a quick look in the menus and noticed also
>>> - a icon + check symbol in category context menu (change visibility)
>>> - a icon + check symbol for View>  Edit toolbox menuitem
>>> I suppose they are also concerned by the bug, which means there are at
>>> least 4 menu-items to fix.
>>>
>> it's worked around generically where the menuitems are generated. i also did 
>> some housekeeping while at it. here's the commit message
>>
>> Revision: 2786
>>
>> http://jump-pilot.svn.sourceforge.net/jump-pilot/?rev=2786&view=rev
>> Author:   edso
>> Date: 2012-04-02 13:14:37 + (Mon, 02 Apr 2012)
>> Log Message:
>> ---
>> bugfix:
>>  http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7122141
>>  
>> http://www.mail-archive.com/jump-pilot-devel@lists.sourceforge.net/msg10601.html
>>
>> com.vividsolutions.jump.workbench.ui.plugin.FeatureInstaller
>> - unified spaces/tabs, general reformatting
>> - privatized internally used behead method
>> - deprecated legacy methods addLayerViewMenuItem(), 
>> addLayerNameViewMenuItem()
>> - added replacement method for deprecated addMainMenuItem to add plain 
>> plugins:
>> addMainMenuPlugin( PlugIn, menuPath, menuItemName, checkBox, icon, 
>> enableCheck, pos),
>>former changes by michael added methods for JMenuItem and 
>> AbstractUiPlugIn only
>> - added addMenuItemIcon() method used by all methods that add menu entries 
>> to popup/mainmenu
>>it contains a workaround for the windows laf missing checkmark ticks for 
>> checkbox entries on vista and later
>> - cleaned up addPopupMenuItem() method that duplicated routines
>> - commented (pending deletion) addMainMenuItemWithJava14Fix() methods
>>classes using the method above modified to use addMainMenuItem()
>>
>> ..ede
>>
>> --
>> 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

-- 
public class WhoDidIt{ // A comment. I love comments 
  private static Person sender;

  public static void main (String[] foo){

  sender = new Person();
  sender.setName(new String[]{"Edgar", "Soldin"});

  Address address = new Address();
  address.setStreet("Stadtweg 119");
  address.setZip(39116);
  address.setCity("Magdeburg");
  address.setCountry("Germany");

  sender.setAddress(address);

  sender.setMobilePhone(" +49(0)171-2782880 ");
  sender.setWebSiteUrl(" http://www.soldin.de ");
  sender.setEmail(" ed...@soldin.de ");
  sender.setPGPPublicKey(" http://www.soldin.de/edgar_soldin.asc ");
  sender.setGender(true);

  System.out.println(sender.toString());
  }
}

--
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


Re: [JPP-Devel] [ jump-pilot-Bugs-3513864 ] Left click on RasterImageLayer name throws exceptions

2012-04-02 Thread Rahkonen Jukka

Michaël Michaud wrote:

> Hi,
>> There is still one little oddness left. For the Sextante raster layers the 
>> toggling of visibility works by doing left click on the layer name. What 
>> looks like a check box is not active.
> Checkbox is slightly shifted on the left because of the missing color
> square, but user click is active where the checkbox should be (on the
> right of the actual checkbox) not where it really is.
> Is this what you mean ?

I supppose it is. On my screen the active area covers the two first letters of 
the Sextante raster layer name. The whole name is not active as I first thought.

-Jukkaä

> I'll try to add a symbol for Sextante raster, so that all check boxes
> are vertically aligned.

Michaël
> -Jukka Rahkonen-
>
>> -Alkuperäinen viesti-
>> Lähettäjä: SourceForge.net [mailto:nore...@sourceforge.net]
>> Lähetetty: 1. huhtikuuta 2012 15:17
>> Vastaanottaja: SourceForge.net
>> Aihe: [JPP-Devel] [ jump-pilot-Bugs-3513864 ] Left click on
>> RasterImageLayer name throws exceptions
>>
>> Bugs item #3513864, was opened at 2012-04-01 01:21
>> Message generated for change (Comment added) made by michaudm
>> You can respond by visiting:
>> https://sourceforge.net/tracker/?func=detail&atid=679906&aid=3
> 513864&group_id=118054
>> Please note that this message will contain a full copy of the
>> comment thread,
>> including the initial issue submission, for this request,
>> not just the latest update.
>> Category: OpenJUMP - Menu - File
>> Group: None
>>> Status: Closed
>>> Resolution: Fixed
>> Priority: 5
>> Private: No
>> Submitted By: michael michaud (michaudm)
>> Assigned to: Nobody/Anonymous (nobody)
>> Summary: Left click on RasterImageLayer name throws exceptions
>>
>> Initial Comment:
>> Load a RasterImageLayer
>> Left click on the layer name
>> Following stacktrace is thrown to the console :
>>
>> Exception in thread "AWT-EventQueue-0"
>> java.lang.ArrayIndexOutOfBoundsException:
>>   0
>>  at
>> com.vividsolutions.jump.workbench.ui.LayerNameRenderer.getColumnWidth
>> (LayerNameRenderer.java:134)
>>  at
>> com.vividsolutions.jump.workbench.ui.LayerNameRenderer.getCheckBoxBou
>> nds(LayerNameRenderer.java:123)
>>  at
>> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel.handleCheckBo
>> xClick(TreeLayerNamePanel.java:519)
>>  at
>> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel.access$000(Tr
>> eeLayerNamePanel.java:94)
>>  at
>> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel$3.mouseClicke
>> d(TreeLayerNamePanel.java:184)
>>  at
>> java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:25
>> 3)
>>  at
>> java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:25
>> 2)
>>  at java.awt.Component.processMouseEvent(Component.java:6293)
>>  at
>> javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
>>  at java.awt.Component.processEvent(Component.java:6055)
>>  at java.awt.Container.processEvent(Container.java:2039)
>>  at java.awt.Component.dispatchEventImpl(Component.java:4653)
>>  at java.awt.Container.dispatchEventImpl(Container.java:2097)
>>  at java.awt.Component.dispatchEvent(Component.java:4481)
>>  at
>> java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4575
>> )
>>  at
>> java.awt.LightweightDispatcher.processMouseEvent(Container.java:4245)
>>
>>  at
>> java.awt.LightweightDispatcher.dispatchEvent(Container.java:4166)
>>  at java.awt.Container.dispatchEventImpl(Container.java:2083)
>>  at java.awt.Window.dispatchEventImpl(Window.java:2482)
>>  at java.awt.Component.dispatchEvent(Component.java:4481)
>>  at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:648)
>>  at java.awt.EventQueue.access$000(EventQueue.java:84)
>>  at java.awt.EventQueue$1.run(EventQueue.java:607)
>>  at java.awt.EventQueue$1.run(EventQueue.java:605)
>>  at java.security.AccessController.doPrivileged(Native Method)
>>  at
>> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
>> ntrolContext.java:87)
>>  at
>> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
>> ntrolContext.java:98)
>>  at java.awt.EventQueue$2.run(EventQueue.java:621)
>>  at java.awt.EventQueue$2.run(EventQueue.java:619)
>>  at java.security.AccessController.doPrivileged(Native Method)
>>  at
>> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
>> ntrolContext.java:87)
>>  at java.awt.EventQueue.dispatchEvent(EventQueue.java:618)
>>  at
>> java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThre
>> ad.java:269)
>>  at
>> java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.
>> java:184)
>>  at
>> java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThre
>> ad.java:174)
>>  

Re: [JPP-Devel] [ jump-pilot-Bugs-3513864 ] Left click on RasterImageLayer name throws exceptions

2012-04-02 Thread Michaël Michaud
Hi,
>> Checkbox is slightly shifted on the left because of the missing color
>> square, but user click is active where the checkbox should be (on the
>> right of the actual checkbox) not where it really is.
>> Is this what you mean ?
> I supppose it is. On my screen the active area covers the two first letters 
> of the Sextante raster layer name. The whole name is not active as I first 
> thought.
This one should be fixed in 2788 : I added a small icon to represent 
RasterImageLayer

Michaël
>
> -Jukkaä
>
>> I'll try to add a symbol for Sextante raster, so that all check boxes
>> are vertically aligned.
> Michaël
>> -Jukka Rahkonen-
>>
>>> -Alkuperäinen viesti-
>>> Lähettäjä: SourceForge.net [mailto:nore...@sourceforge.net]
>>> Lähetetty: 1. huhtikuuta 2012 15:17
>>> Vastaanottaja: SourceForge.net
>>> Aihe: [JPP-Devel] [ jump-pilot-Bugs-3513864 ] Left click on
>>> RasterImageLayer name throws exceptions
>>>
>>> Bugs item #3513864, was opened at 2012-04-01 01:21
>>> Message generated for change (Comment added) made by michaudm
>>> You can respond by visiting:
>>> https://sourceforge.net/tracker/?func=detail&atid=679906&aid=3
>> 513864&group_id=118054
>>> Please note that this message will contain a full copy of the
>>> comment thread,
>>> including the initial issue submission, for this request,
>>> not just the latest update.
>>> Category: OpenJUMP - Menu - File
>>> Group: None
 Status: Closed
 Resolution: Fixed
>>> Priority: 5
>>> Private: No
>>> Submitted By: michael michaud (michaudm)
>>> Assigned to: Nobody/Anonymous (nobody)
>>> Summary: Left click on RasterImageLayer name throws exceptions
>>>
>>> Initial Comment:
>>> Load a RasterImageLayer
>>> Left click on the layer name
>>> Following stacktrace is thrown to the console :
>>>
>>> Exception in thread "AWT-EventQueue-0"
>>> java.lang.ArrayIndexOutOfBoundsException:
>>>0
>>>   at
>>> com.vividsolutions.jump.workbench.ui.LayerNameRenderer.getColumnWidth
>>> (LayerNameRenderer.java:134)
>>>   at
>>> com.vividsolutions.jump.workbench.ui.LayerNameRenderer.getCheckBoxBou
>>> nds(LayerNameRenderer.java:123)
>>>   at
>>> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel.handleCheckBo
>>> xClick(TreeLayerNamePanel.java:519)
>>>   at
>>> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel.access$000(Tr
>>> eeLayerNamePanel.java:94)
>>>   at
>>> com.vividsolutions.jump.workbench.ui.TreeLayerNamePanel$3.mouseClicke
>>> d(TreeLayerNamePanel.java:184)
>>>   at
>>> java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:25
>>> 3)
>>>   at
>>> java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:25
>>> 2)
>>>   at java.awt.Component.processMouseEvent(Component.java:6293)
>>>   at
>>> javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
>>>   at java.awt.Component.processEvent(Component.java:6055)
>>>   at java.awt.Container.processEvent(Container.java:2039)
>>>   at java.awt.Component.dispatchEventImpl(Component.java:4653)
>>>   at java.awt.Container.dispatchEventImpl(Container.java:2097)
>>>   at java.awt.Component.dispatchEvent(Component.java:4481)
>>>   at
>>> java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4575
>>> )
>>>   at
>>> java.awt.LightweightDispatcher.processMouseEvent(Container.java:4245)
>>>
>>>   at
>>> java.awt.LightweightDispatcher.dispatchEvent(Container.java:4166)
>>>   at java.awt.Container.dispatchEventImpl(Container.java:2083)
>>>   at java.awt.Window.dispatchEventImpl(Window.java:2482)
>>>   at java.awt.Component.dispatchEvent(Component.java:4481)
>>>   at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:648)
>>>   at java.awt.EventQueue.access$000(EventQueue.java:84)
>>>   at java.awt.EventQueue$1.run(EventQueue.java:607)
>>>   at java.awt.EventQueue$1.run(EventQueue.java:605)
>>>   at java.security.AccessController.doPrivileged(Native Method)
>>>   at
>>> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
>>> ntrolContext.java:87)
>>>   at
>>> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
>>> ntrolContext.java:98)
>>>   at java.awt.EventQueue$2.run(EventQueue.java:621)
>>>   at java.awt.EventQueue$2.run(EventQueue.java:619)
>>>   at java.security.AccessController.doPrivileged(Native Method)
>>>   at
>>> java.security.AccessControlContext$1.doIntersectionPrivilege(AccessCo
>>> ntrolContext.java:87)
>>>   at java.awt.EventQueue.dispatchEvent(EventQueue.java:618)
>>>   at
>>> java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThre
>>> ad.java:269)
>>>   at
>>> java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.
>>> java:184)
>>>   at
>>> java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThre
>

Re: [JPP-Devel] New csv driver

2012-04-02 Thread Michaël Michaud

Hi,

Just uploaded the csv-driver-0.6.0beta4.jar

Finnish translation attached. I have also a few comments.

I'll add finnish translation, thanks,

Uploaded (I added one key :
#T:drivers.csv.invert-selection=Invert Selection
- All multi word menu texts contain dashes on my computer instead of 
space, like this

"Kommenttirivin-rakenne-(regexp)"
Did not investigate into that yet. This is changed into dashes by the 
IO framework I used and I
think it has been done to safely use these labels as keys. But I'm not 
sure why labels containing

whitespaces would be a problem.
Removed that (but don't know why Paul Austin replaced all whitespaces by 
dashes)
- When saving without giving file extension the result is a bit odd 
"New.csv, txt, xyz"
Oh, not nice. I must check if it does the same thing for other 
multi-extension

Fixed.

- "Inverse selection" -button cannot be translated

OK, will add it

See above

- Where is the string "queted", English translation "Queted string" used?
Unused. Finally, the parser should know how to parse quoted or 
unquoted fields.
It should write quotes only for records and fields containing a quote 
or a separator, but

this is a place where more tests would be useful.
- Canceling saving from the optional "Select attributes" menu leads 
to NPE

Oh, very bad.
I hope I'll be able to have a look into all your points by the end of 
the week.
I just hide the cancel button. Currently, when the choose attribute 
dialog popup,
the writing process has already begun, so that it is not really possible 
to cancel.

I mean without refactoring the code.

Michaël



Thanks,

Michaël

java.lang.NullPointerException
 at 
fr.michaelm.jump.drivers.csv.CSVDataSource$1.executeUpdate(CSVDataSource.java:184)
 at 
com.vividsolutions.jump.workbench.datasource.AbstractSaveDatasetAsPlugIn.run(AbstractSaveDatasetAsPlugIn.java:33)
 at 
com.vividsolutions.jump.workbench.ui.task.TaskMonitorManager$TaskWrapper.run(TaskMonitorManager.java:152)

 at java.lang.Thread.run(Unknown Source)
-Jukka Rahkonen-


Michaël Michaud  wrote:
Thanks Arnd,

I escaped accented characters and uploaded the extension again as
csv-driver-0.6.0beta3.jar

Michaël


Hallo Michaël,

here is the german language file for You new csv driver.

Arnd


Hi Arnd,

Hi uploaded the internationalized plugin at the same url
Here attached is the english i18n properties file

Thanks,

Michaël


Le 29/03/2012 11:58, Arnd Kielhorn a écrit :

Hello Michael,

I could make the german translation of the dialog.

Arnd


Hi,

thank You for that nice driver.
Is there already an englsih and german translation for it because when

I

load OJ the csv driver dialog is in french?
Sorry, forgot it. I'll send the internationalized version by the end of
the week (english only).

Michaël

Arnd



Hi,

I uploaded a beta version of a new csv driver : csv-driver-0.6

beta.jar

http://geo.michaelm.free.fr/OpenJUMP/resources/drivers/

This is a complete rewrite of my old txt driver.
It should be more flexible and currently includes
- an auto-mode able to guess the structure of most character

delimited

files (including pirol formatted files)
- a set-options mode where the user has to set the file structure to
help the driver to read the file correctly
- a writer with only a few options
I've not yet implemented anything to read csvt files from OGR, but

tried

to make the driver
flexible enough to make it possible.

At the same address, you'll find two documents to help you understand
the options :
OpenJUMP_CSV.odt




OpenJUMP_CSV_auto.odt





It is quite difficult to make such a driver robust and it is in its
early stage.
All your tests and remarks are welcome.

Michaël







--

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




--