mstover1    2004/07/27 07:38:19

  Modified:    src/core/org/apache/jmeter/save/converters
                        BooleanPropertyConverter.java
                        IntegerPropertyConverter.java
                        LongPropertyConverter.java
                        MultiPropertyConverter.java
                        StringPropertyConverter.java
                        TestElementPropertyConverter.java
  Added:       src/core/org/apache/jmeter/save/converters
                        ConversionHelp.java
  Log:
  url encoding strings saved in .jmx file
  
  Revision  Changes    Path
  1.4       +3 -3      
jakarta-jmeter/src/core/org/apache/jmeter/save/converters/BooleanPropertyConverter.java
  
  Index: BooleanPropertyConverter.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jmeter/src/core/org/apache/jmeter/save/converters/BooleanPropertyConverter.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- BooleanPropertyConverter.java     21 Jun 2004 23:37:40 -0000      1.3
  +++ BooleanPropertyConverter.java     27 Jul 2004 14:38:18 -0000      1.4
  @@ -52,7 +52,7 @@
            MarshallingContext arg2)
      {
         BooleanProperty prop = (BooleanProperty)obj;
  -      writer.addAttribute("name",prop.getName());
  +      writer.addAttribute("name",ConversionHelp.encode(prop.getName()));
         writer.setValue(prop.getStringValue());
   
      }
  @@ -63,7 +63,7 @@
      public Object unmarshal(HierarchicalStreamReader reader,
            UnmarshallingContext arg1)
      {
  -      BooleanProperty prop = new 
BooleanProperty(reader.getAttribute("name"),Boolean.valueOf(reader.getValue()).booleanValue());
  +      BooleanProperty prop = new 
BooleanProperty(ConversionHelp.decode(reader.getAttribute("name")),Boolean.valueOf(reader.getValue()).booleanValue());
         return prop;
      }
   }
  
  
  
  1.3       +2 -2      
jakarta-jmeter/src/core/org/apache/jmeter/save/converters/IntegerPropertyConverter.java
  
  Index: IntegerPropertyConverter.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jmeter/src/core/org/apache/jmeter/save/converters/IntegerPropertyConverter.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- IntegerPropertyConverter.java     21 Jun 2004 23:37:40 -0000      1.2
  +++ IntegerPropertyConverter.java     27 Jul 2004 14:38:18 -0000      1.3
  @@ -51,7 +51,7 @@
            MarshallingContext arg2)
      {
         IntegerProperty prop = (IntegerProperty)obj;
  -      writer.addAttribute("name",prop.getName());
  +      writer.addAttribute("name",ConversionHelp.encode(prop.getName()));
         writer.setValue(prop.getStringValue());
      }
   
  @@ -61,7 +61,7 @@
      public Object unmarshal(HierarchicalStreamReader reader,
            UnmarshallingContext arg1)
      {
  -      IntegerProperty prop = new 
IntegerProperty(reader.getAttribute("name"),Integer.parseInt(reader.getValue()));
  +      IntegerProperty prop = new 
IntegerProperty(ConversionHelp.decode(reader.getAttribute("name")),Integer.parseInt(reader.getValue()));
         return prop;
      }
   }
  
  
  
  1.3       +2 -2      
jakarta-jmeter/src/core/org/apache/jmeter/save/converters/LongPropertyConverter.java
  
  Index: LongPropertyConverter.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jmeter/src/core/org/apache/jmeter/save/converters/LongPropertyConverter.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- LongPropertyConverter.java        21 Jun 2004 23:37:40 -0000      1.2
  +++ LongPropertyConverter.java        27 Jul 2004 14:38:18 -0000      1.3
  @@ -52,7 +52,7 @@
            MarshallingContext arg2)
      {
         LongProperty prop = (LongProperty)obj;
  -      writer.addAttribute("name",prop.getName());
  +      writer.addAttribute("name",ConversionHelp.encode(prop.getName()));
         writer.setValue(prop.getStringValue());
      }
   
  @@ -62,7 +62,7 @@
      public Object unmarshal(HierarchicalStreamReader reader,
            UnmarshallingContext arg1)
      {
  -      LongProperty prop = new 
LongProperty(reader.getAttribute("name"),Long.parseLong(reader.getValue()));
  +      LongProperty prop = new 
LongProperty(ConversionHelp.decode(reader.getAttribute("name")),Long.parseLong(reader.getValue()));
         return prop;
      }
   }
  
  
  
  1.3       +2 -2      
