Re: [JPP-Devel] SVN: [4663] core/trunk

2015-12-24 Thread edgar . soldin
hey Alberto,

i have some questions/comments

1. 
in several files eg. Parser.java you added lines ending with '//$NON-NLS-1$' . 
why?

2. 
you insert wms code (url appending) directly into the featureinfotool up to the 
point that it even runs the web request. 
why don't you use the wms package? i see you added the info url to the 
capabilities already. would you mind implementing that cleanly with a 
InfoRequest object along the lines of existing MapRequest.java . this way you 
can reuse all the magic there is already including auth/proxy usage.
changing the API of TaskFrame to an IOException throwing getInfoFrame() is an 
absolute no go from my point of view.

how long would it take for you to fix it? keep in mind, we can easily slip in a 
clean version with the first maintenance release. so no hurry.

..ede


On 24.12.2015 00:18, Alberto De Luca wrote:
> Right sorry,
> 
> I think Parser.java was the culprit. I had deleted it locally, but not on
> the server. It is no longer needed actually, but instead of deleting it I
> edited to make it compatible with the modified WMS classes (so we can keep
> it, you never know...).
> 
> Apparently it builds now, let me know.
> Alberto
> 
> On 23 December 2015 at 22:00,  wrote:
> 
>> Alberto,
>>
>> your commit seems incomplete.. ede
>>
>>
>> On 23.12.2015 18:28, Rahkonen Jukka (MML) wrote:
>>> Hi,
>>>
>>> r4663 did not build
>>>
>> http://sourceforge.net/projects/jump-pilot/files/OpenJUMP_snapshots/OpenJUMP-20151223-r4663.log/download
>>>
>>> -Jukka Rahkonen-
>>>
>>>
>>> -Alkuperäinen viesti-
>>> Lähettäjä: jump-pilot-...@lists.sourceforge.net [mailto:
>> jump-pilot-...@lists.sourceforge.net]
>>> Lähetetty: 23. joulukuuta 2015 18:27
>>> Vastaanottaja: jump-pilot-devel@lists.sourceforge.net
>>> Aihe: [JPP-Devel] SVN: [4663] core/trunk
>>>
>>> Revision: 4663
>>>   http://sourceforge.net/p/jump-pilot/code/4663
>>> Author:   bertazza
>>> Date: 2015-12-23 16:26:41 + (Wed, 23 Dec 2015)
>>> Log Message:
>>> ---
>>> Info feature tool: added a pane to show WMS info.
>>>
>>> Modified Paths:
>>> --
>>> core/trunk/ChangeLog
>>> core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
>>>
>>  core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
>>> core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
>>>
>>  
>> core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
>>> core/trunk/src/com/vividsolutions/wms/AbstractParser.java
>>> core/trunk/src/com/vividsolutions/wms/Capabilities.java
>>> core/trunk/src/com/vividsolutions/wms/MapLayer.java
>>> core/trunk/src/com/vividsolutions/wms/ParserWMS1_0.java
>>> core/trunk/src/com/vividsolutions/wms/ParserWMS1_1.java
>>> core/trunk/src/com/vividsolutions/wms/ParserWMS1_3.java
>>> core/trunk/src/com/vividsolutions/wms/WMService.java
>>> core/trunk/src/org/openjump/core/ui/plugin/queries/QueryDialog.java
>>>
>>> Added Paths:
>>> ---
>>>
>>  core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/clean.xsl
>>> core/trunk/src/com/vividsolutions/wms/MapStyle.java
>>>
>>> Modified: core/trunk/ChangeLog
>>> ===
>>> --- core/trunk/ChangeLog  2015-12-23 15:39:50 UTC (rev 4662)
>>> +++ core/trunk/ChangeLog  2015-12-23 16:26:41 UTC (rev 4663)
>>> @@ -3,6 +3,8 @@
>>>  # 2. make sure that lines break at 80 chars for constricted display
>> situations
>>>  #< 80 chars
>> -->#
>>>
>>> +2015-12-23 bertazza
>>> +  * Info feature tool: added a pane to show WMS info.
>>>
>>>  <--- Changes.txt updated
>> 'til here
>>>  2015-12-21 bertazza
>>>
>>> Modified:
>> core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
>>> ===
>>> --- core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
>>   2015-12-23 15:39:50 UTC (rev 4662)
>>> +++ core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
>>   2015-12-23 16:26:41 UTC (rev 4663)
>>> @@ -61,6 +61,8 @@
>>>  import com.vividsolutions.jump.workbench.ui.images.IconLoader;
>>>  import
>> com.vividsolutions.jump.workbench.ui.plugin.PersistentBlackboardPlugIn;
>>>  import com.vividsolutions.jump.workbench.ui.plugin.ViewAttributesPlugIn;
>>> +import java.io.IOException;
>>> +import javax.swing.JEditorPane;
>>>  import javax.swing.JScrollPane;
>>>  import javax.swing.JTable;
>>>  import javax.swing.table.DefaultTableModel;
>>> @@ -99,16 +101,18 @@
>>>  private GeometryInfoTab geometryInfoTab;
>>>  private JTabbedPane tabbedPane = new JTabbedPane();
>>>  private RasterInfoTab rasterInfoTab;
>>> +private WMSInfoTab wmsInfoTab;
>>>  private WorkbenchFrame workbenchFrame;
>>>  private static ImageIcon ICON =
>> IconLoader.icon("information_16x

[JPP-Devel] SVN: [4665] core/trunk

2015-12-24 Thread jump-pilot-svn
Revision: 4665
  http://sourceforge.net/p/jump-pilot/code/4665
Author:   bertazza
Date: 2015-12-24 12:03:26 + (Thu, 24 Dec 2015)
Log Message:
---
Info feature tool for WMS: info shown only for visible layers, added 
&FEATURE_COUNT=10 parameter, and changed method to retrieve url as suggested by 
Jukka (should now support basic authentication).

Modified Paths:
--
core/trunk/ChangeLog

core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java

Modified: core/trunk/ChangeLog
===
--- core/trunk/ChangeLog2015-12-23 23:13:38 UTC (rev 4664)
+++ core/trunk/ChangeLog2015-12-24 12:03:26 UTC (rev 4665)
@@ -4,6 +4,11 @@
 #< 80 chars 
-->#
 
 2015-12-24 bertazza
+  * Info feature tool for WMS: info shown only for visible layers, added
+&FEATURE_COUNT=10 parameter, and changed method to retrieve url as
+suggested by Jukka (should now support basic authentication).
+
+2015-12-24 bertazza
   * Fixed broken 4663 commit
 
 2015-12-23 bertazza

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
===
--- 
core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
 2015-12-23 23:13:38 UTC (rev 4664)
+++ 
core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
 2015-12-24 12:03:26 UTC (rev 4665)
@@ -113,11 +113,20 @@
 
 // WMS
 List wmsLay_l = 
getWorkbench().getContext().getLayerManager().getLayerables(WMSLayer.class);
+//Iterator iter = 
getWorkbench().getContext().getLayerNamePanel().selectedNodes(WMSLayer.class).iterator();
+
 String response = "";
-for(Layerable lay : wmsLay_l) {
+for(Layerable lay : wmsLay_l) {   
+
+// We only want visible layers
 WMSLayer wmsLayer = (WMSLayer) lay;
+if(!wmsLayer.isVisible()) {
+continue;
+}
 
-String featInfoUrl = 
wmsLayer.getService().getCapabilities().getFeatureInfoURL(); 
+//String featInfoUrl = 
wmsLayer.getService().getCapabilities().getFeatureInfoURL();
+String featInfoUrl = 
wmsLayer.createRequest(getWorkbench().getContext().getLayerViewPanel()).getURL().toString();
+
 String names = getWmsLayeNames(wmsLayer);
 
 Point2D point = getPanel().getViewport().toViewPoint(coord);
@@ -128,6 +137,7 @@
 } else {
 featInfoUrl += "?";
 }
+
 String version = wmsLayer.getWmsVersion();
 if (WMService.WMS_1_0_0.equals(version)) {
 featInfoUrl += "REQUEST=feature_info&WMTVER=1.0.0";
@@ -162,6 +172,8 @@
 }
 }
 
+featInfoUrl = featInfoUrl.concat("&FEATURE_COUNT=10 ");
+
 URL url = stripXhtmlTags(featInfoUrl);
 
 String newLine = System.getProperty("line.separator");


--
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


[JPP-Devel] SVN: [4666] core/trunk

2015-12-24 Thread jump-pilot-svn
Revision: 4666
  http://sourceforge.net/p/jump-pilot/code/4666
Author:   bertazza
Date: 2015-12-24 12:33:54 + (Thu, 24 Dec 2015)
Log Message:
---
Removed IOExpcetion throw from TaskFrame

Modified Paths:
--
core/trunk/ChangeLog
core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
core/trunk/src/com/vividsolutions/wms/AbstractParser.java
core/trunk/src/com/vividsolutions/wms/Parser.java

Modified: core/trunk/ChangeLog
===
--- core/trunk/ChangeLog2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/ChangeLog2015-12-24 12:33:54 UTC (rev 4666)
@@ -4,6 +4,9 @@
 #< 80 chars 
-->#
 
 2015-12-24 bertazza
+  * Removed IOExpcetion throw from TaskFrame
+
+2015-12-24 bertazza
   * Info feature tool for WMS: info shown only for visible layers, added
 &FEATURE_COUNT=10 parameter, and changed method to retrieve url as
 suggested by Jukka (should now support basic authentication).

Modified: core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
===
--- core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java  
2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java  
2015-12-24 12:33:54 UTC (rev 4666)
@@ -108,7 +108,7 @@
 public InfoFrame(
 WorkbenchContext workbenchContext,
 LayerManagerProxy layerManagerProxy,
-final TaskFrame taskFrame) throws IOException {
+final TaskFrame taskFrame) {
blackboard = PersistentBlackboardPlugIn.get(workbenchContext);
 geometryInfoTab = new GeometryInfoTab(model, workbenchContext);
 rasterInfoTab = new RasterInfoTab(null, null);
@@ -375,7 +375,7 @@
 
 private final JEditorPane jEditorPane;
 
-public WMSInfoTab() throws IOException {
+public WMSInfoTab() {
 
 setLayout(new BorderLayout());
 

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
===
--- core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java   
2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java   
2015-12-24 12:33:54 UTC (rev 4666)
@@ -45,7 +45,7 @@
  * positions InfoFrames differently depending on whether or not they are 
primary.
  */
 public class PrimaryInfoFrame extends InfoFrame {
-public PrimaryInfoFrame(WorkbenchContext workbenchContext, 
LayerManagerProxy layerManagerProxy, TaskFrame taskFrame) throws IOException {
+public PrimaryInfoFrame(WorkbenchContext workbenchContext, 
LayerManagerProxy layerManagerProxy, TaskFrame taskFrame) {
 super(workbenchContext, layerManagerProxy, taskFrame);   
 }
 }

Modified: core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
===
--- core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java  
2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java  
2015-12-24 12:33:54 UTC (rev 4666)
@@ -224,7 +224,7 @@
 return task.getLayerManager();
 }
 
-public InfoFrame getInfoFrame() throws IOException {
+public InfoFrame getInfoFrame() {
 if (infoFrame == null || infoFrame.isClosed()) {
 infoFrame = new PrimaryInfoFrame(workbenchContext, this, this);
 }

Modified: core/trunk/src/com/vividsolutions/wms/AbstractParser.java
===
--- core/trunk/src/com/vividsolutions/wms/AbstractParser.java   2015-12-24 
12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/wms/AbstractParser.java   2015-12-24 
12:33:54 UTC (rev 4666)
@@ -220,20 +220,20 @@
 boundingBoxList.add ( new BoundingBox("Geographics", 
geographicBBox.getEnvelope()) );
 } else if( n.getNodeName().equals( "Layer" ) ) {
 subLayers.add( wmsLayerFromNode( n ) );
-} else if (n.getNodeName().equals("Style")) { //$NON-NLS-1$
-String styleName = ""; //$NON-NLS-1$
-String titleName = ""; //$NON-NLS-1$
-String legendFormat = ""; //$NON-NLS-1$
-String url = ""; //$NON-NLS-1$
+} else if (n.getNodeName().equals("Style")) { 
+String styleName = ""; 
+String titleName = ""; 
+String legendFormat = ""; 
+   

Re: [JPP-Devel] SVN: [4663] core/trunk

2015-12-24 Thread Alberto De Luca
Hi.

@Ede
1) that is something added by eclipse (
http://stackoverflow.com/questions/654037/what-does-non-nls-1-mean). I took
it away from the WMS-related files, but you will find it in several other
files.

2) yes, changing the API of TaskFrame wasn't brilliant... I reverted to
it's original state. For the neat implementation of InfoRequest, it will be
done, but I need some more time.

@Jukka
I changed the method used to retrieve the url as you suggested. I added the
FEATURE_COUNT=10 parameter, and I limited the WMS info to the visible
layers. Not visible and selected, just visible, to make it consistent with
vector layers.

Alberto

On 24 December 2015 at 13:02,  wrote:

> hey Alberto,
>
> i have some questions/comments
>
> 1.
> in several files eg. Parser.java you added lines ending with
> '//$NON-NLS-1$' . why?
>
> 2.
> you insert wms code (url appending) directly into the featureinfotool up
> to the point that it even runs the web request.
> why don't you use the wms package? i see you added the info url to the
> capabilities already. would you mind implementing that cleanly with a
> InfoRequest object along the lines of existing MapRequest.java . this way
> you can reuse all the magic there is already including auth/proxy usage.
> changing the API of TaskFrame to an IOException throwing getInfoFrame() is
> an absolute no go from my point of view.
>
> how long would it take for you to fix it? keep in mind, we can easily slip
> in a clean version with the first maintenance release. so no hurry.
>
> ..ede
>
>
> On 24.12.2015 00:18, Alberto De Luca wrote:
> > Right sorry,
> >
> > I think Parser.java was the culprit. I had deleted it locally, but not on
> > the server. It is no longer needed actually, but instead of deleting it I
> > edited to make it compatible with the modified WMS classes (so we can
> keep
> > it, you never know...).
> >
> > Apparently it builds now, let me know.
> > Alberto
> >
> > On 23 December 2015 at 22:00,  wrote:
> >
> >> Alberto,
> >>
> >> your commit seems incomplete.. ede
> >>
> >>
> >> On 23.12.2015 18:28, Rahkonen Jukka (MML) wrote:
> >>> Hi,
> >>>
> >>> r4663 did not build
> >>>
> >>
> http://sourceforge.net/projects/jump-pilot/files/OpenJUMP_snapshots/OpenJUMP-20151223-r4663.log/download
> >>>
> >>> -Jukka Rahkonen-
> >>>
> >>>
> >>> -Alkuperäinen viesti-
> >>> Lähettäjä: jump-pilot-...@lists.sourceforge.net [mailto:
> >> jump-pilot-...@lists.sourceforge.net]
> >>> Lähetetty: 23. joulukuuta 2015 18:27
> >>> Vastaanottaja: jump-pilot-devel@lists.sourceforge.net
> >>> Aihe: [JPP-Devel] SVN: [4663] core/trunk
> >>>
> >>> Revision: 4663
> >>>   http://sourceforge.net/p/jump-pilot/code/4663
> >>> Author:   bertazza
> >>> Date: 2015-12-23 16:26:41 + (Wed, 23 Dec 2015)
> >>> Log Message:
> >>> ---
> >>> Info feature tool: added a pane to show WMS info.
> >>>
> >>> Modified Paths:
> >>> --
> >>> core/trunk/ChangeLog
> >>> core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
> >>>
> >>
> core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
> >>> core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
> >>>
> >>
> core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
> >>> core/trunk/src/com/vividsolutions/wms/AbstractParser.java
> >>> core/trunk/src/com/vividsolutions/wms/Capabilities.java
> >>> core/trunk/src/com/vividsolutions/wms/MapLayer.java
> >>> core/trunk/src/com/vividsolutions/wms/ParserWMS1_0.java
> >>> core/trunk/src/com/vividsolutions/wms/ParserWMS1_1.java
> >>> core/trunk/src/com/vividsolutions/wms/ParserWMS1_3.java
> >>> core/trunk/src/com/vividsolutions/wms/WMService.java
> >>> core/trunk/src/org/openjump/core/ui/plugin/queries/QueryDialog.java
> >>>
> >>> Added Paths:
> >>> ---
> >>>
> >>
> core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/clean.xsl
> >>> core/trunk/src/com/vividsolutions/wms/MapStyle.java
> >>>
> >>> Modified: core/trunk/ChangeLog
> >>> ===
> >>> --- core/trunk/ChangeLog  2015-12-23 15:39:50 UTC (rev 4662)
> >>> +++ core/trunk/ChangeLog  2015-12-23 16:26:41 UTC (rev 4663)
> >>> @@ -3,6 +3,8 @@
> >>>  # 2. make sure that lines break at 80 chars for constricted display
> >> situations
> >>>  #< 80 chars
> >> -->#
> >>>
> >>> +2015-12-23 bertazza
> >>> +  * Info feature tool: added a pane to show WMS info.
> >>>
> >>>  <--- Changes.txt updated
> >> 'til here
> >>>  2015-12-21 bertazza
> >>>
> >>> Modified:
> >> core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
> >>> ===
> >>> --- core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
> >>   2015-12-23 15:39:50 UTC (rev 4662)
> >>> +++ core/tru

Re: [JPP-Devel] SVN: [4666] core/trunk

2015-12-24 Thread Rahkonen Jukka (MML)
Hi,

R4666 did not build

http://sourceforge.net/projects/jump-pilot/files/OpenJUMP_snapshots/OpenJUMP-20151224-r4666.log/download

-Jukka-

-Alkuperäinen viesti-
Lähettäjä: jump-pilot-...@lists.sourceforge.net 
[mailto:jump-pilot-...@lists.sourceforge.net] 
Lähetetty: 24. joulukuuta 2015 14:34
Vastaanottaja: jump-pilot-devel@lists.sourceforge.net
Aihe: [JPP-Devel] SVN: [4666] core/trunk

Revision: 4666
  http://sourceforge.net/p/jump-pilot/code/4666
Author:   bertazza
Date: 2015-12-24 12:33:54 + (Thu, 24 Dec 2015)
Log Message:
---
Removed IOExpcetion throw from TaskFrame

Modified Paths:
--
core/trunk/ChangeLog
core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
core/trunk/src/com/vividsolutions/wms/AbstractParser.java
core/trunk/src/com/vividsolutions/wms/Parser.java

Modified: core/trunk/ChangeLog
===
--- core/trunk/ChangeLog2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/ChangeLog2015-12-24 12:33:54 UTC (rev 4666)
@@ -4,6 +4,9 @@
 #< 80 chars 
-->#
 
 2015-12-24 bertazza
+  * Removed IOExpcetion throw from TaskFrame
+
+2015-12-24 bertazza
   * Info feature tool for WMS: info shown only for visible layers, added
 &FEATURE_COUNT=10 parameter, and changed method to retrieve url as
 suggested by Jukka (should now support basic authentication).

Modified: core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
===
--- core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java  
2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java  
2015-12-24 12:33:54 UTC (rev 4666)
@@ -108,7 +108,7 @@
 public InfoFrame(
 WorkbenchContext workbenchContext,
 LayerManagerProxy layerManagerProxy,
-final TaskFrame taskFrame) throws IOException {
+final TaskFrame taskFrame) {
blackboard = PersistentBlackboardPlugIn.get(workbenchContext);
 geometryInfoTab = new GeometryInfoTab(model, workbenchContext);
 rasterInfoTab = new RasterInfoTab(null, null); @@ -375,7 +375,7 @@
 
 private final JEditorPane jEditorPane;
 
-public WMSInfoTab() throws IOException {
+public WMSInfoTab() {
 
 setLayout(new BorderLayout());
 

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
===
--- core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java   
2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java   
2015-12-24 12:33:54 UTC (rev 4666)
@@ -45,7 +45,7 @@
  * positions InfoFrames differently depending on whether or not they are 
primary.
  */
 public class PrimaryInfoFrame extends InfoFrame {
-public PrimaryInfoFrame(WorkbenchContext workbenchContext, 
LayerManagerProxy layerManagerProxy, TaskFrame taskFrame) throws IOException {
+public PrimaryInfoFrame(WorkbenchContext workbenchContext, 
+ LayerManagerProxy layerManagerProxy, TaskFrame taskFrame) {
 super(workbenchContext, layerManagerProxy, taskFrame);   
 }
 }

Modified: core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
===
--- core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java  
2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java  
2015-12-24 12:33:54 UTC (rev 4666)
@@ -224,7 +224,7 @@
 return task.getLayerManager();
 }
 
-public InfoFrame getInfoFrame() throws IOException {
+public InfoFrame getInfoFrame() {
 if (infoFrame == null || infoFrame.isClosed()) {
 infoFrame = new PrimaryInfoFrame(workbenchContext, this, this);
 }

Modified: core/trunk/src/com/vividsolutions/wms/AbstractParser.java
===
--- core/trunk/src/com/vividsolutions/wms/AbstractParser.java   2015-12-24 
12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/wms/AbstractParser.java   2015-12-24 
12:33:54 UTC (rev 4666)
@@ -220,20 +220,20 @@
 boundingBoxList.add ( new BoundingBox("Geographics", 
geographicBBox.getEnvelope()) );
 } else if( n.getNodeName().equals( "Layer" ) ) {
 subLayers.add( wmsLayerFromNode( n ) );
-} else if (n.getNodeName().equals("Style")) { //$NON-NLS-1$
-   

Re: [JPP-Devel] SVN: [4665] core/trunk

2015-12-24 Thread Rahkonen Jukka (MML)
Hi,

It looks that GetMap is not removed from the GetFeatureInfo request which makes 
at least Geoserver to fail

Single value expected for request parameter request but instead 
found: [GetMap, GetFeatureInfo]

-Jukka-


-Alkuperäinen viesti-
Lähettäjä: jump-pilot-...@lists.sourceforge.net 
[mailto:jump-pilot-...@lists.sourceforge.net] 
Lähetetty: 24. joulukuuta 2015 14:03
Vastaanottaja: jump-pilot-devel@lists.sourceforge.net
Aihe: [JPP-Devel] SVN: [4665] core/trunk

Revision: 4665
  http://sourceforge.net/p/jump-pilot/code/4665
Author:   bertazza
Date: 2015-12-24 12:03:26 + (Thu, 24 Dec 2015)
Log Message:
---
Info feature tool for WMS: info shown only for visible layers, added 
&FEATURE_COUNT=10 parameter, and changed method to retrieve url as suggested by 
Jukka (should now support basic authentication).

Modified Paths:
--
core/trunk/ChangeLog

core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java

Modified: core/trunk/ChangeLog
===
--- core/trunk/ChangeLog2015-12-23 23:13:38 UTC (rev 4664)
+++ core/trunk/ChangeLog2015-12-24 12:03:26 UTC (rev 4665)
@@ -4,6 +4,11 @@
 #< 80 chars 
-->#
 
 2015-12-24 bertazza
+  * Info feature tool for WMS: info shown only for visible layers, added
+&FEATURE_COUNT=10 parameter, and changed method to retrieve url as
+suggested by Jukka (should now support basic authentication).
+
+2015-12-24 bertazza
   * Fixed broken 4663 commit
 
 2015-12-23 bertazza

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
===
--- 
core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
 2015-12-23 23:13:38 UTC (rev 4664)
+++ 
core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
 2015-12-24 12:03:26 UTC (rev 4665)
@@ -113,11 +113,20 @@
 
 // WMS
 List wmsLay_l = 
getWorkbench().getContext().getLayerManager().getLayerables(WMSLayer.class);
+//Iterator iter = 
getWorkbench().getContext().getLayerNamePanel().selectedNodes(WMSLayer.class).iterator();
+
 String response = "";
-for(Layerable lay : wmsLay_l) {
+for(Layerable lay : wmsLay_l) {   
+
+// We only want visible layers
 WMSLayer wmsLayer = (WMSLayer) lay;
+if(!wmsLayer.isVisible()) {
+continue;
+}
 
-String featInfoUrl = 
wmsLayer.getService().getCapabilities().getFeatureInfoURL(); 
+//String featInfoUrl = 
wmsLayer.getService().getCapabilities().getFeatureInfoURL();
+String featInfoUrl = 
wmsLayer.createRequest(getWorkbench().getContext().getLayerViewPanel()).getURL().toString();
+
 String names = getWmsLayeNames(wmsLayer);
 
 Point2D point = getPanel().getViewport().toViewPoint(coord);
@@ -128,6 +137,7 @@
 } else {
 featInfoUrl += "?";
 }
+
 String version = wmsLayer.getWmsVersion();
 if (WMService.WMS_1_0_0.equals(version)) {
 featInfoUrl += "REQUEST=feature_info&WMTVER=1.0.0";
@@ -162,6 +172,8 @@
 }
 }
 
+featInfoUrl = featInfoUrl.concat("&FEATURE_COUNT=10 ");
+
 URL url = stripXhtmlTags(featInfoUrl);
 
 String newLine = System.getProperty("line.separator");


--
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

--
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


Re: [JPP-Devel] SVN: [4666] core/trunk

2015-12-24 Thread Alberto De Luca
Jukka, what error do you get? It builds for me...

Alberto

On 24 December 2015 at 13:58, Rahkonen Jukka (MML) <
jukka.rahko...@maanmittauslaitos.fi> wrote:

> Hi,
>
> R4666 did not build
>
>
> http://sourceforge.net/projects/jump-pilot/files/OpenJUMP_snapshots/OpenJUMP-20151224-r4666.log/download
>
> -Jukka-
>
> -Alkuperäinen viesti-
> Lähettäjä: jump-pilot-...@lists.sourceforge.net [mailto:
> jump-pilot-...@lists.sourceforge.net]
> Lähetetty: 24. joulukuuta 2015 14:34
> Vastaanottaja: jump-pilot-devel@lists.sourceforge.net
> Aihe: [JPP-Devel] SVN: [4666] core/trunk
>
> Revision: 4666
>   http://sourceforge.net/p/jump-pilot/code/4666
> Author:   bertazza
> Date: 2015-12-24 12:33:54 + (Thu, 24 Dec 2015)
> Log Message:
> ---
> Removed IOExpcetion throw from TaskFrame
>
> Modified Paths:
> --
> core/trunk/ChangeLog
> core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
>
> core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
> core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
> core/trunk/src/com/vividsolutions/wms/AbstractParser.java
> core/trunk/src/com/vividsolutions/wms/Parser.java
>
> Modified: core/trunk/ChangeLog
> ===
> --- core/trunk/ChangeLog2015-12-24 12:03:26 UTC (rev 4665)
> +++ core/trunk/ChangeLog2015-12-24 12:33:54 UTC (rev 4666)
> @@ -4,6 +4,9 @@
>  #< 80 chars
> -->#
>
>  2015-12-24 bertazza
> +  * Removed IOExpcetion throw from TaskFrame
> +
> +2015-12-24 bertazza
>* Info feature tool for WMS: info shown only for visible layers, added
>  &FEATURE_COUNT=10 parameter, and changed method to retrieve url as
>  suggested by Jukka (should now support basic authentication).
>
> Modified:
> core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
> ===
> --- core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
> 2015-12-24 12:03:26 UTC (rev 4665)
> +++ core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
> 2015-12-24 12:33:54 UTC (rev 4666)
> @@ -108,7 +108,7 @@
>  public InfoFrame(
>  WorkbenchContext workbenchContext,
>  LayerManagerProxy layerManagerProxy,
> -final TaskFrame taskFrame) throws IOException {
> +final TaskFrame taskFrame) {
> blackboard =
> PersistentBlackboardPlugIn.get(workbenchContext);
>  geometryInfoTab = new GeometryInfoTab(model, workbenchContext);
>  rasterInfoTab = new RasterInfoTab(null, null); @@ -375,7 +375,7 @@
>
>  private final JEditorPane jEditorPane;
>
> -public WMSInfoTab() throws IOException {
> +public WMSInfoTab() {
>
>  setLayout(new BorderLayout());
>
>
> Modified:
> core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
> ===
> ---
> core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
>  2015-12-24 12:03:26 UTC (rev 4665)
> +++
> core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
>  2015-12-24 12:33:54 UTC (rev 4666)
> @@ -45,7 +45,7 @@
>   * positions InfoFrames differently depending on whether or not they are
> primary.
>   */
>  public class PrimaryInfoFrame extends InfoFrame {
> -public PrimaryInfoFrame(WorkbenchContext workbenchContext,
> LayerManagerProxy layerManagerProxy, TaskFrame taskFrame) throws
> IOException {
> +public PrimaryInfoFrame(WorkbenchContext workbenchContext,
> + LayerManagerProxy layerManagerProxy, TaskFrame taskFrame) {
>  super(workbenchContext, layerManagerProxy, taskFrame);
>  }
>  }
>
> Modified:
> core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
> ===
> --- core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
> 2015-12-24 12:03:26 UTC (rev 4665)
> +++ core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
> 2015-12-24 12:33:54 UTC (rev 4666)
> @@ -224,7 +224,7 @@
>  return task.getLayerManager();
>  }
>
> -public InfoFrame getInfoFrame() throws IOException {
> +public InfoFrame getInfoFrame() {
>  if (infoFrame == null || infoFrame.isClosed()) {
>  infoFrame = new PrimaryInfoFrame(workbenchContext, this,
> this);
>  }
>
> Modified: core/trunk/src/com/vividsolutions/wms/AbstractParser.j

Re: [JPP-Devel] SVN: [4666] core/trunk

2015-12-24 Thread Rahkonen Jukka (MML)
Hi,

I do not build myself, the link above contains the error from the Ede’s 
snapshot build system.

-Jukka-

Lähettäjä: Alberto De Luca [mailto:berta...@gmail.com]
Lähetetty: 24. joulukuuta 2015 15:34
Vastaanottaja: OpenJump develop and use
Aihe: Re: [JPP-Devel] SVN: [4666] core/trunk

Jukka, what error do you get? It builds for me...

Alberto

On 24 December 2015 at 13:58, Rahkonen Jukka (MML) 
mailto:jukka.rahko...@maanmittauslaitos.fi>>
 wrote:
Hi,

R4666 did not build

http://sourceforge.net/projects/jump-pilot/files/OpenJUMP_snapshots/OpenJUMP-20151224-r4666.log/download

-Jukka-

-Alkuperäinen viesti-
Lähettäjä: 
jump-pilot-...@lists.sourceforge.net<mailto:jump-pilot-...@lists.sourceforge.net>
 
[mailto:jump-pilot-...@lists.sourceforge.net<mailto:jump-pilot-...@lists.sourceforge.net>]
Lähetetty: 24. joulukuuta 2015 14:34
Vastaanottaja: 
jump-pilot-devel@lists.sourceforge.net<mailto:jump-pilot-devel@lists.sourceforge.net>
Aihe: [JPP-Devel] SVN: [4666] core/trunk

Revision: 4666
  http://sourceforge.net/p/jump-pilot/code/4666
Author:   bertazza
Date: 2015-12-24 12:33:54 + (Thu, 24 Dec 2015)
Log Message:
---
Removed IOExpcetion throw from TaskFrame

Modified Paths:
--
core/trunk/ChangeLog
core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
core/trunk/src/com/vividsolutions/wms/AbstractParser.java
core/trunk/src/com/vividsolutions/wms/Parser.java

Modified: core/trunk/ChangeLog
===
--- core/trunk/ChangeLog2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/ChangeLog2015-12-24 12:33:54 UTC (rev 4666)
@@ -4,6 +4,9 @@
 #< 80 chars 
-->#

 2015-12-24 bertazza
+  * Removed IOExpcetion throw from TaskFrame
+
+2015-12-24 bertazza
   * Info feature tool for WMS: info shown only for visible layers, added
 &FEATURE_COUNT=10 parameter, and changed method to retrieve url as
 suggested by Jukka (should now support basic authentication).

Modified: core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
===
--- core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java  
2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java  
2015-12-24 12:33:54 UTC (rev 4666)
@@ -108,7 +108,7 @@
 public InfoFrame(
 WorkbenchContext workbenchContext,
 LayerManagerProxy layerManagerProxy,
-final TaskFrame taskFrame) throws IOException {
+final TaskFrame taskFrame) {
blackboard = PersistentBlackboardPlugIn.get(workbenchContext);
 geometryInfoTab = new GeometryInfoTab(model, workbenchContext);
 rasterInfoTab = new RasterInfoTab(null, null); @@ -375,7 +375,7 @@

 private final JEditorPane jEditorPane;

-public WMSInfoTab() throws IOException {
+public WMSInfoTab() {

 setLayout(new BorderLayout());


Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
===
--- core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java   
2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java   
2015-12-24 12:33:54 UTC (rev 4666)
@@ -45,7 +45,7 @@
  * positions InfoFrames differently depending on whether or not they are 
primary.
  */
 public class PrimaryInfoFrame extends InfoFrame {
-public PrimaryInfoFrame(WorkbenchContext workbenchContext, 
LayerManagerProxy layerManagerProxy, TaskFrame taskFrame) throws IOException {
+public PrimaryInfoFrame(WorkbenchContext workbenchContext,
+ LayerManagerProxy layerManagerProxy, TaskFrame taskFrame) {
 super(workbenchContext, layerManagerProxy, taskFrame);
 }
 }

Modified: core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
===
--- core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java  
2015-12-24 12:03:26 UTC (rev 4665)
+++ core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java  
2015-12-24 12:33:54 UTC (rev 4666)
@@ -224,7 +224,7 @@
 return task.getLayerManager();
 }

-public InfoFrame getInfoFrame() throws IOException {
+public InfoFrame getInfoFrame() {
 if (infoFrame == null || infoFrame.isClosed()) {
 infoFrame = new PrimaryInfoFrame(workbenchContext, this, this);
 }

Modified: core/trunk/src/com/vividsolutions/wms/AbstractParser.java
===
--- core/trunk/src/com/vividsolutions

[JPP-Devel] SVN: [4667] core/trunk

2015-12-24 Thread jump-pilot-svn
Revision: 4667
  http://sourceforge.net/p/jump-pilot/code/4667
Author:   bertazza
Date: 2015-12-24 14:05:02 + (Thu, 24 Dec 2015)
Log Message:
---
Fixed broken 4666 commit
Info feature tool: fixed issue related to dirty url

Modified Paths:
--
core/trunk/ChangeLog

core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
core/trunk/src/org/openjump/core/ui/plugin/queries/QueryDialog.java

Modified: core/trunk/ChangeLog
===
--- core/trunk/ChangeLog2015-12-24 12:33:54 UTC (rev 4666)
+++ core/trunk/ChangeLog2015-12-24 14:05:02 UTC (rev 4667)
@@ -4,6 +4,10 @@
 #< 80 chars 
-->#
 
 2015-12-24 bertazza
+  * Fixed broken 4666 commit
+  * Info feature tool: fixed issue related to dirty url
+  
+2015-12-24 bertazza
   * Removed IOExpcetion throw from TaskFrame
 
 2015-12-24 bertazza

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
===
--- 
core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
 2015-12-24 12:33:54 UTC (rev 4666)
+++ 
core/trunk/src/com/vividsolutions/jump/workbench/ui/cursortool/FeatureInfoTool.java
 2015-12-24 14:05:02 UTC (rev 4667)
@@ -124,9 +124,13 @@
 continue;
 }
 
-//String featInfoUrl = 
wmsLayer.getService().getCapabilities().getFeatureInfoURL();
-String featInfoUrl = 
wmsLayer.createRequest(getWorkbench().getContext().getLayerViewPanel()).getURL().toString();
+String featInfoUrl = 
wmsLayer.getService().getCapabilities().getFeatureInfoURL();
+String userInfo = 
wmsLayer.createRequest(getWorkbench().getContext().getLayerViewPanel()).getURL().getUserInfo();
 
+if(userInfo != null) {
+featInfoUrl = featInfoUrl.concat(userInfo);
+}
+
 String names = getWmsLayeNames(wmsLayer);
 
 Point2D point = getPanel().getViewport().toViewPoint(coord);

Modified: core/trunk/src/org/openjump/core/ui/plugin/queries/QueryDialog.java
===
--- core/trunk/src/org/openjump/core/ui/plugin/queries/QueryDialog.java 
2015-12-24 12:33:54 UTC (rev 4666)
+++ core/trunk/src/org/openjump/core/ui/plugin/queries/QueryDialog.java 
2015-12-24 14:05:02 UTC (rev 4667)
@@ -903,13 +903,9 @@
 // initialization for infoframe
 InfoFrame info = null;
 if(display.getState()) {
-try {
-info = new InfoFrame(context.getWorkbenchContext(),
-(LayerManagerProxy)context,
-
(TaskFrame)context.getWorkbenchFrame().getActiveInternalFrame());
-} catch (IOException ex) {
-ex.printStackTrace();
-}
+info = new InfoFrame(context.getWorkbenchContext(),
+(LayerManagerProxy)context,
+
(TaskFrame)context.getWorkbenchFrame().getActiveInternalFrame());
 }
 
 // Loop on the requested layers


--
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


Re: [JPP-Devel] SVN: [4666] core/trunk

2015-12-24 Thread Alberto De Luca
Right, got it... NetBeans would build anyway, needed to clean the project
to see it...

Give it a try. I also tried to solve the problem with GeoServer. Let me
know.

Alberto

On 24 December 2015 at 14:52, Rahkonen Jukka (MML) <
jukka.rahko...@maanmittauslaitos.fi> wrote:

> Hi,
>
>
>
> I do not build myself, the link above contains the error from the Ede’s
> snapshot build system.
>
>
>
> -Jukka-
>
>
>
> *Lähettäjä:* Alberto De Luca [mailto:berta...@gmail.com]
> *Lähetetty:* 24. joulukuuta 2015 15:34
> *Vastaanottaja:* OpenJump develop and use
> *Aihe:* Re: [JPP-Devel] SVN: [4666] core/trunk
>
>
>
> Jukka, what error do you get? It builds for me...
>
>
>
> Alberto
>
>
>
> On 24 December 2015 at 13:58, Rahkonen Jukka (MML) <
> jukka.rahko...@maanmittauslaitos.fi> wrote:
>
> Hi,
>
> R4666 did not build
>
>
> http://sourceforge.net/projects/jump-pilot/files/OpenJUMP_snapshots/OpenJUMP-20151224-r4666.log/download
>
> -Jukka-
>
> -Alkuperäinen viesti-
> Lähettäjä: jump-pilot-...@lists.sourceforge.net [mailto:
> jump-pilot-...@lists.sourceforge.net]
> Lähetetty: 24. joulukuuta 2015 14:34
> Vastaanottaja: jump-pilot-devel@lists.sourceforge.net
> Aihe: [JPP-Devel] SVN: [4666] core/trunk
>
> Revision: 4666
>   http://sourceforge.net/p/jump-pilot/code/4666
> Author:   bertazza
> Date: 2015-12-24 12:33:54 + (Thu, 24 Dec 2015)
> Log Message:
> ---
> Removed IOExpcetion throw from TaskFrame
>
> Modified Paths:
> --
> core/trunk/ChangeLog
> core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
>
> core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
> core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
> core/trunk/src/com/vividsolutions/wms/AbstractParser.java
> core/trunk/src/com/vividsolutions/wms/Parser.java
>
> Modified: core/trunk/ChangeLog
> ===
> --- core/trunk/ChangeLog2015-12-24 12:03:26 UTC (rev 4665)
> +++ core/trunk/ChangeLog2015-12-24 12:33:54 UTC (rev 4666)
> @@ -4,6 +4,9 @@
>  #< 80 chars
> -->#
>
>  2015-12-24 bertazza
> +  * Removed IOExpcetion throw from TaskFrame
> +
> +2015-12-24 bertazza
>* Info feature tool for WMS: info shown only for visible layers, added
>  &FEATURE_COUNT=10 parameter, and changed method to retrieve url as
>  suggested by Jukka (should now support basic authentication).
>
> Modified:
> core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
> ===
> --- core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
> 2015-12-24 12:03:26 UTC (rev 4665)
> +++ core/trunk/src/com/vividsolutions/jump/workbench/ui/InfoFrame.java
> 2015-12-24 12:33:54 UTC (rev 4666)
> @@ -108,7 +108,7 @@
>  public InfoFrame(
>  WorkbenchContext workbenchContext,
>  LayerManagerProxy layerManagerProxy,
> -final TaskFrame taskFrame) throws IOException {
> +final TaskFrame taskFrame) {
> blackboard =
> PersistentBlackboardPlugIn.get(workbenchContext);
>  geometryInfoTab = new GeometryInfoTab(model, workbenchContext);
>  rasterInfoTab = new RasterInfoTab(null, null); @@ -375,7 +375,7 @@
>
>  private final JEditorPane jEditorPane;
>
> -public WMSInfoTab() throws IOException {
> +public WMSInfoTab() {
>
>  setLayout(new BorderLayout());
>
>
> Modified:
> core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
> ===
> ---
> core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
>  2015-12-24 12:03:26 UTC (rev 4665)
> +++
> core/trunk/src/com/vividsolutions/jump/workbench/ui/PrimaryInfoFrame.java
>  2015-12-24 12:33:54 UTC (rev 4666)
> @@ -45,7 +45,7 @@
>   * positions InfoFrames differently depending on whether or not they are
> primary.
>   */
>  public class PrimaryInfoFrame extends InfoFrame {
> -public PrimaryInfoFrame(WorkbenchContext workbenchContext,
> LayerManagerProxy layerManagerProxy, TaskFrame taskFrame) throws
> IOException {
> +public PrimaryInfoFrame(WorkbenchContext workbenchContext,
> + LayerManagerProxy layerManagerProxy, TaskFrame taskFrame) {
>  super(workbenchContext, layerManagerProxy, taskFrame);
>  }
>  }
>
> Modified:
> core/trunk/src/com/vividsolutions/jump/workbench/ui/TaskFrame.java
> ===

Re: [JPP-Devel] update sqlite

2015-12-24 Thread Larry Reeder
Ede and other OJ'ers,

An early Merry Christmas to you!  I've released a new version of the
DBQuery plugin.  No significant changes - I just added a default properties
file and updated SQLite to v3.8.11.2 as Ede suggested.

Jukka, once things settle down after Christmas, I will look at patching the
Xerial SQLite JDBC libraries to do a better job of handling the various
timestamp formats supported by SQLite.

 -lreeder



On Wed, Dec 16, 2015 at 9:56 PM, Larry Reeder  wrote:

> Yes, that's a good idea, Ede.  I'll try out the newer SQLite jars.
>
>  -lreeder
>
> On Wed, Dec 16, 2015 at 4:28 AM,  wrote:
>
>> hey Larry,
>>
>> when you fix up DBQuery would it make sense to update the libs? i see
>> there is at least a new
>>
>> sqlite-jdbc-3.8.11.2.jar
>>
>> out there.
>>
>> ..ede
>>
>>
>
--
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


Re: [JPP-Devel] update sqlite

2015-12-24 Thread Larry Reeder
Forgot the link:

https://sourceforge.net/projects/jumpdbqplugin/files/jumpdbqplugin/jumpdbqplugin-1.1.1/

 -lreeder

On Thu, Dec 24, 2015 at 8:44 AM, Larry Reeder  wrote:

> Ede and other OJ'ers,
>
> An early Merry Christmas to you!  I've released a new version of the
> DBQuery plugin.  No significant changes - I just added a default properties
> file and updated SQLite to v3.8.11.2 as Ede suggested.
>
> Jukka, once things settle down after Christmas, I will look at patching
> the Xerial SQLite JDBC libraries to do a better job of handling the various
> timestamp formats supported by SQLite.
>
>  -lreeder
>
>
>
> On Wed, Dec 16, 2015 at 9:56 PM, Larry Reeder  wrote:
>
>> Yes, that's a good idea, Ede.  I'll try out the newer SQLite jars.
>>
>>  -lreeder
>>
>> On Wed, Dec 16, 2015 at 4:28 AM,  wrote:
>>
>>> hey Larry,
>>>
>>> when you fix up DBQuery would it make sense to update the libs? i see
>>> there is at least a new
>>>
>>> sqlite-jdbc-3.8.11.2.jar
>>>
>>> out there.
>>>
>>> ..ede
>>>
>>>
>>
>
--
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


[JPP-Devel] happy holidays to the OpenJUMPers

2015-12-24 Thread Stefan Steiniger
Hey guys,

just want to send you my happy holiday wishes and congratulate to the 
tremendous work that you guys have been doing, in particular the last 
month! :).

I just looked at the stats and am impressed: OpenJUMP 1.8 has been 
downloaded around 28.000 times (!!) since its release.
Its kind of weird, if you meet GIS folks then they are "Ohhh OpenJUMP, 
is it still around?". But if I look at the download stats now, it has 
never been better/healthier (in terms of number of downloads). :)
And, perhaps for a real surveying CAD or an all-extension-loaded-ArcGIS, 
OpenJUMP is still the best tool for creation, editing and cleaning of 
features and layers, I think.

Top five download countries are the US:6000 (well, huge population), 
Germany: 3700, France: 1650 (both always strong), Brazil: 1500 
(surprise!) and Italy: 1400 (Peppe + Alberto). If I actually sum up the 
Spanish speaking countries then this group comes into third place 
(2500), before France.
Anyway, I think this shows that each ones effort into translation can be 
seen by its home country :) ... well perhaps we need some "Pro" for 
Portuguese?

so, keep up the good work!

And Ede: let me know when we are like 5 days away from a release; to 
dedicate some hours with translations.

cheers and best holiday wishes from the southern hemisphere where I am 
getting toasted at 30°, phewww... not my thing at Xmas.

cheers,
stefan

PS: I am still following the emails on this list, if I can.

--
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


Re: [JPP-Devel] happy holidays to the OpenJUMPers

2015-12-24 Thread Giuseppe Aruta
Hey Stefan!
Happy Holydays and Merry Christmass to you and to all OJ members.
I am at the Southern Emisphere "tambien" and met Stefan a week ago (It was
nice to see again after 2 years! - like Second International OpenJUPers
Meeting in South America).
We are doing a great job and think we still have many "trump cards" to play
on next weeks.
Thanks to everyone, thanks to Ede for his work on SVN and the patience to
solve my mistakes!!
Best regards from 38° Celsius pre-Xmass
Peppe

2015-12-24 18:18 GMT+01:00 Stefan Steiniger :

> Hey guys,
>
> just want to send you my happy holiday wishes and congratulate to the
> tremendous work that you guys have been doing, in particular the last
> month! :).
>
> I just looked at the stats and am impressed: OpenJUMP 1.8 has been
> downloaded around 28.000 times (!!) since its release.
> Its kind of weird, if you meet GIS folks then they are "Ohhh OpenJUMP,
> is it still around?". But if I look at the download stats now, it has
> never been better/healthier (in terms of number of downloads). :)
> And, perhaps for a real surveying CAD or an all-extension-loaded-ArcGIS,
> OpenJUMP is still the best tool for creation, editing and cleaning of
> features and layers, I think.
>
> Top five download countries are the US:6000 (well, huge population),
> Germany: 3700, France: 1650 (both always strong), Brazil: 1500
> (surprise!) and Italy: 1400 (Peppe + Alberto). If I actually sum up the
> Spanish speaking countries then this group comes into third place
> (2500), before France.
> Anyway, I think this shows that each ones effort into translation can be
> seen by its home country :) ... well perhaps we need some "Pro" for
> Portuguese?
>
> so, keep up the good work!
>
> And Ede: let me know when we are like 5 days away from a release; to
> dedicate some hours with translations.
>
> cheers and best holiday wishes from the southern hemisphere where I am
> getting toasted at 30°, phewww... not my thing at Xmas.
>
> cheers,
> stefan
>
> PS: I am still following the emails on this list, if I can.
>
>
> --
> ___
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
--
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel