Bugzilla 322673
Tweaks to version textbox listeners
diff --git a/plugins/org.eclipse.tigerstripe.workbench.ui.base/src/java/org/eclipse/tigerstripe/workbench/ui/internal/editors/descriptor/plugins/OsgiPluggablePluginSection.java b/plugins/org.eclipse.tigerstripe.workbench.ui.base/src/java/org/eclipse/tigerstripe/workbench/ui/internal/editors/descriptor/plugins/OsgiPluggablePluginSection.java
index e29f0a0..83a6229 100644
--- a/plugins/org.eclipse.tigerstripe.workbench.ui.base/src/java/org/eclipse/tigerstripe/workbench/ui/internal/editors/descriptor/plugins/OsgiPluggablePluginSection.java
+++ b/plugins/org.eclipse.tigerstripe.workbench.ui.base/src/java/org/eclipse/tigerstripe/workbench/ui/internal/editors/descriptor/plugins/OsgiPluggablePluginSection.java
@@ -243,46 +243,58 @@
 
 	public void handleModifyText(ModifyEvent e) {
 		if (!isSilentUpdate()) {
+			
 			OSGIRef ref;
 			if (getPluginConfig() != null){
+				
 				ref = OSGIRef.parseRef(getPluginConfig().getVersion());
 
-
 				if (e.getSource() == minVersionText){
+					
 					if (minVersionText.getText().trim().equals("") ){
-						// But we MUST have a min version ?
+						
 						ref.setMinVersion(new Version("0.0.0"));
 						
 					} else {
+						
 						try {
 							Version v = new Version(minVersionText.getText().trim());
 							ref.setMinVersion(v);
 							
 						} catch (IllegalArgumentException iae){
-							// TODO Don't set this for now?
-							minVersionText.setText(ref.getMinVersion().toString());
+							
+							// not a valid OSGi ref, but we want to allow editing to continue.
+							markPageModified();
+							updateForm();
+							return;
 						}
 					}
 					if (!ref.toString().equals(((PluginConfig) getPluginConfig()).getVersion()))
 						((PluginConfig) getPluginConfig()).setVersion(ref.toString());
+					
 				} else if (e.getSource() == maxVersionText){
+					
 					if (maxVersionText.getText().trim().equals("") ){
+						
 						ref.setMaxVersion(null);
 					} else {
+						
 						try {
+							
 							Version v = new Version(maxVersionText.getText().trim());
 							ref.setMaxVersion(v);
 							
 						} catch (IllegalArgumentException iae){
-							// TODO Don't set this for now?
-							maxVersionText.setText(ref.getMaxVersion().toString());
 							
+							// not a valid OSGi ref, but we want to allow editing to continue.
+							markPageModified();
+							updateForm();
+							return;
 						}
 					}
 					if (!ref.toString().equals(((PluginConfig) getPluginConfig()).getVersion()))
 						((PluginConfig) getPluginConfig()).setVersion(ref.toString());
 				}
-				System.out.println("");
 			} 
 			markPageModified();
 		}