Hi Matthias,

Thanks a lot for contribution.
There is much activity on the list, and I hardly follow all discussions 
and developpments.

1) I should have missed something but I cannot get the option you added 
(I just updated from svn and compiled a few minutes ago)
Can you give me some information about how it is supposed to work. Am I 
supposed to get a panel with a charset option for any shapefile loading ?

2) if we want to keep java 5 compatibility, you have to use new 
String(byte[], int, int, String) instead of new String(byte[], int, int, 
Charset) on line 264

3) about your proposition of saving the charset in the project when 
different from default jvm charset : Seems a good idea to me.

Michaël

Le 03/09/2010 20:34, Matthias Scholz a écrit :
> Hi,
>
> sorry my mistake. I have it only tested with open and open recent ;-) In
> the SVN it is fixed for the first time, but there is a general question
> and I want to hear the other developers/users. If I open a shapefile
> with a special charset setting for the first time, it would be logic for
> me, if we save the charset setting for this file in the project. So if
> the user opens this file again with the project, the file will be opened
> with the same charset.
> Should I implement this or have anyone an other suggestion?
>
> Regards
>
> Matthias
>> Nice feature, but it is going to take a little more work.  It
>> currently breaks shapefile loading from a project file and the
>> lright-click on layer legacy method "Load Dataset".  I think you need
>> to set a default charset on start up.
>>
>> regards,
>>
>> Larry
>>
>> java.lang.IllegalArgumentException: Null charset name
>>      at java.nio.charset.Charset.lookup(Charset.java:430)
>>      at java.nio.charset.Charset.forName(Charset.java:503)
>>      at
>> com.vividsolutions.jump.io.ShapefileReader.read(ShapefileReader.java:137)
>>      at
>> com.vividsolutions.jump.io.datasource.DelegatingCompressedFileHandler.read(DelegatingCompressedFileHandler.java:79)
>>      at
>> com.vividsolutions.jump.io.datasource.ReaderWriterFileDataSource$1.executeQuery(ReaderWriterFileDataSource.java:61)
>>      at
>> com.vividsolutions.jump.io.datasource.ReaderWriterFileDataSource$1.executeQuery(ReaderWriterFileDataSource.java:79)
>>      at
>> org.openjump.core.ui.plugin.file.open.OpenProjectWizard.executeQuery(OpenProjectWizard.java:282)
>>      at
>> org.openjump.core.ui.plugin.file.open.OpenProjectWizard.load(OpenProjectWizard.java:271)
>>      at
>> org.openjump.core.ui.plugin.file.open.OpenProjectWizard.loadLayers(OpenProjectWizard.java:208)
>>      at
>> org.openjump.core.ui.plugin.file.open.OpenProjectWizard.open(OpenProjectWizard.java:152)
>>      at
>> org.openjump.core.ui.plugin.file.open.OpenProjectWizard.open(OpenProjectWizard.java:107)
>>      at
>> org.openjump.core.ui.plugin.file.open.OpenProjectWizard.run(OpenProjectWizard.java:99)
>>      at
>> org.openjump.core.ui.plugin.AbstractWizardPlugin.run(AbstractWizardPlugin.java:71)
>>      at
>> com.vividsolutions.jump.workbench.ui.task.TaskMonitorManager$TaskWrapper.run(TaskMonitorManager.java:151)
>>      at java.lang.Thread.run(Thread.java:619)
>>
>> On Fri, Sep 3, 2010 at 4:46 AM, Matthias Scholz<m...@jammerhund.de
>> <mailto:m...@jammerhund.de>>  wrote:
>>
>>      Hi,
>>
>>      I've commited some little changes for selecting an other charset while
>>      opening an ESRI shapefile.
>>      Thank you Michael and Stefan for your hints to the wizard framework.
>>      Thereby the code changes was very small and consitent :-) On the other
>>      hand Michael is right with his opinion about the wizard framework.
>>      Without good documentation you need many hours to understand and
>>      implement such options for file loading.
>>      Please take a look at my changes and tell me your opinion.
>>      In the language files (en and de) I've added a
>>      "org.openjump.core.ui.plugin.file.charset" key.
>>
>>      Regards
>>
>>      Matthias
>>      >    Le 30/08/2010 18:50, Stefan Steiniger a écrit :
>>      >
>>      >>  Hei,
>>      >>
>>      >>  not sure, but wasn't the idea of the wizard that several
>>      dialogs can be
>>      >>  displayed after each other?
>>      >>
>>      
>> http://sourceforge.net/apps/mediawiki/jump-pilot/index.php?title=How_to_Create_an_Open_Wizard
>>      >>
>>      >  You're right Stefan, this is the idea of this wizard
>>      >
>>      >>  wouldn't that work for you too?
>>      >>
>>      >  I found it was not very easy to use the wizard framework to add
>>      options
>>      >  to an open file dialog box, but I think this is the way to go if
>>      we want
>>      >  to have something consistent in OpenJUMP
>>      >
>>      >>  Though I am not sure which loader uses that option currently.
>>      But it
>>      >>  looks like the "WMS Layer" option does that. (you see not a
>>      "finish" but
>>      >>  a "next" button)
>>      >>
>>      >  I used it to make a text/csv/wkt format driver with some options.
>>      >  You can also find my code here :
>>      >
>>      
>> http://geo.michaelm.free.fr/OpenJUMP/resources/drivers/txt-driver-src-0.2.1.zip
>>      >
>>      >  Michaël
>>      >
>>      >>  stefan
>>      >>
>>      >>  Sunburned Surveyor wrote:
>>      >>
>>      >>>  I don't see any problems with your approach. Is the idea to
>>      allow the
>>      >>>  charset of the shapefile to be selected from the new JComboBox?
>>      >>>
>>      >>>  Landon
>>      >>>
>>      >>>  On Sat, Aug 28, 2010 at 3:03 AM, Matthias
>>      Scholz<m...@jammerhund.de<mailto:m...@jammerhund.de>>   wrote:
>>      >>>
>>      >>>>  Hi,
>>      >>>>
>>      >>>>  in the last few weeks i had some problems with different
>>      charsets in
>>      >>>>  shape/dbf files. In the openjump.bat or openjump.sh file I
>>      have inserted
>>      >>>>  the java system property "file.encoding=...". My problems are
>>      the german
>>      >>>>  "Umlaute" in the attributes.
>>      >>>>  I would like to add a JComboBox to the
>>      >>>>  org.openjump.core.ui.plugin.file.open.SelectFilesPanel.
>>      Because the
>>      >>>>  SelectFilesPanel extends  JFileChooser I don't know any
>>      possibility to
>>      >>>>  add any other widget. I would prefer to change
>>      >>>>
>>      >>>>  public class SelectFilesPanel extends JFileChooser implements
>>      WizardPanel
>>      >>>>
>>      >>>>  to
>>      >>>>
>>      >>>>  public class SelectFilesPanel extends JPanel implements
>>      WizardPanel
>>      >>>>
>>      >>>>  then add the JFileChooser and a JComboBox to the JPanel. I
>>      don't know if
>>      >>>>  this have impacts to other classes? Before I start to code, I
>>      would like
>>      >>>>  to hear from the other developers.
>>      >>>>
>>      >>>>
>>      >>>>  Regards
>>      >>>>
>>      >>>>  Matthias
>>      >>>>
>>      >>>>
>>      
>> ------------------------------------------------------------------------------
>>      >>>>  Sell apps to millions through the Intel(R) Atom(Tm) Developer
>>      Program
>>      >>>>  Be part of this innovative community and reach millions of
>>      netbook users
>>      >>>>  worldwide. Take advantage of special opportunities to
>>      increase revenue and
>>      >>>>  speed time-to-market. Join now, and jumpstart your future.
>>      >>>>  http://p.sf.net/sfu/intel-atom-d2d
>>      >>>>  _______________________________________________
>>      >>>>  Jump-pilot-devel mailing list
>>      >>>>  Jump-pilot-devel@lists.sourceforge.net
>>      <mailto:Jump-pilot-devel@lists.sourceforge.net>
>>      >>>>  https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>      >>>>
>>      >>>>
>>      >>>
>>      
>> ------------------------------------------------------------------------------
>>      >>>  Sell apps to millions through the Intel(R) Atom(Tm) Developer
>>      Program
>>      >>>  Be part of this innovative community and reach millions of
>>      netbook users
>>      >>>  worldwide. Take advantage of special opportunities to increase
>>      revenue and
>>      >>>  speed time-to-market. Join now, and jumpstart your future.
>>      >>>  http://p.sf.net/sfu/intel-atom-d2d
>>      >>>  _______________________________________________
>>      >>>  Jump-pilot-devel mailing list
>>      >>>  Jump-pilot-devel@lists.sourceforge.net
>>      <mailto:Jump-pilot-devel@lists.sourceforge.net>
>>      >>>  https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>      >>>
>>      >>>
>>      >>>
>>      >>
>>      
>> ------------------------------------------------------------------------------
>>      >>  This SF.net Dev2Dev email is sponsored by:
>>      >>
>>      >>  Show off your parallel programming skills.
>>      >>  Enter the Intel(R) Threading Challenge 2010.
>>      >>  http://p.sf.net/sfu/intel-thread-sfd
>>      >>  _______________________________________________
>>      >>  Jump-pilot-devel mailing list
>>      >>  Jump-pilot-devel@lists.sourceforge.net
>>      <mailto:Jump-pilot-devel@lists.sourceforge.net>
>>      >>  https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>      >>
>>      >>
>>      >>
>>      >
>>      >
>>      >
>>      
>> ------------------------------------------------------------------------------
>>      >  This SF.net Dev2Dev email is sponsored by:
>>      >
>>      >  Show off your parallel programming skills.
>>      >  Enter the Intel(R) Threading Challenge 2010.
>>      >  http://p.sf.net/sfu/intel-thread-sfd
>>      >  _______________________________________________
>>      >  Jump-pilot-devel mailing list
>>      >  Jump-pilot-devel@lists.sourceforge.net
>>      <mailto:Jump-pilot-devel@lists.sourceforge.net>
>>      >  https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>      >
>>
>>
>>      
>> ------------------------------------------------------------------------------
>>      This SF.net Dev2Dev email is sponsored by:
>>
>>      Show off your parallel programming skills.
>>      Enter the Intel(R) Threading Challenge 2010.
>>      http://p.sf.net/sfu/intel-thread-sfd
>>      _______________________________________________
>>      Jump-pilot-devel mailing list
>>      Jump-pilot-devel@lists.sourceforge.net
>>      <mailto:Jump-pilot-devel@lists.sourceforge.net>
>>      https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>
>>
>> ------------------------------------------------------------------------
>>
>> ------------------------------------------------------------------------------
>> This SF.net Dev2Dev email is sponsored by:
>>
>> Show off your parallel programming skills.
>> Enter the Intel(R) Threading Challenge 2010.
>> http://p.sf.net/sfu/intel-thread-sfd
>> ------------------------------------------------------------------------
>>
>> _______________________________________________
>> Jump-pilot-devel mailing list
>> Jump-pilot-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>
>
> ------------------------------------------------------------------------------
> This SF.net Dev2Dev email is sponsored by:
>
> Show off your parallel programming skills.
> Enter the Intel(R) Threading Challenge 2010.
> http://p.sf.net/sfu/intel-thread-sfd
> _______________________________________________
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
>


------------------------------------------------------------------------------
This SF.net Dev2Dev email is sponsored by:

Show off your parallel programming skills.
Enter the Intel(R) Threading Challenge 2010.
http://p.sf.net/sfu/intel-thread-sfd
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to