Revision: 5097 http://sourceforge.net/p/jump-pilot/code/5097 Author: michaudm Date: 2016-10-24 19:04:27 +0000 (Mon, 24 Oct 2016) Log Message: ----------- Clean some useless log message in ViewManager
Modified Paths: -------------- plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/StyledLayer.java plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewManagerExtension.java plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewManagerPlugIn.java plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewSet.java plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewSetPanel.java plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/style/PLabelStyle.java plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/style/PVertexStyle.java Modified: plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/StyledLayer.java =================================================================== --- plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/StyledLayer.java 2016-10-24 19:02:25 UTC (rev 5096) +++ plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/StyledLayer.java 2016-10-24 19:04:27 UTC (rev 5097) @@ -1,9 +1,6 @@ package org.openjump.ext.viewmanager; -import com.vividsolutions.jump.workbench.model.Category; -import com.vividsolutions.jump.workbench.model.Layer; -import com.vividsolutions.jump.workbench.model.LayerManager; -import com.vividsolutions.jump.workbench.model.Layerable; +import com.vividsolutions.jump.workbench.model.*; import com.vividsolutions.jump.workbench.ui.renderer.style.*; import org.apache.log4j.Logger; import org.openjump.ext.viewmanager.style.*; @@ -156,31 +153,29 @@ layer.setEditable(editable); if (styles != null) { LOG.info("Nombre de styles enregistrés : " + styles.size()); - LOG.info("Styles : " + styles); for (PStyle pstyle : styles) { - LOG.info(" - transform " + pstyle + " to"); try { Style newStyle = pstyle.getStyle(lyr); - LOG.info(" - apply " + newStyle.getClass().getSimpleName()); Style rm = layer.getStyle(newStyle.getClass()); if (rm != null) { layer.removeStyle(rm); } else if (BasicStyle.class.isInstance(newStyle)) { layer.removeStyle(layer.getBasicStyle()); } + LOG.info(" - replace " + newStyle.getClass().getSimpleName()); layer.addStyle(newStyle); - //System.out.println(layer.getName() + ":" + newStyle.getClass().getSimpleName() + " : " + newStyle.isEnabled()); } catch (Exception e) { LOG.warn(pstyle, e); } } } + layer.getLayerManager().fireLayerChanged(layer, LayerEventType.APPEARANCE_CHANGED); } } - boolean accept(String categoryFilter, String layerFilter, Category category, Layerable layer) { - Pattern categoryPattern = null; - Pattern layerPattern = null; + private boolean accept(String categoryFilter, String layerFilter, Category category, Layerable layer) { + Pattern categoryPattern = Pattern.compile(".*"); + Pattern layerPattern = Pattern.compile(".*"); try { categoryPattern = getPattern(categoryFilter); layerPattern = getPattern(layerFilter); Modified: plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewManagerExtension.java =================================================================== --- plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewManagerExtension.java 2016-10-24 19:02:25 UTC (rev 5096) +++ plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewManagerExtension.java 2016-10-24 19:04:27 UTC (rev 5097) @@ -14,7 +14,7 @@ } public String getVersion() { - return "0.3.3 (2016-10-12)"; + return "0.3.4 (2016-10-24)"; } public void configure(PlugInContext context) throws Exception { Modified: plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewManagerPlugIn.java =================================================================== --- plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewManagerPlugIn.java 2016-10-24 19:02:25 UTC (rev 5096) +++ plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewManagerPlugIn.java 2016-10-24 19:04:27 UTC (rev 5097) @@ -134,7 +134,6 @@ saveViewSetAsButton.addActionListener(this); toolBar.add(saveViewSetAsButton); - //dialog.add(toolBar, BorderLayout.PAGE_START); dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE); dialog.add(toolBar, BorderLayout.NORTH); @@ -213,11 +212,6 @@ return dialog; } - //FilenameFilter xmlFileFilter = new FilenameFilter() { - // public boolean accept(File dir, String name) { - // return name.toLowerCase().endsWith(".xml"); - // } - //}; FileFilter xmlFileFilter = new FileFilter() { public boolean accept(File file) { @@ -326,14 +320,13 @@ public void saveViewSetAs() { JFileChooser jfc = new JFileChooser(viewDir); jfc.setDialogType(JFileChooser.SAVE_DIALOG); - //jfc.setFileSelectionMode(JFileChooser.FILES_ONLY); jfc.setFileFilter(xmlFileFilter); int r = jfc.showSaveDialog(dialog); if (r == JFileChooser.APPROVE_OPTION) { File viewSetFile = jfc.getSelectedFile(); try { if (currentViewSet == null) { - System.out.println("No current view set"); + context.getWorkbenchFrame().warnUser("No current view set"); return; } currentViewSet.setName(viewSetFile.getName()); Modified: plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewSet.java =================================================================== --- plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewSet.java 2016-10-24 19:02:25 UTC (rev 5096) +++ plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewSet.java 2016-10-24 19:04:27 UTC (rev 5097) @@ -134,7 +134,7 @@ } public static interface Listener { - public void actionPerformed(ViewSet viewSet, int mod, View view); + void actionPerformed(ViewSet viewSet, int mod, View view); } } Modified: plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewSetPanel.java =================================================================== --- plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewSetPanel.java 2016-10-24 19:02:25 UTC (rev 5096) +++ plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/ViewSetPanel.java 2016-10-24 19:04:27 UTC (rev 5097) @@ -2,7 +2,6 @@ import com.vividsolutions.jump.I18N; import com.vividsolutions.jump.workbench.plugin.PlugInContext; -import org.apache.log4j.Logger; import javax.swing.*; import javax.swing.event.DocumentEvent; @@ -26,10 +25,10 @@ */ public class ViewSetPanel extends JPanel { - I18N I18N_ = I18N.getInstance("view_manager"); + private I18N I18N_ = I18N.getInstance("view_manager"); - ViewSet viewSet; - PlugInContext context; + private ViewSet viewSet; + private PlugInContext context; public ViewSetPanel(final PlugInContext context, final ViewSet viewSet) { super(new GridBagLayout()); @@ -76,7 +75,7 @@ } } - class ViewPanel extends JPanel implements ActionListener { + private class ViewPanel extends JPanel implements ActionListener { ImageIcon deleteIcon = new ImageIcon(ViewSetPanel.class.getClassLoader().getResource("/images/delete.png")); ImageIcon replaceIcon = new ImageIcon(ViewSetPanel.class.getClassLoader().getResource("/images/replace.png")); @@ -203,7 +202,6 @@ constraints.weightx = 0.0; constraints.fill = GridBagConstraints.NONE; - //constraints.anchor = GridBagConstraints.EAST; constraints.gridx = 1; add(applyButton, constraints); constraints.gridx = 2; Modified: plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/style/PLabelStyle.java =================================================================== --- plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/style/PLabelStyle.java 2016-10-24 19:02:25 UTC (rev 5096) +++ plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/style/PLabelStyle.java 2016-10-24 19:04:27 UTC (rev 5097) @@ -83,7 +83,7 @@ public Style getStyle(Layerable layerable) { LabelStyle labelStyle = new LabelStyle(); - LOG.info(" getLabelStyle"); + //LOG.info(" getLabelStyle"); if (layerable instanceof Layer) { labelStyle.initialize((Layer) layerable); labelStyle.setEnabled(isEnabled()); @@ -93,7 +93,7 @@ labelStyle.setHideAtScale(hideAtScale); labelStyle.setScaleToHideAt(maxScale); - LOG.info(" getLabelStyle 1"); + //LOG.info(" getLabelStyle 1"); int s = 0; s += font.bold ? java.awt.Font.BOLD : 0; @@ -103,7 +103,7 @@ labelStyle.setColor(ColorUtil.decode(font.color)); labelStyle.setHeightAttribute(font.heightAttribute); - LOG.info(" getLabelStyle 2"); + //LOG.info(" getLabelStyle 2"); if (outline != null) { labelStyle.setOutlineShowing(outline.enabled); @@ -111,7 +111,7 @@ labelStyle.setOutlineWidth(outline.width); } else labelStyle.setOutlineShowing(false); - LOG.info(" getLabelStyle 3"); + //LOG.info(" getLabelStyle 3"); if (alignment != null) { labelStyle.setHorizontalAlignment(alignment.horizontalAlignment.alignment); @@ -120,11 +120,11 @@ labelStyle.setAngleAttribute(alignment.angleAttribute); } } - LOG.info(labelStyle); - LOG.info(" " + labelStyle.getAttribute()); - LOG.info(" " + labelStyle.isEnabled()); - LOG.info(" " + labelStyle.isScaling()); - LOG.info(" " + labelStyle.getHeight()); + //LOG.info(labelStyle); + //LOG.info(" " + labelStyle.getAttribute()); + //LOG.info(" " + labelStyle.isEnabled()); + //LOG.info(" " + labelStyle.isScaling()); + //LOG.info(" " + labelStyle.getHeight()); return labelStyle; } Modified: plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/style/PVertexStyle.java =================================================================== --- plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/style/PVertexStyle.java 2016-10-24 19:02:25 UTC (rev 5096) +++ plug-ins/ViewManagerPlugin/trunk/src/org/openjump/ext/viewmanager/style/PVertexStyle.java 2016-10-24 19:04:27 UTC (rev 5097) @@ -13,7 +13,7 @@ import java.awt.*; /** - * Created by UMichael on 13/06/2015. + * Persistent vertex style */ @XmlRootElement(name="vertexStyle") public class PVertexStyle extends AbstractPStyle { @@ -45,7 +45,6 @@ } public Style getStyle(Layerable layerable) { - //BasicStyle style = new BasicStyle(); VertexStyle vStyle; if (shape.equals("Circle")) { vStyle = new CircleVertexStyle(); @@ -68,12 +67,4 @@ return vStyle; } - //public Style getStyle(Layerable layerable) { - // return new VertexStyle() { - // @Override - // public void setEnabled(boolean enabled) { - // super.setEnabled(enabled); - // } - // }; - //} } ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel