Hei Andreas,

please commit both. We trust you and it is not a core change, but rather 
some cosmetics.

stefan

Andreas Schmitz schrieb:
> Andreas Schmitz wrote:
> 
> 
> Hi,
> 
> I got another one.
> 
> When editing WMS requests and selecting some SRS in the combobox, this
> selection will vanish (and the first SRS will be selected again) when
> moving layers up and down.
> 
> The attached patch fixes the problem.
> 
> If nobody objects, I'll check it in.
> 
> Best regards, Andreas
> 
> 
> ------------------------------------------------------------------------
> 
> Index: 
> src/com/vividsolutions/jump/workbench/ui/plugin/wms/EditWMSQueryPanel.java
> ===================================================================
> --- 
> src/com/vividsolutions/jump/workbench/ui/plugin/wms/EditWMSQueryPanel.java    
>     (Revision 907)
> +++ 
> src/com/vividsolutions/jump/workbench/ui/plugin/wms/EditWMSQueryPanel.java    
>     (Arbeitskopie)
> @@ -36,6 +36,7 @@
>  import java.awt.Insets;
>  import java.util.Iterator;
>  import java.util.List;
> +
>  import javax.swing.BorderFactory;
>  import javax.swing.DefaultComboBoxModel;
>  import javax.swing.JComboBox;
> @@ -48,7 +49,6 @@
>  import com.vividsolutions.jump.I18N;
>  import com.vividsolutions.jump.workbench.plugin.EnableCheck;
>  import com.vividsolutions.jump.workbench.ui.InputChangedListener;
> -import com.vividsolutions.jump.workbench.ui.MultiInputDialog;
>  import com.vividsolutions.jump.workbench.ui.TransparencyPanel;
>  import com.vividsolutions.wms.WMService;
>  
> @@ -130,6 +130,13 @@
>      */
>      private void updateComboBox() {
>          String selectedSRS = (String) srsComboBox.getSelectedItem();
> +        
> +        // this method does get called many times when no SRS are available 
> here
> +        // this makes sure that the selected SRS stays selected when 
> available
> +        if(mapLayerPanel.commonSRSList().size() == 0) {
> +            return;
> +        }
> +        
>          comboBoxModel.removeAllElements();
>  
>          for (Iterator i = mapLayerPanel.commonSRSList().iterator(); 
> i.hasNext();) {
> 
> 
> ------------------------------------------------------------------------
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc.
> Still grepping through log files to find problems?  Stop.
> Now Search log events and configuration files using AJAX and a browser.
> Download your FREE copy of Splunk now >>  http://get.splunk.com/
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to