Fix PREF handling (was not correct relative to the RFC!)

[jvcard.git] / src / be / nikiroo / jvcard / Data.java

diff --git a/src/be/nikiroo/jvcard/Data.java b/src/be/nikiroo/jvcard/Data.java
index ef44813a0fca7d62dcaf1ed032434c97a0960be2..4d3da7e4c7d37f0aa3bd1754065cfbcbdc0e4872 100644 (file)
--- a/src/be/nikiroo/jvcard/Data.java
+++ b/src/be/nikiroo/jvcard/Data.java
@@ -1,6 +1,7 @@
 package be.nikiroo.jvcard;
 
 import java.security.InvalidParameterException;
+import java.util.LinkedList;
 import java.util.List;
 
 /**
@@ -30,17 +31,17 @@ public class Data extends BaseClass<TypeInfo> {
         * @param types
         *            the types of this {@link Data}
         * @param name
-        *            its name
+        *            its name (<b>MUST NOT</b> be NULL)
         * @param value
-        *            its value
+        *            its value (<b>MUST NOT</b> be NULL)
         * @param group
-        *            its group if any
+        *            its group if any (or NULL if none)
         */
        public Data(List<TypeInfo> types, String name, String value, String group) {
                super(types);
 
-               this.name = name;
-               this.value = value;
+               this.name = name.toUpperCase();
+               this.value = value.toString(); // crash NOW if null
                this.group = group;
 
                b64 = -1;
@@ -68,7 +69,7 @@ public class Data extends BaseClass<TypeInfo> {
         * @return the value
         */
        public String getValue() {
-               return value;
+               return unescape(value);
        }
 
        /**
@@ -78,6 +79,25 @@ public class Data extends BaseClass<TypeInfo> {
         *            the new value
         */
        public void setValue(String value) {
+               setRawValue(escape(value));
+       }
+
+       /**
+        * Return the raw value of this {@link Data}
+        * 
+        * @return the raw value
+        */
+       public String getRawValue() {
+               return value;
+       }
+
+       /**
+        * Change the raw value of this {@link Data}
+        * 
+        * @param value
+        *            the new raw value
+        */
+       public void setRawValue(String value) {
                if ((value == null && this.value != null)
                                || (value != null && !value.equals(this.value))) {
                        this.value = value;
@@ -85,6 +105,45 @@ public class Data extends BaseClass<TypeInfo> {
                }
        }
 
+       /**
+        * Return the {@link List} of comma-listed values from this {@link Data}.
+        * 
+        * @return the {@link List} of values
+        */
+       public List<String> getValues() {
+               return getList(',');
+       }
+
+       /**
+        * Set the {@link List} of comma-listed values from this {@link Data}.
+        * 
+        * @param values
+        *            the {@link List} of values
+        */
+       public void setValues(List<String> values) {
+               setList(values, ',');
+       }
+
+       /**
+        * Return the {@link List} of semi-column-listed fields from this
+        * {@link Data}.
+        * 
+        * @return the {@link List} of values
+        */
+       public List<String> getFields() {
+               return getList(';');
+       }
+
+       /**
+        * Set the {@link List} of comma-listed values from this {@link Data}.
+        * 
+        * @param values
+        *            the {@link List} of values
+        */
+       public void setFields(List<String> values) {
+               setList(values, ';');
+       }
+
        /**
         * Return the group of this {@link Data}
         * 
@@ -117,6 +176,35 @@ public class Data extends BaseClass<TypeInfo> {
                return b64;
        }
 
+       /**
+        * Check if this {@link Data} is binary
+        * 
+        * @return TRUE if it is
+        */
+       public boolean isBinary() {
+               return b64 >= 0;
+       }
+
+       /**
+        * Return the preferred value of this {@link Data}, or
+        * {@link Integer#MAX_VALUE} if none.
+        * 
+        * @return the preferred value
+        */
+       public int getPreferred() {
+               for (TypeInfo type : this) {
+                       if (type.getName().equals("PRE")) {
+                               try {
+                                       return Integer.parseInt(type.getValue());
+                               } catch (NumberFormatException e) {
+                                       e.printStackTrace();
+                               }
+                       }
+               }
+
+               return Integer.MAX_VALUE;
+       }
+
        /**
         * Change the bkey of this {@link Data}
         * 
@@ -138,12 +226,62 @@ public class Data extends BaseClass<TypeInfo> {
        }
 
        /**
-        * Check if this {@link Data} is binary
+        * Return the {@link List} of sep-listed values from this {@link String}
+        * data.
         * 
-        * @return TRUE if it is
+        * @param value
+        *            the data
+        * 
+        * @param the
+        *            separator
+        * 
+        * @return the {@link List} of values
         */
-       public boolean isBinary() {
-               return b64 >= 0;
+       private List<String> getList(char sep) {
+               List<String> rep = new LinkedList<String>();
+
+               if (value != null && value.length() > 0) {
+                       int last = 0;
+                       for (int i = 0; i < value.length(); i++) {
+                               if (value.charAt(i) == sep
+                                               && (i == 0 || value.charAt(i - 1) != '\\')) {
+                                       rep.add(value.substring(last, i));
+                                       last = i + 1;
+                               }
+                       }
+
+                       if (last < value.length())
+                               rep.add(value.substring(last));
+               }
+
+               return rep;
+       }
+
+       /**
+        * Create the {@link String}-encoded {@link List} of sep-listed values from
+        * the given values.
+        * 
+        * @param values
+        *            the {@link List} of values
+        * 
+        * @param sep
+        *            the separator
+        * 
+        * @return the {@link String}
+        */
+       private void setList(List<String> values, char sep) {
+               StringBuilder builder = new StringBuilder();
+               boolean first = true;
+               for (String value : values) {
+                       if (!first)
+                               builder.append(sep);
+
+                       builder.append(escape(value));
+
+                       first = false;
+               }
+
+               value = builder.toString();
        }
 
        @Override
@@ -153,6 +291,6 @@ public class Data extends BaseClass<TypeInfo> {
 
        @Override
        public String getState() {
-               return "" + name + value + group;
+               return ("" + name + value + group).replace(' ', '_');
        }
 }