Hi Stefan,
Having one charset choice doesn't support multiple charsets per project
as the current method does. I'm not sure how often this happens, but it
isn't too unusual to use datasets from multiple sources. Instead of
commenting out the dialogs, it might be better to have a workbench
blackboard option to skip them and use the default instead.
Surely we are all going to run into an "Umlaute" or similar in our data
eventually.
regards,
Larry
On Wed, Sep 8, 2010 at 6:54 PM, Stefan Steiniger <sst...@geo.uzh.ch> wrote:
> I like the charset choice, thanks Matthias.
>
> However, I wonder what users feel having every time to chose (even it is
> only one more click) and probably not knowing what it means.
>
> So one thought I had is to "comment out" the dialog call and put the
> dialog options in (a new?) options tab instead and store it the
> blackboard? Then the currently set value is read while loading the shp
> (i.e. exposing the option in a different way to the user).
>
> I know its additional work but may be more user friendly for those who
> never care about charsets and have to load quite a bit of data during a
> day? Like I do ;)
>
> Actually my co-worker just suggested the option to have a checkbox for
> the additional dialog asking "Do you want to use that value every time".
> so it doesn't pop up all the time during one session.
>
> my 2 cents...
>
> stefan
>
> Michaël Michaud wrote:
> > Hi Matthias,
> >
> >> 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
> ?
> > Sorry for that question, I did not launch the new compiled version. I
> > know get the charset choice dialog.
> >
> > Thanks,
> >
> > Michaël
> >
> >> 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
> >>
> >>
> >
> >
> >
> ------------------------------------------------------------------------------
> > 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