jakarta-jmeter/src/core/org/apache/jmeter/save/converters/MultiPropertyConverter.java
  
  Index: MultiPropertyConverter.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jmeter/src/core/org/apache/jmeter/save/converters/MultiPropertyConverter.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- MultiPropertyConverter.java       21 Jun 2004 23:37:40 -0000      1.2
  +++ MultiPropertyConverter.java       27 Jul 2004 14:38:18 -0000      1.3
  @@ -57,7 +57,7 @@
            MarshallingContext context)
      {
         MultiProperty prop = (MultiProperty)arg0;
  -      writer.addAttribute("name",prop.getName());
  +      writer.addAttribute("name",ConversionHelp.encode(prop.getName()));
         PropertyIterator iter = prop.iterator();
         while(iter.hasNext())
         {
  @@ -73,7 +73,7 @@
            UnmarshallingContext context)
      {
         MultiProperty prop = 
(MultiProperty)createCollection(context.getRequiredType());
  -      prop.setName(reader.getAttribute("name"));
  +      prop.setName(ConversionHelp.decode(reader.getAttribute("name")));
         while (reader.hasMoreChildren()) {
            reader.moveDown();
            JMeterProperty subProp = (JMeterProperty)readItem(reader, context, prop);
  
  
  
  1.5       +36 -34    
jakarta-jmeter/src/core/org/apache/jmeter/save/converters/StringPropertyConverter.java
  
  Index: StringPropertyConverter.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jmeter/src/core/org/apache/jmeter/save/converters/StringPropertyConverter.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- StringPropertyConverter.java      27 Jul 2004 14:19:40 -0000      1.4
  +++ StringPropertyConverter.java      27 Jul 2004 14:38:18 -0000      1.5
  @@ -13,7 +13,7 @@
    * See the License for the specific language governing permissions and
    * limitations under the License.
    * 
  -*/
  + */
   
   package org.apache.jmeter.save.converters;
   
  @@ -33,58 +33,60 @@
   
   /**
    * @author mstover
  - *
  - * To change the template for this generated type comment go to
  - * Window - Preferences - Java - Code Generation - Code and Comments
  + * 
  + * To change the template for this generated type comment go to Window -
  + * Preferences - Java - Code Generation - Code and Comments
    */
   public class StringPropertyConverter implements Converter
   {
  -   transient private static final Logger log = LoggingManager.getLoggerForClass();
  +   transient private static final Logger log = LoggingManager
  +         .getLoggerForClass();
   
  -    /** Returns the converter version; used to check for possible incompatibilities 
*/
  -     public static String getVersion(){      return "$Revision$";}
  +   /**
  +    * Returns the converter version; used to check for possible
  +    * incompatibilities
  +    */
  +   public static String getVersion()
  +   {
  +      return "$Revision$";
  +   }
   
  -   /* (non-Javadoc)
  +   /*
  +    * (non-Javadoc)
  +    * 
       * @see com.thoughtworks.xstream.converters.Converter#canConvert(java.lang.Class)
       */
      public boolean canConvert(Class arg0)
      {
  -      return StringProperty.class.equals(arg0);         
  +      return StringProperty.class.equals(arg0);
      }
   
  -   /* (non-Javadoc)
  -    * @see com.thoughtworks.xstream.converters.Converter#marshal(java.lang.Object, 
com.thoughtworks.xstream.io.HierarchicalStreamWriter, 
com.thoughtworks.xstream.converters.MarshallingContext)
  +   /*
  +    * (non-Javadoc)
  +    * 
  +    * @see com.thoughtworks.xstream.converters.Converter#marshal(java.lang.Object,
  +    *      com.thoughtworks.xstream.io.HierarchicalStreamWriter,
  +    *      com.thoughtworks.xstream.converters.MarshallingContext)
       */
      public void marshal(Object obj, HierarchicalStreamWriter writer,
            MarshallingContext arg2)
      {
  -      StringProperty prop = (StringProperty)obj;
  -      try
  -      {
  -         writer.addAttribute("name",URLEncoder.encode(prop.getName(),"UTF-8"));
  -      }
  -      catch (UnsupportedEncodingException e)
  -      {
  -         log.warn("System doesn't support utf-8",e);
  -      }
  -      writer.setValue(prop.getStringValue());
  +      StringProperty prop = (StringProperty) obj;
  +      writer.addAttribute("name", ConversionHelp.encode(prop.getName()));
  +      writer.setValue(ConversionHelp.encode(prop.getStringValue()));
      }
   
  -   /* (non-Javadoc)
  -    * @see 
com.thoughtworks.xstream.converters.Converter#unmarshal(com.thoughtworks.xstream.io.HierarchicalStreamReader,
 com.thoughtworks.xstream.converters.UnmarshallingContext)
  +   /*
  +    * (non-Javadoc)
  +    * 
  +    * @see 
com.thoughtworks.xstream.converters.Converter#unmarshal(com.thoughtworks.xstream.io.HierarchicalStreamReader,
  +    *      com.thoughtworks.xstream.converters.UnmarshallingContext)
       */
      public Object unmarshal(HierarchicalStreamReader reader,
            UnmarshallingContext arg1)
      {
  -      try
  -      {
  -         StringProperty prop = new 
StringProperty(URLDecoder.decode(reader.getAttribute("name"),"UTF-8"),reader.getValue());
  -         return prop;
  -      }
  -      catch (UnsupportedEncodingException e)
  -      {
  -         log.warn("System doesn't support utf-8",e);
  -         return null;
  -      }
  +      StringProperty prop = new StringProperty(ConversionHelp.decode(reader
  +            .getAttribute("name")), ConversionHelp.decode(reader.getValue()));
  +      return prop;
      }
  -}
  +}
  \ No newline at end of file
  
  
  
  1.3       +2 -2      
jakarta-jmeter/src/core/org/apache/jmeter/save/converters/TestElementPropertyConverter.java
  
  Index: TestElementPropertyConverter.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jmeter/src/core/org/apache/jmeter/save/converters/TestElementPropertyConverter.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- TestElementPropertyConverter.java 21 Jun 2004 23:37:40 -0000      1.2
  +++ TestElementPropertyConverter.java 27 Jul 2004 14:38:18 -0000      1.3
  @@ -66,7 +66,7 @@
            MarshallingContext context)
      {
         TestElementProperty prop = (TestElementProperty) arg0;
  -      writer.addAttribute("name", prop.getName());
  +      writer.addAttribute("name", ConversionHelp.encode(prop.getName()));
         writer.addAttribute("elementType", prop.getObjectValue().getClass()
               .getName());
         PropertyIterator iter = prop.iterator();
  @@ -89,7 +89,7 @@
         {
            TestElementProperty prop = (TestElementProperty) createCollection(context
                  .getRequiredType());
  -         prop.setName(reader.getAttribute("name"));
  +         prop.setName(ConversionHelp.decode(reader.getAttribute("name")));
            prop.setObjectValue((TestElement) Class.forName(
                  reader.getAttribute("elementType")).newInstance());
            while (reader.hasMoreChildren())
  
  
  
  1.1                  
jakarta-jmeter/src/core/org/apache/jmeter/save/converters/ConversionHelp.java
  
  Index: ConversionHelp.java
  ===================================================================
  /*
   * Created on Jul 27, 2004
   *
   * To change the template for this generated file go to
   * Window - Preferences - Java - Code Generation - Code and Comments
   */
  package org.apache.jmeter.save.converters;
  
  import java.io.UnsupportedEncodingException;
  import java.net.URLDecoder;
  import java.net.URLEncoder;
  
  import org.apache.jorphan.logging.LoggingManager;
  import org.apache.log.Logger;
  
  /**
   * @author mstover
   *
   * To change the template for this generated type comment go to
   * Window - Preferences - Java - Code Generation - Code and Comments
   */
  public class ConversionHelp
  {
     private static final String CHAR_SET = "UTF-8";
     transient private static final Logger log = LoggingManager.getLoggerForClass();
  
     public static String encode(String p)
     {
        try
        {
           String p1 = URLEncoder.encode(p,CHAR_SET);
           return p1;
        }
        catch (Exception e)
        {
           log.warn("System doesn't support utf-8",e);
           return p;
        }
     }
     
     public static String decode(String p)
     {
        try
        {
           return URLDecoder.decode(p,CHAR_SET);
        }
        catch (Exception e)
        {
           log.warn("System doesn't support utf-8",e);
           return p;
        }
     }
  }
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to