2006/unicorn/org/w3c/unicorn/tasklist RDFUnmarshaller.java,1.1.1.1,1.2 RDFUnmarshallerJena.java,1.3,1.4 TasksListUnmarshaller.java,1.3,1.4 TaskListUnmarshallerBeans.java,1.12,1.13 Task.java,1.4,1.5

Update of /sources/public/2006/unicorn/org/w3c/unicorn/tasklist
In directory hutz:/tmp/cvs-serv6157/org/w3c/unicorn/tasklist

Modified Files:
	RDFUnmarshaller.java RDFUnmarshallerJena.java 
	TasksListUnmarshaller.java TaskListUnmarshallerBeans.java 
	Task.java 
Log Message:
Code cleanup

Index: RDFUnmarshallerJena.java
===================================================================
RCS file: /sources/public/2006/unicorn/org/w3c/unicorn/tasklist/RDFUnmarshallerJena.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -d -r1.3 -r1.4
--- RDFUnmarshallerJena.java	26 Aug 2008 15:26:37 -0000	1.3
+++ RDFUnmarshallerJena.java	11 Aug 2009 13:43:00 -0000	1.4
@@ -26,11 +26,12 @@
 
 /**
  * @author Damien LEROY
- *
+ * 
  */
 public class RDFUnmarshallerJena implements RDFUnmarshaller {
 
-	private static final Log logger = LogFactory.getLog("org.w3c.unicorn.tasklist");
+	private static final Log logger = LogFactory
+			.getLog("org.w3c.unicorn.tasklist");
 
 	private static final Model MODEL = ModelFactory.createDefaultModel();
 
@@ -39,59 +40,82 @@
 	private static Resource RESOURCE_TASK = null;
 
 	private static Property PROPERTY_DESCRIPTION = null;
+
 	private static Property PROPERTY_HASMAPPING = null;
+
 	private static Property PROPERTY_HASPARAMETER = null;
+
 	private static Property PROPERTY_HASVALUE = null;
+
 	private static Property PROPERTY_LONGNAME = null;
+
 	private static Property PROPERTY_OBSERVER = null;
+
 	private static Property PROPERTY_PARAMETER = null;
+
 	private static Property PROPERTY_REFERENCE = null;
+
 	private static Property PROPERTY_TYPE = null;
+
 	private static Property PROPERTY_VALUE = null;
 
 	static {
-		RDFUnmarshallerJena.MODEL.read(
-				org.w3c.unicorn.util.Property.get("TASKLIST_RDF_MODEL"),
-				null);
+		RDFUnmarshallerJena.MODEL.read(org.w3c.unicorn.util.Property
+				.get("TASKLIST_RDF_MODEL"), null);
 
 		// define resource use to find information into the RDF graph
-		RDFUnmarshallerJena.RESOURCE_TASK = RDFUnmarshallerJena.MODEL.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE+"Task");
+		RDFUnmarshallerJena.RESOURCE_TASK = RDFUnmarshallerJena.MODEL
+				.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE + "Task");
 
 		// define property use to find information into the RDF graph
-		RDFUnmarshallerJena.PROPERTY_DESCRIPTION = RDFUnmarshallerJena.MODEL.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE+"description");
-		RDFUnmarshallerJena.PROPERTY_HASMAPPING = RDFUnmarshallerJena.MODEL.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE+"hasMapping");
-		RDFUnmarshallerJena.PROPERTY_HASPARAMETER = RDFUnmarshallerJena.MODEL.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE+"hasParameter");
-		RDFUnmarshallerJena.PROPERTY_HASVALUE = RDFUnmarshallerJena.MODEL.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE+"hasValue");
-		RDFUnmarshallerJena.PROPERTY_LONGNAME = RDFUnmarshallerJena.MODEL.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE+"longName");
-		RDFUnmarshallerJena.PROPERTY_OBSERVER = RDFUnmarshallerJena.MODEL.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE+"observer");
-		RDFUnmarshallerJena.PROPERTY_PARAMETER = RDFUnmarshallerJena.MODEL.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE+"parameter");
-		RDFUnmarshallerJena.PROPERTY_REFERENCE = RDFUnmarshallerJena.MODEL.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE+"reference");
-		RDFUnmarshallerJena.PROPERTY_TYPE = RDFUnmarshallerJena.MODEL.getProperty("http://www.w3.org/1999/02/22-rdf-syntax-ns#type");
-		RDFUnmarshallerJena.PROPERTY_VALUE = RDFUnmarshallerJena.MODEL.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE+"value");
+		RDFUnmarshallerJena.PROPERTY_DESCRIPTION = RDFUnmarshallerJena.MODEL
+				.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE + "description");
+		RDFUnmarshallerJena.PROPERTY_HASMAPPING = RDFUnmarshallerJena.MODEL
+				.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE + "hasMapping");
+		RDFUnmarshallerJena.PROPERTY_HASPARAMETER = RDFUnmarshallerJena.MODEL
+				.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE + "hasParameter");
+		RDFUnmarshallerJena.PROPERTY_HASVALUE = RDFUnmarshallerJena.MODEL
+				.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE + "hasValue");
+		RDFUnmarshallerJena.PROPERTY_LONGNAME = RDFUnmarshallerJena.MODEL
+				.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE + "longName");
+		RDFUnmarshallerJena.PROPERTY_OBSERVER = RDFUnmarshallerJena.MODEL
+				.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE + "observer");
+		RDFUnmarshallerJena.PROPERTY_PARAMETER = RDFUnmarshallerJena.MODEL
+				.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE + "parameter");
+		RDFUnmarshallerJena.PROPERTY_REFERENCE = RDFUnmarshallerJena.MODEL
+				.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE + "reference");
+		RDFUnmarshallerJena.PROPERTY_TYPE = RDFUnmarshallerJena.MODEL
+				.getProperty("http://www.w3.org/1999/02/22-rdf-syntax-ns#type");
+		RDFUnmarshallerJena.PROPERTY_VALUE = RDFUnmarshallerJena.MODEL
+				.getProperty(RDFUnmarshallerJena.UCN_NAMESPACE + "value");
 
 	}
 
 	private Map<String, Observer> mapOfObserver = null;
+
 	private Map<String, Task> mapOfTask = null;
+
 	private Model aModel = null;
 
 	/**
 	 * Default constructor.
-	 *
+	 * 
 	 */
-	public RDFUnmarshallerJena () {
+	public RDFUnmarshallerJena() {
 		RDFUnmarshallerJena.logger.trace("Constructor");
 		this.aModel = ModelFactory.createDefaultModel();
 	}
 
 	/**
 	 * Adds a Model with the given URL to this aModel.
-	 * @param aURL The URL to add.
+	 * 
+	 * @param aURL
+	 *            The URL to add.
 	 */
-	public void addURL (final URL aURL) throws IOException {
+	public void addURL(final URL aURL) throws IOException {
 		RDFUnmarshallerJena.logger.trace("addURL");
 		if (RDFUnmarshallerJena.logger.isDebugEnabled()) {
-			RDFUnmarshallerJena.logger.debug("URL : "+aURL+".");
+			RDFUnmarshallerJena.logger.debug("URL : " + aURL + ".");
 		}
 		final Model aModel = ModelFactory.createDefaultModel();
 		aModel.read(aURL.openStream(), null);
@@ -100,67 +124,88 @@
 
 	/**
 	 * Adds a name (with its language) to the given task.
-	 * @param aTask The task to name.
-	 * @param aLiteral The name of the task.
+	 * 
+	 * @param aTask
+	 *            The task to name.
+	 * @param aLiteral
+	 *            The name of the task.
 	 */
-	private void addLongName (final Task aTask, final Literal aLiteral) {
-		RDFUnmarshallerJena.logger.debug(
-				"LongName lang:" + aLiteral.getLanguage() +
-				" value:" + aLiteral.getString() + ".");
+	private void addLongName(final Task aTask, final Literal aLiteral) {
+		RDFUnmarshallerJena.logger.debug("LongName lang:"
+				+ aLiteral.getLanguage() + " value:" + aLiteral.getString()
+				+ ".");
 		aTask.addLongName(aLiteral.getLanguage(), aLiteral.getString());
 	}
 
 	/**
 	 * Adds a description (with its language) to the given task.
-	 * @param aTask The task to describe.
-	 * @param aLiteral The description of the task.
+	 * 
+	 * @param aTask
+	 *            The task to describe.
+	 * @param aLiteral
+	 *            The description of the task.
 	 */
-	private void addDescription (final Task aTask, final Literal aLiteral) {
-		RDFUnmarshallerJena.logger.debug(
-				"Description lang:" + aLiteral.getLanguage() +
-				" value:" + aLiteral.getString() + ".");
+	private void addDescription(final Task aTask, final Literal aLiteral) {
+		RDFUnmarshallerJena.logger.debug("Description lang:"
+				+ aLiteral.getLanguage() + " value:" + aLiteral.getString()
+				+ ".");
 		aTask.addDescription(aLiteral.getLanguage(), aLiteral.getString());
 	}
 
 	/**
 	 * Adds a name (with its language) to the given parameter.
-	 * @param aTask The parameter to name.
-	 * @param aLiteral The name of the parameter.
+	 * 
+	 * @param aTask
+	 *            The parameter to name.
+	 * @param aLiteral
+	 *            The name of the parameter.
 	 */
-	private void addLongName (final Parameter aParameter, final Literal aLiteral) {
-		RDFUnmarshallerJena.logger.debug(
-				"Parameter long name lang:" + aLiteral.getLanguage() +
-				" value:" + aLiteral.getString() + ".");
+	private void addLongName(final Parameter aParameter, final Literal aLiteral) {
+		RDFUnmarshallerJena.logger.debug("Parameter long name lang:"
+				+ aLiteral.getLanguage() + " value:" + aLiteral.getString()
+				+ ".");
 		aParameter.addLongName(aLiteral.getLanguage(), aLiteral.getString());
 	}
 
 	/**
 	 * Adds a name (with its language) to the given value.
-	 * @param aTask The value to name.
-	 * @param aLiteral The name of the value.
+	 * 
+	 * @param aTask
+	 *            The value to name.
+	 * @param aLiteral
+	 *            The name of the value.
 	 */
-	private void addLongName (final Value aValue, final Literal aLiteral) {
-		RDFUnmarshallerJena.logger.debug(
-				"Value long name lang:" + aLiteral.getLanguage() +
-				" value:" + aLiteral.getString() + ".");
+	private void addLongName(final Value aValue, final Literal aLiteral) {
+		RDFUnmarshallerJena.logger.debug("Value long name lang:"
+				+ aLiteral.getLanguage() + " value:" + aLiteral.getString()
+				+ ".");
 		aValue.addLongName(aLiteral.getLanguage(), aLiteral.getString());
 	}
 
 	/**
 	 * Adds a mapping to the given value.
-	 * @param value The value to map.
-	 * @param aMapping The mapping for the value.
+	 * 
+	 * @param value
+	 *            The value to map.
+	 * @param aMapping
+	 *            The mapping for the value.
 	 */
-	private void addMapping (final Value aValue, final Resource aMapping) {
+	private void addMapping(final Value aValue, final Resource aMapping) {
 		RDFUnmarshallerJena.logger.trace("addMapping");
 		if (RDFUnmarshallerJena.logger.isDebugEnabled()) {
-			RDFUnmarshallerJena.logger.debug("Value : "+aValue.getName()+".");
-			RDFUnmarshallerJena.logger.debug("Mapping : "+aMapping.getLocalName()+".");
+			RDFUnmarshallerJena.logger.debug("Value : " + aValue.getName()
+					+ ".");
+			RDFUnmarshallerJena.logger.debug("Mapping : "
+					+ aMapping.getLocalName() + ".");
 		}
 		// TODO verifier ce brol
-		final String sObserver = aMapping.getProperty(RDFUnmarshallerJena.PROPERTY_OBSERVER).getLiteral().getString();
-		final String sParameter = aMapping.getProperty(RDFUnmarshallerJena.PROPERTY_PARAMETER).getLiteral().getString();
-		final String sValue = aMapping.getProperty(RDFUnmarshallerJena.PROPERTY_VALUE).getLiteral().getString();
+		final String sObserver = aMapping.getProperty(
+				RDFUnmarshallerJena.PROPERTY_OBSERVER).getLiteral().getString();
+		final String sParameter = aMapping.getProperty(
+				RDFUnmarshallerJena.PROPERTY_PARAMETER).getLiteral()
+				.getString();
+		final String sValue = aMapping.getProperty(
+				RDFUnmarshallerJena.PROPERTY_VALUE).getLiteral().getString();
 		final Mapping oMapping = new Mapping();
 		oMapping.setObserver(this.mapOfObserver.get(sObserver));
 		oMapping.setParam(sParameter);
@@ -169,71 +214,74 @@
 
 	/**
 	 * Adds a value to the given parameter.
-	 * @param aParameter The parameter to consider.
-	 * @param aValue The value to add.
+	 * 
+	 * @param aParameter
+	 *            The parameter to consider.
+	 * @param aValue
+	 *            The value to add.
 	 */
-	private void addValue (final Parameter aParameter, final Resource aValue) {
-		final String sValue =
-			aValue.getProperty(RDFUnmarshallerJena.PROPERTY_VALUE).getLiteral().getString();
+	private void addValue(final Parameter aParameter, final Resource aValue) {
+		final String sValue = aValue.getProperty(
+				RDFUnmarshallerJena.PROPERTY_VALUE).getLiteral().getString();
 		final Value oValue = aParameter.getValue(sValue);
 		if (null == oValue) {
-			RDFUnmarshallerJena.logger.warn(
-					"Value " + sValue +
-					" not found in parameter " + aParameter.getName() + ".");
+			RDFUnmarshallerJena.logger.warn("Value " + sValue
+					+ " not found in parameter " + aParameter.getName() + ".");
 			return;
 		}
 		RDFUnmarshallerJena.logger.debug("Parameter value  : " + sValue + ".");
 		// find and add longName of the Value
-		for (
-				final StmtIterator siLongName = aValue.listProperties(
-						RDFUnmarshallerJena.PROPERTY_LONGNAME);
-				siLongName.hasNext();) {
+		for (final StmtIterator siLongName = aValue
+				.listProperties(RDFUnmarshallerJena.PROPERTY_LONGNAME); siLongName
+				.hasNext();) {
 			final Literal lLongName = siLongName.nextStatement().getLiteral();
 			this.addLongName(oValue, lLongName);
 		} // find and add longName of the Value
 		// TODO completer l'ajout de Value
 		// find and add mapping of the Value
-		for (
-				final StmtIterator siMapping = aValue.listProperties(
-						RDFUnmarshallerJena.PROPERTY_HASMAPPING);
-				siMapping.hasNext();) {
-			final Resource aMapping = (Resource) siMapping.nextStatement().getObject();
+		for (final StmtIterator siMapping = aValue
+				.listProperties(RDFUnmarshallerJena.PROPERTY_HASMAPPING); siMapping
+				.hasNext();) {
+			final Resource aMapping = (Resource) siMapping.nextStatement()
+					.getObject();
 			this.addMapping(oValue, aMapping);
 		} // find and add mapping of the Value
 	}
 
 	/**
 	 * Adds a parameter to the given task.
-	 * @param aTask The task to consider.
-	 * @param aParameter The parameter to add.
+	 * 
+	 * @param aTask
+	 *            The task to consider.
+	 * @param aParameter
+	 *            The parameter to add.
 	 */
-	private void addParameter (final Task aTask, final Resource aParameter) {
-		final String sParameterReference =
-			aParameter.getProperty(RDFUnmarshallerJena.PROPERTY_REFERENCE).getLiteral().getString();
-		final Parameter oParameter = aTask.getMapOfParameter().get(sParameterReference);
+	private void addParameter(final Task aTask, final Resource aParameter) {
+		final String sParameterReference = aParameter.getProperty(
+				RDFUnmarshallerJena.PROPERTY_REFERENCE).getLiteral()
+				.getString();
+		final Parameter oParameter = aTask.getMapOfParameter().get(
+				sParameterReference);
 		if (null == oParameter) {
-			RDFUnmarshallerJena.logger.warn(
-					"Parameter " + sParameterReference +
-					" not found in task " + aTask.getID() + ".");
+			RDFUnmarshallerJena.logger.warn("Parameter " + sParameterReference
+					+ " not found in task " + aTask.getID() + ".");
 			return;
 		}
-		RDFUnmarshallerJena.logger.debug("Parameter : "+sParameterReference+".");
+		RDFUnmarshallerJena.logger.debug("Parameter : " + sParameterReference
+				+ ".");
 		// find and add longName of the Parameter
-		for (
-				StmtIterator siLongName = aParameter.listProperties(
-						RDFUnmarshallerJena.PROPERTY_LONGNAME);
-				siLongName.hasNext();) {
+		for (StmtIterator siLongName = aParameter
+				.listProperties(RDFUnmarshallerJena.PROPERTY_LONGNAME); siLongName
+				.hasNext();) {
 			final Literal lLongName = siLongName.nextStatement().getLiteral();
 			this.addLongName(oParameter, lLongName);
 		} // find and add longName of the Parameter
 		// find and add value of the Parameter
-		for (
-				final StmtIterator siValue = this.aModel.listStatements(
-						aParameter,
-						RDFUnmarshallerJena.PROPERTY_HASVALUE,
-						(RDFNode) null);
-				siValue.hasNext();) {
-			final Resource aValue = (Resource) siValue.nextStatement().getObject();
+		for (final StmtIterator siValue = this.aModel.listStatements(
+				aParameter, RDFUnmarshallerJena.PROPERTY_HASVALUE,
+				(RDFNode) null); siValue.hasNext();) {
+			final Resource aValue = (Resource) siValue.nextStatement()
+					.getObject();
 			if (null == aValue) {
 				RDFUnmarshallerJena.logger.error("Resource value == null.");
 				continue;
@@ -244,46 +292,47 @@
 
 	/**
 	 * Adds a task to this object.
-	 * @param aTask The task to add.
+	 * 
+	 * @param aTask
+	 *            The task to add.
 	 * @throws Exception
 	 */
-	private void addTask (final Resource aTask) throws Exception {
-		final Statement aReference = aTask.getProperty(RDFUnmarshallerJena.PROPERTY_REFERENCE);
-		final Task oTask = this.mapOfTask.get(aReference.getLiteral().getString());
+	private void addTask(final Resource aTask) throws Exception {
+		final Statement aReference = aTask
+				.getProperty(RDFUnmarshallerJena.PROPERTY_REFERENCE);
+		final Task oTask = this.mapOfTask.get(aReference.getLiteral()
+				.getString());
 		if (oTask == null) {
 			// TODO creer et initialiser une Task si les informations peuvent
 			// TODO etre mise uniquement dans le fichier rdf
-			RDFUnmarshallerJena.logger.error(
-					"No task with reference : " +
-					aReference.getLiteral().getString() +
-					".");
-			throw new Exception("No task with reference : "+aReference.getLiteral().getString()+".");
+			RDFUnmarshallerJena.logger.error("No task with reference : "
+					+ aReference.getLiteral().getString() + ".");
+			throw new Exception("No task with reference : "
+					+ aReference.getLiteral().getString() + ".");
 		}
-		RDFUnmarshallerJena.logger.debug("Reference : "+aReference.getObject().toString()+".");
+		RDFUnmarshallerJena.logger.debug("Reference : "
+				+ aReference.getObject().toString() + ".");
 		// find and add longName of the task
-		for (
-				final StmtIterator siLongName = aTask.listProperties(
-						RDFUnmarshallerJena.PROPERTY_LONGNAME);
-				siLongName.hasNext();) {
+		for (final StmtIterator siLongName = aTask
+				.listProperties(RDFUnmarshallerJena.PROPERTY_LONGNAME); siLongName
+				.hasNext();) {
 			final Literal lLongName = siLongName.nextStatement().getLiteral();
 			this.addLongName(oTask, lLongName);
 		} // find and add longName of the task
 		// find and add description of the task
-		for (
-				final StmtIterator siDescription = aTask.listProperties(
-						RDFUnmarshallerJena.PROPERTY_DESCRIPTION);
-				siDescription.hasNext();) {
-			final Literal lDescription = siDescription.nextStatement().getLiteral();
+		for (final StmtIterator siDescription = aTask
+				.listProperties(RDFUnmarshallerJena.PROPERTY_DESCRIPTION); siDescription
+				.hasNext();) {
+			final Literal lDescription = siDescription.nextStatement()
+					.getLiteral();
 			this.addDescription(oTask, lDescription);
 		} // find and add description of the task
 		// find and add Parameter of the task
-		for (
-				final StmtIterator siParameter = this.aModel.listStatements(
-						aTask,
-						RDFUnmarshallerJena.PROPERTY_HASPARAMETER,
-						(RDFNode) null);
-				siParameter.hasNext();) {
-			final Resource aParameter = (Resource) siParameter.nextStatement().getObject();
+		for (final StmtIterator siParameter = this.aModel.listStatements(aTask,
+				RDFUnmarshallerJena.PROPERTY_HASPARAMETER, (RDFNode) null); siParameter
+				.hasNext();) {
+			final Resource aParameter = (Resource) siParameter.nextStatement()
+					.getObject();
 			if (null == aParameter) {
 				RDFUnmarshallerJena.logger.error("Resource parameter == null.");
 				continue;
@@ -292,19 +341,18 @@
 		} // find and add Parameter of the task
 	}
 
-	/* (non-Javadoc)
+	/*
+	 * (non-Javadoc)
+	 * 
 	 * @see org.w3c.unicorn.util.Unmarshaller#unmarshal(java.net.URL)
 	 */
-	public void unmarshal () throws Exception {
+	public void unmarshal() throws Exception {
 		RDFUnmarshallerJena.logger.trace("unmarshal");
 
 		// find and add task
-		for (
-				final StmtIterator siTask = this.aModel.listStatements(
-						null,
-						RDFUnmarshallerJena.PROPERTY_TYPE,
-						RDFUnmarshallerJena.RESOURCE_TASK);
-				siTask.hasNext();) {
+		for (final StmtIterator siTask = this.aModel.listStatements(null,
+				RDFUnmarshallerJena.PROPERTY_TYPE,
+				RDFUnmarshallerJena.RESOURCE_TASK); siTask.hasNext();) {
 			final Resource aTask = siTask.nextStatement().getSubject();
 			this.addTask(aTask);
 		} // find and add task
@@ -313,29 +361,32 @@
 
 	/**
 	 * @return Returns the observers.
-	 *//*
-	public Map<String, Observer> getMapOfObserver () {
-		return this.mapOfObserver;
-	}*/
+	 */
+	/*
+	 * public Map<String, Observer> getMapOfObserver () { return
+	 * this.mapOfObserver; }
+	 */
 
 	/**
-	 * @param mapOfTask The observers to set.
+	 * @param mapOfTask
+	 *            The observers to set.
 	 */
-	public void setMapOfObserver (final Map<String, Observer> mapOfObserver) {
+	public void setMapOfObserver(final Map<String, Observer> mapOfObserver) {
 		this.mapOfObserver = mapOfObserver;
 	}
 
 	/**
 	 * @return Returns the tasks.
 	 */
-	public Map<String, Task> getMapOfTask () {
+	public Map<String, Task> getMapOfTask() {
 		return this.mapOfTask;
 	}
 
 	/**
-	 * @param mapOfTask The tasks to set.
+	 * @param mapOfTask
+	 *            The tasks to set.
 	 */
-	public void setMapOfTask (final Map<String, Task> mapOfTask) {
+	public void setMapOfTask(final Map<String, Task> mapOfTask) {
 		this.mapOfTask = mapOfTask;
 	}
 

Index: RDFUnmarshaller.java
===================================================================
RCS file: /sources/public/2006/unicorn/org/w3c/unicorn/tasklist/RDFUnmarshaller.java,v
retrieving revision 1.1.1.1
retrieving revision 1.2
diff -u -d -r1.1.1.1 -r1.2
--- RDFUnmarshaller.java	31 Aug 2006 09:09:26 -0000	1.1.1.1
+++ RDFUnmarshaller.java	11 Aug 2009 13:43:00 -0000	1.2
@@ -11,13 +11,15 @@
 
 /**
  * @author Damien LEROY
- *
+ * 
  */
 public interface RDFUnmarshaller extends Unmarshaller {
 
 	public abstract Map<String, Task> getMapOfTask();
 
-	public abstract void setMapOfObserver (final Map<String, Observer> mapOfObserver);
-	public abstract void setMapOfTask (final Map<String, Task> mapOfTask);
+	public abstract void setMapOfObserver(
+			final Map<String, Observer> mapOfObserver);
+
+	public abstract void setMapOfTask(final Map<String, Task> mapOfTask);
 
 }

Index: TaskListUnmarshallerBeans.java
===================================================================
RCS file: /sources/public/2006/unicorn/org/w3c/unicorn/tasklist/TaskListUnmarshallerBeans.java,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -d -r1.12 -r1.13
--- TaskListUnmarshallerBeans.java	12 Sep 2008 18:01:51 -0000	1.12
+++ TaskListUnmarshallerBeans.java	11 Aug 2009 13:43:00 -0000	1.13
@@ -1,4 +1,4 @@
- package org.w3c.unicorn.tasklist;
+package org.w3c.unicorn.tasklist;
 
 import java.io.IOException;
 import java.net.URL;
@@ -21,7 +21,6 @@
 import org.w3.unicorn.tasklist.TUi;
 import org.w3.unicorn.tasklist.TaskType;
 import org.w3.unicorn.tasklist.TasklistDocument;
-import org.w3.unicorn.tasklist.TasklistType;
 import org.w3.unicorn.tasklist.ThenType;
 import org.w3.unicorn.tasklist.ValueType;
 import org.w3c.unicorn.Framework;
@@ -43,31 +42,31 @@
 
 /**
  * Unmarshals the tasklist thanks to the XMLBeans tools.
- *
+ * 
  * @author Florent Batard, Jonathan Barouh
- *
+ * 
  */
 public class TaskListUnmarshallerBeans implements TasksListUnmarshaller {
 
 	private TasklistDocument aTaskList;
 
-	private static final Log logger = LogFactory.getLog("org.w3c.unicorn.tasklist");
-
-	private int NodeID=0;
+	private static final Log logger = LogFactory
+			.getLog("org.w3c.unicorn.tasklist");
 
+	private int NodeID = 0;
 
 	/**
 	 * The tasklist corresponding to the xml file
 	 */
-	private Map<String,Task> mapOfTask;
+	private Map<String, Task> mapOfTask;
 
 	/**
 	 * The observers' list used to check some constraints on the tasks
 	 */
 	private Map<String, Observer> mapOfObserver;
 
-
-	public TaskListUnmarshallerBeans(){}
+	public TaskListUnmarshallerBeans() {
+	}
 
 	public TaskListUnmarshallerBeans(final Map<String, Observer> mapOfObserver) {
 		TaskListUnmarshallerBeans.logger.trace("Constructor");
@@ -75,8 +74,6 @@
 		this.mapOfObserver = mapOfObserver;
 	}
 
-
-
 	private void addTask(final TaskType aTask) throws ParameterException {
 		TaskListUnmarshallerBeans.logger.trace("addTask");
 
@@ -86,15 +83,16 @@
 		}
 
 		if (TaskListUnmarshallerBeans.logger.isDebugEnabled()) {
-			TaskListUnmarshallerBeans.logger.trace("Add task : " + aTask.getId());
+			TaskListUnmarshallerBeans.logger.trace("Add task : "
+					+ aTask.getId());
 		}
 
 		final Task aTaskCurrent = new Task();
-//		Create the execution level tree
+		// Create the execution level tree
 		aTaskCurrent.setID(aTask.getId());
 		aTaskCurrent.setTree(this.ExpandTree(aTask));
 
-//		 parameters
+		// parameters
 		final ParametersType aParameters = aTask.getParameters();
 		if (aParameters != null) {
 			final ParameterType[] listOfParameter;
@@ -114,19 +112,14 @@
 				Parameter aParameter = null;
 
 				if (sObserver != null && !"".equals(sObserver)) {
-					aParameter = getParameterFromObserver(
-							sName,
-							sObserver,
-							aTUi,
-							sDefaultValues,
-							aParamType);
-				}
-				else {
+					aParameter = getParameterFromObserver(sName, sObserver,
+							aTUi, sDefaultValues, aParamType);
+				} else {
 
 					// Values
 					final Map<String, Value> mapOfValue = new LinkedHashMap<String, Value>();
-					for (final ValueType aValue :
-						aParameterBeans.getValueArray()) {
+					for (final ValueType aValue : aParameterBeans
+							.getValueArray()) {
 
 						// name of the value
 						String sValueName = aValue.getName();
@@ -136,11 +129,15 @@
 
 						// Mappings of the value
 						final Map<String, List<Mapping>> mapOfMapping = new LinkedHashMap<String, List<Mapping>>();
-						for (final MappedType aMappedBeans : aValue.getMappedArray()) {
-							final Mapping aMapping = this.createMapping(aMappedBeans);
+						for (final MappedType aMappedBeans : aValue
+								.getMappedArray()) {
+							final Mapping aMapping = this
+									.createMapping(aMappedBeans);
 							if (aMapping != null) {
-								final String sObs = aMapping.getObserver().getID();
-								List<Mapping> listOfMapping = mapOfMapping.get(sObs);
+								final String sObs = aMapping.getObserver()
+										.getID();
+								List<Mapping> listOfMapping = mapOfMapping
+										.get(sObs);
 								if (null == listOfMapping) {
 									listOfMapping = new ArrayList<Mapping>();
 									mapOfMapping.put(sObs, listOfMapping);
@@ -148,19 +145,12 @@
 								listOfMapping.add(aMapping);
 							}
 						}
-						mapOfValue.put(
-								sValueName,
-								new Value(
-										mapOfMapping,
-										sValueName));
+						mapOfValue.put(sValueName, new Value(mapOfMapping,
+								sValueName));
 					}
 
-					aParameter = this.createParameter(
-							aParamType,
-							sName,
-							aTUi,
-							sDefaultValues,
-							mapOfValue);
+					aParameter = this.createParameter(aParamType, sName, aTUi,
+							sDefaultValues, mapOfValue);
 				}
 
 				if (aParameter != null) {
@@ -169,27 +159,29 @@
 			}
 		}
 
-		this.mapOfTask.put(aTaskCurrent.getID(),aTaskCurrent);
+		this.mapOfTask.put(aTaskCurrent.getID(), aTaskCurrent);
 
 	}
 
-
 	/**
 	 * Creates a usable mapping from a JAXB-generated one.
-	 * @param aMappedJAXB the JAXB-generated mapping
+	 * 
+	 * @param aMappedJAXB
+	 *            the JAXB-generated mapping
 	 * @return the created mapping
 	 */
-	private Mapping createMapping (final MappedType aMapped) {
+	private Mapping createMapping(final MappedType aMapped) {
 		TaskListUnmarshallerBeans.logger.trace("createMapping");
 
 		// The mapped observer
 		final String sMappingObserver = aMapped.getObserver();
-		final Observer aObserverMapped = this.mapOfObserver.get(sMappingObserver);
+		final Observer aObserverMapped = this.mapOfObserver
+				.get(sMappingObserver);
 
 		if (aObserverMapped == null) {
-			TaskListUnmarshallerBeans.logger.error(
-					"The observer " + sMappingObserver +
-					" does not seem to exist... Skipping mapping.");
+			TaskListUnmarshallerBeans.logger.error("The observer "
+					+ sMappingObserver
+					+ " does not seem to exist... Skipping mapping.");
 			return null;
 		}
 
@@ -206,11 +198,10 @@
 
 		// The list of mapped input methods
 
-		final List<TInputMethod.Enum> listOfTInputMethodBeans =
-			new ArrayList<TInputMethod.Enum>();
+		final List<TInputMethod.Enum> listOfTInputMethodBeans = new ArrayList<TInputMethod.Enum>();
 		for (Object methodString : aMapped.getInputmethod()) {
-			listOfTInputMethodBeans.add(
-					TInputMethod.Enum.forString(methodString.toString()));
+			listOfTInputMethodBeans.add(TInputMethod.Enum
+					.forString(methodString.toString()));
 		}
 		// by default a parameter is mapped to all input methods
 		if (listOfTInputMethodBeans.size() == 0) {
@@ -220,38 +211,37 @@
 		}
 
 		/*
-		 * For each input method, we check that the mapped observer:
-		 *  - can handle this input method
-		 *  - has a parameter with the corresponding name for this input
-		 *    method
-		 *  - can handle this value for this parameter
+		 * For each input method, we check that the mapped observer: - can
+		 * handle this input method - has a parameter with the corresponding
+		 * name for this input method - can handle this value for this parameter
 		 */
 
 		for (final TInputMethod.Enum aTInputMethod : listOfTInputMethodBeans) {
 			final EnumInputMethod aEnumInputMethod;
-			aEnumInputMethod = TaskListUnmarshallerBeans.getEnumInputMethod(aTInputMethod);
+			aEnumInputMethod = TaskListUnmarshallerBeans
+					.getEnumInputMethod(aTInputMethod);
 			// the observer can handle this input method
 			if (aObserverMapped.getInputMethod(aEnumInputMethod) == null) {
-				TaskListUnmarshallerBeans.logger.warn(
-						sMappingObserver + " does not support " +
-						aEnumInputMethod.value() + " input method.");
+				TaskListUnmarshallerBeans.logger.warn(sMappingObserver
+						+ " does not support " + aEnumInputMethod.value()
+						+ " input method.");
 				continue;
 			}
 			final CallParameter aCallParameterMapped;
-			aCallParameterMapped = aObserverMapped.getInputMethod(aEnumInputMethod).getCallParameterByName(sMappingParam);
+			aCallParameterMapped = aObserverMapped.getInputMethod(
+					aEnumInputMethod).getCallParameterByName(sMappingParam);
 			// the parameter exists
 			if (aCallParameterMapped == null) {
-				TaskListUnmarshallerBeans.logger.error(
-						sMappingObserver + " does not have " +
-						"a parameter named " + sMappingParam + ".");
+				TaskListUnmarshallerBeans.logger.error(sMappingObserver
+						+ " does not have " + "a parameter named "
+						+ sMappingParam + ".");
 				continue;
 			}
 			// the value exists
 			if (!aCallParameterMapped.contains(sMappingValue)) {
-				TaskListUnmarshallerBeans.logger.error(
-						"Parameter " + sMappingParam +
-						" does not accept " + sMappingValue +
-						" as a value.");
+				TaskListUnmarshallerBeans.logger.error("Parameter "
+						+ sMappingParam + " does not accept " + sMappingValue
+						+ " as a value.");
 				continue;
 			}
 			listOfEnumInputMethod.add(aEnumInputMethod);
@@ -261,36 +251,36 @@
 			return null;
 		}
 
-		return new Mapping(
-				aObserverMapped,
-				sMappingParam,
-				sMappingValue/*,
-				listOfEnumInputMethod*/);
+		return new Mapping(aObserverMapped, sMappingParam, sMappingValue/*
+																		 * ,
+																		 * listOfEnumInputMethod
+																		 */);
 	}
 
-
-	private Parameter getParameterFromObserver (
-			final String sParamName,
-			final String sObserverName,
-			final TUi.Enum aTUi,
-			final String sDefaultValues,
-			final TParamType.Enum aTParamType) throws ParameterException {
+	private Parameter getParameterFromObserver(final String sParamName,
+			final String sObserverName, final TUi.Enum aTUi,
+			final String sDefaultValues, final TParamType.Enum aTParamType)
+			throws ParameterException {
 		TaskListUnmarshallerBeans.logger.trace("getParameterFromObserver");
 		if (TaskListUnmarshallerBeans.logger.isDebugEnabled()) {
-			TaskListUnmarshallerBeans.logger.debug("Parameter name : "+sParamName+".");
-			TaskListUnmarshallerBeans.logger.debug("Observer name : "+sObserverName+".");
-			TaskListUnmarshallerBeans.logger.debug("TUi : "+aTUi+".");
-			TaskListUnmarshallerBeans.logger.debug("Default values : "+sDefaultValues+".");
-			TaskListUnmarshallerBeans.logger.debug("TParamType : "+aTParamType+".");
+			TaskListUnmarshallerBeans.logger.debug("Parameter name : "
+					+ sParamName + ".");
+			TaskListUnmarshallerBeans.logger.debug("Observer name : "
+					+ sObserverName + ".");
+			TaskListUnmarshallerBeans.logger.debug("TUi : " + aTUi + ".");
+			TaskListUnmarshallerBeans.logger.debug("Default values : "
+					+ sDefaultValues + ".");
+			TaskListUnmarshallerBeans.logger.debug("TParamType : "
+					+ aTParamType + ".");
 		}
 
 		final Observer aObserver = this.mapOfObserver.get(sObserverName);
 
 		// does the requested observer exist?
 		if (aObserver == null) {
-			TaskListUnmarshallerBeans.logger.warn(
-					"The parameter " + sParamName + "refers to a " +
-					"non-existing observer: " + sObserverName + ".");
+			TaskListUnmarshallerBeans.logger.warn("The parameter " + sParamName
+					+ "refers to a " + "non-existing observer: "
+					+ sObserverName + ".");
 			return null;
 		}
 
@@ -302,23 +292,27 @@
 		final Map<String, Value> mapOfValue = new LinkedHashMap<String, Value>();
 
 		// iterate over the observer's methods
-		final Map<EnumInputMethod, InputMethod> mapOfInputMethod = aObserver.getMapOfInputMethod();
+		final Map<EnumInputMethod, InputMethod> mapOfInputMethod = aObserver
+				.getMapOfInputMethod();
 		for (final EnumInputMethod aEnumInputMethod : mapOfInputMethod.keySet()) {
-			final InputMethod aInputMethod = mapOfInputMethod.get(aEnumInputMethod);
+			final InputMethod aInputMethod = mapOfInputMethod
+					.get(aEnumInputMethod);
 
 			if (aInputMethod.getCallParameter().getName().equals(sParamName)) {
 				// the referenced parameter is an input one for the current
 				// method, so it must be ignored
-				TaskListUnmarshallerBeans.logger.info(
-						"The referenced parameter is an input one for the current method, so it must be ignored.");
+				TaskListUnmarshallerBeans.logger
+						.info("The referenced parameter is an input one for the current method, so it must be ignored.");
 				continue;
 			}
 
 			final CallMethod aCallMethod = aInputMethod.getCallMethod();
-			final CallParameter aCallParameter = aCallMethod.getCallParameterByName(sParamName);
+			final CallParameter aCallParameter = aCallMethod
+					.getCallParameterByName(sParamName);
 			// A parameter with this name exists for this method
 			if (aCallParameter != null) {
-				for (final String sValue : aCallParameter.getListOfPossibleValue()) {
+				for (final String sValue : aCallParameter
+						.getListOfPossibleValue()) {
 					final Value aValueCurrent = mapOfValue.get(sValue);
 					if (aValueCurrent != null) {
 						// the newly created parameter already contains a
@@ -326,22 +320,20 @@
 
 						// we know that the created parameter contains exactly
 						// ONE mapping for this value
-						//aValueCurrent.getMappings().get(sObserverName).get(0).addInputMethod(aEnumInputMethod);
-					}
-					else {
+						// aValueCurrent.getMappings().get(sObserverName).get(0).addInputMethod(aEnumInputMethod);
+					} else {
 						// the newly created parameter does not contain a
 						// similar value
 						final LocalizedString aInternationalizedMessageValueName;
 						aInternationalizedMessageValueName = new LocalizedString();
 						// TODO Add localized names in RDF contract
-						aInternationalizedMessageValueName.addLocalization("en", sValue);
+						aInternationalizedMessageValueName.addLocalization(
+								"en", sValue);
 
 						final List<EnumInputMethod> mapOfNewInputMethod = new ArrayList<EnumInputMethod>();
 						mapOfNewInputMethod.add(aEnumInputMethod);
-						final Mapping aMapping = new Mapping(
-								aObserver,
-								sParamName,
-								sValue);
+						final Mapping aMapping = new Mapping(aObserver,
+								sParamName, sValue);
 
 						final List<Mapping> listOfMapping = new ArrayList<Mapping>();
 						listOfMapping.add(aMapping);
@@ -352,74 +344,69 @@
 
 						final Value aValueToAdd = new Value(
 								aInternationalizedMessageValueName,
-								mapOfListOfMapping,
-								sValue);
+								mapOfListOfMapping, sValue);
 						mapOfValue.put(sValue, aValueToAdd);
 					}
 				}
 			}
 		}
-		return this.createParameter(
-				aTParamType,
-				sParamName,
-				aTUi,
-				sDefaultValues,
-				mapOfValue);
+		return this.createParameter(aTParamType, sParamName, aTUi,
+				sDefaultValues, mapOfValue);
 	}
 
-
 	/**
 	 * Create the tree from a TaskType element
-	 *
+	 * 
 	 * @param myTask
 	 *            the task to transform into a tree
 	 */
 	public TLTNode ExpandTree(TaskType myTask) {
-		TaskListUnmarshallerBeans.logger.trace("Creation of the tree based on the Task "
-				+ myTask.getId());
+		TaskListUnmarshallerBeans.logger
+				.trace("Creation of the tree based on the Task "
+						+ myTask.getId());
 		TLTNode root = new TLTNode();
 		root.setID(NodeID++);
 		for (ExecType exec : myTask.getRoutine().getExecArray()) {
-			final Observer obs=Framework.mapOfObserver.get(exec.getValue());
-			root.addExec(new TLTExec(exec.getId(), obs,exec.getValue(), exec
+			final Observer obs = Framework.mapOfObserver.get(exec.getValue());
+			root.addExec(new TLTExec(exec.getId(), obs, exec.getValue(), exec
 					.getType(), exec.getParam()));
 		}
 		for (IfType iflist : myTask.getRoutine().getIfArray()) {
-			root.addIf(fillIfs(myTask,iflist));
+			root.addIf(fillIfs(myTask, iflist));
 		}
 		return root;
 	}
 
 	/**
 	 * Recursion over the Then tag to fill the tree
-	 *
+	 * 
 	 * @param myThen
 	 *            the ThenType node for the recursion
 	 * @return the node created
 	 */
-	private TLTNode FillNode(TaskType myTask,ThenType myThen) {
+	private TLTNode FillNode(TaskType myTask, ThenType myThen) {
 		TaskListUnmarshallerBeans.logger.trace("Creation of a then branch ");
 		TLTNode node = new TLTNode();
 		node.setID(NodeID++);
 		for (ExecType exec : myThen.getExecArray()) {
-			final Observer obs=Framework.mapOfObserver.get(exec.getValue());
-			node.addExec(new TLTExec(exec.getId(),obs, exec.getValue(), exec
+			final Observer obs = Framework.mapOfObserver.get(exec.getValue());
+			node.addExec(new TLTExec(exec.getId(), obs, exec.getValue(), exec
 					.getType(), exec.getParam()));
 		}
 		for (IfType iflist : myThen.getIfArray()) {
-			node.addIf(fillIfs(myTask,iflist));
+			node.addIf(fillIfs(myTask, iflist));
 		}
 		return node;
 	}
 
 	/**
 	 * Created the Ifnode in the tree
-	 *
+	 * 
 	 * @param ifs
 	 *            the IfType node template to create the Ifnode
 	 * @return the node created
 	 */
-	private TLTIf fillIfs(TaskType myTask,IfType ifs) {
+	private TLTIf fillIfs(TaskType myTask, IfType ifs) {
 		TaskListUnmarshallerBeans.logger.trace("Creation of an If ");
 		// Create the if node
 		TLTIf ifnode = new TLTIf();
@@ -431,9 +418,11 @@
 
 			for (CondType condlist : myTask.getConds().getCondArray()) {
 				if (condlist.getId().equals(cond)) {
-					TaskListUnmarshallerBeans.logger.trace("Creation of a condition " + cond);
+					TaskListUnmarshallerBeans.logger
+							.trace("Creation of a condition " + cond);
 					myCond.setId(condlist.getId());
-					final Observer obs=Framework.mapOfObserver.get(condlist.getObserver());
+					final Observer obs = Framework.mapOfObserver.get(condlist
+							.getObserver());
 					myCond.setObserver(obs);
 					myCond
 							.setResult(condlist.getResult().equals("passed") ? true
@@ -450,36 +439,37 @@
 
 		// Add recursively the inner ifs in the then part
 		if (ifs.getThen() != null) {
-			TaskListUnmarshallerBeans.logger.trace("Call recursion for the Then ");
-			ifnode.setIfOk(FillNode(myTask,ifs.getThen()));
+			TaskListUnmarshallerBeans.logger
+					.trace("Call recursion for the Then ");
+			ifnode.setIfOk(FillNode(myTask, ifs.getThen()));
 			// Add recursively the inner if in the else part
 			if (ifs.getElse() != null) {
-				TaskListUnmarshallerBeans.logger.trace("Call recursion for the else");
-				ifnode.setIfNotOk(FillNode(myTask,ifs.getElse()));
+				TaskListUnmarshallerBeans.logger
+						.trace("Call recursion for the else");
+				ifnode.setIfNotOk(FillNode(myTask, ifs.getElse()));
 			}
 		}
 		return ifnode;
 	}
 
-
-
-
-	private org.w3c.unicorn.tasklist.parameters.Parameter createParameter (
-			final TParamType.Enum aTParamType,
-			final String sName,
-			final TUi.Enum aTUi,
-			final String sDefaultValues,
+	private org.w3c.unicorn.tasklist.parameters.Parameter createParameter(
+			final TParamType.Enum aTParamType, final String sName,
+			final TUi.Enum aTUi, final String sDefaultValues,
 			final Map<String, Value> mapOfValue) throws ParameterException {
 		TaskListUnmarshallerBeans.logger.trace("createParameter");
 		if (TaskListUnmarshallerBeans.logger.isDebugEnabled()) {
-			TaskListUnmarshallerBeans.logger.debug("TParamType : "+aTParamType+".");
-			TaskListUnmarshallerBeans.logger.debug("Name : "+sName+".");
-			TaskListUnmarshallerBeans.logger.debug("TUi : "+aTUi+".");
-			TaskListUnmarshallerBeans.logger.debug("Default values : "+sDefaultValues+".");
-			TaskListUnmarshallerBeans.logger.debug("Map of value : "+mapOfValue+".");
+			TaskListUnmarshallerBeans.logger.debug("TParamType : "
+					+ aTParamType + ".");
+			TaskListUnmarshallerBeans.logger.debug("Name : " + sName + ".");
+			TaskListUnmarshallerBeans.logger.debug("TUi : " + aTUi + ".");
+			TaskListUnmarshallerBeans.logger.debug("Default values : "
+					+ sDefaultValues + ".");
+			TaskListUnmarshallerBeans.logger.debug("Map of value : "
+					+ mapOfValue + ".");
 		}
 
-		final org.w3c.unicorn.tasklist.parameters.Parameter aParameter = ParameterFactory.getParameter(aTParamType);
+		final org.w3c.unicorn.tasklist.parameters.Parameter aParameter = ParameterFactory
+				.getParameter(aTParamType);
 		if (null == aParameter) {
 			return null;
 		}
@@ -490,55 +480,53 @@
 		return aParameter;
 	}
 
-
 	/**
 	 * Wraps a TInputMethod instance on an EnumInputMethod
+	 * 
 	 * @param aTInputMethod
 	 * @return
 	 */
-	private static EnumInputMethod getEnumInputMethod (final TInputMethod.Enum aTInputMethod) {
+	private static EnumInputMethod getEnumInputMethod(
+			final TInputMethod.Enum aTInputMethod) {
 		switch (aTInputMethod.intValue()) {
-			case TInputMethod.INT_DIRECT:
-				return EnumInputMethod.DIRECT;
-			case TInputMethod.INT_FILE:
-				return EnumInputMethod.UPLOAD;
-			case TInputMethod.INT_URI:
-				return EnumInputMethod.URI;
-			default:
-				return EnumInputMethod.URI;
+		case TInputMethod.INT_DIRECT:
+			return EnumInputMethod.DIRECT;
+		case TInputMethod.INT_FILE:
+			return EnumInputMethod.UPLOAD;
+		case TInputMethod.INT_URI:
+			return EnumInputMethod.URI;
+		default:
+			return EnumInputMethod.URI;
 		}
 	}
 
-
-
 	public Map<String, org.w3c.unicorn.tasklist.Task> getMapOfTask() {
 		TaskListUnmarshallerBeans.logger.trace("getMapOfTask");
 		return this.mapOfTask;
 	}
 
-	public void addURL(URL aURL) throws IOException{
+	public void addURL(URL aURL) throws IOException {
 		TaskListUnmarshallerBeans.logger.trace("addURL");
 		if (TaskListUnmarshallerBeans.logger.isDebugEnabled()) {
-			TaskListUnmarshallerBeans.logger.debug("URL : "+aURL+".");
+			TaskListUnmarshallerBeans.logger.debug("URL : " + aURL + ".");
 		}
 
-			try {
-        ClassLoader classLoader = Thread.currentThread().getContextClassLoader();
-				this.aTaskList = TasklistDocument.Factory.parse(aURL.openStream());
-			} catch (XmlException e) {
-				TaskListUnmarshallerBeans.logger.error("Parsing error in TasklistUnmarshaller",e);
-				e.printStackTrace();
-			}
+		try {
+			this.aTaskList = TasklistDocument.Factory.parse(aURL.openStream());
+		} catch (XmlException e) {
+			TaskListUnmarshallerBeans.logger.error(
+					"Parsing error in TasklistUnmarshaller", e);
+			e.printStackTrace();
+		}
 	}
 
-
-
 	public void unmarshal() throws Exception {
 		TaskListUnmarshallerBeans.logger.trace("unmarshal tasklist");
 		// creates the tasklist without computing references
 		for (final TaskType aTask : this.aTaskList.getTasklist().getTaskArray()) {
 			if (this.mapOfTask.containsKey(aTask.getId())) {
-				TaskListUnmarshallerBeans.logger.warn("Task with id "+aTask.getId()+" already defined.");
+				TaskListUnmarshallerBeans.logger.warn("Task with id "
+						+ aTask.getId() + " already defined.");
 			} else {
 				this.addTask(aTask);
 			}
@@ -546,8 +534,10 @@
 
 		// computes and replaces references by their corresponding observations
 		// and parameters
-		for (final org.w3c.unicorn.tasklist.Task aTask : this.mapOfTask.values()) {
-			TaskListUnmarshallerBeans.logger.debug("Expand task : "+aTask.getID()+".");
+		for (final org.w3c.unicorn.tasklist.Task aTask : this.mapOfTask
+				.values()) {
+			TaskListUnmarshallerBeans.logger.debug("Expand task : "
+					+ aTask.getID() + ".");
 			aTask.setTree(aTask.expandNode(mapOfTask, aTask.getTree()));
 		}
 

Index: TasksListUnmarshaller.java
===================================================================
RCS file: /sources/public/2006/unicorn/org/w3c/unicorn/tasklist/TasksListUnmarshaller.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -d -r1.3 -r1.4
--- TasksListUnmarshaller.java	26 Aug 2008 15:26:37 -0000	1.3
+++ TasksListUnmarshaller.java	11 Aug 2009 13:43:00 -0000	1.4
@@ -12,14 +12,16 @@
  * TasksListUnmarshaller<br />
  * Created: May 30, 2006 5:54:45 PM<br />
  * Interface used to retreive a list of tasks
+ * 
  * @author Jean-Guilhem ROUEL
  */
 public interface TasksListUnmarshaller extends Unmarshaller {
 
-    /**
-     * Returns the map of tasks.
-     * @return Returns the tasks.
-     */
-    public abstract Map<String, org.w3c.unicorn.tasklist.Task> getMapOfTask();
-    
+	/**
+	 * Returns the map of tasks.
+	 * 
+	 * @return Returns the tasks.
+	 */
+	public abstract Map<String, org.w3c.unicorn.tasklist.Task> getMapOfTask();
+
 }

Index: Task.java
===================================================================
RCS file: /sources/public/2006/unicorn/org/w3c/unicorn/tasklist/Task.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -d -r1.4 -r1.5
--- Task.java	19 Sep 2008 18:57:12 -0000	1.4
+++ Task.java	11 Aug 2009 13:43:00 -0000	1.5
@@ -8,17 +8,16 @@
 import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
+
 import javax.activation.MimeType;
 
-import org.apache.commons.logging.Log;
-import org.apache.commons.logging.LogFactory;
+import org.w3c.unicorn.contract.Observer;
 import org.w3c.unicorn.tasklist.parameters.Parameter;
 import org.w3c.unicorn.tasklisttree.TLTCond;
 import org.w3c.unicorn.tasklisttree.TLTExec;
 import org.w3c.unicorn.tasklisttree.TLTIf;
 import org.w3c.unicorn.tasklisttree.TLTNode;
 import org.w3c.unicorn.util.LocalizedString;
-import org.w3c.unicorn.contract.Observer;
 
 /**
  * Task<br />
@@ -26,9 +25,6 @@
  */
 public class Task {
 
-	private static final Log logger = LogFactory
-			.getLog("org.w3c.unicorn.tasklist");
-
 	/**
 	 * Id of the task
 	 */
@@ -44,7 +40,6 @@
 	 */
 	private LocalizedString aLocalizedStringDescription;
 
-	
 	/**
 	 * Parameters of the task
 	 */
@@ -56,16 +51,10 @@
 	private List<String> listOfReference;
 
 	/**
-	 * Used to expand the task
-	 */
-	private boolean bExpandingOrExpanded = false;
-
-	/**
 	 * Root of the execution level tree
 	 */
 	private TLTNode root;
 
-
 	/**
 	 * Creates a new Task.
 	 */
@@ -76,7 +65,7 @@
 		this.mapOfTaskParameter = new LinkedHashMap<String, Parameter>();
 		this.listOfReference = new ArrayList<String>();
 	}
-	
+
 	/**
 	 * Allows to display the tree of execution level
 	 * 
@@ -89,29 +78,28 @@
 		}
 		for (TLTIf ifs : root.getIfList()) {
 			displayTree(ifs.getIfOk());
-			for (TLTCond conds : ifs.getCondArray())
+			for (TLTCond conds : ifs.getCondArray()) {
 				System.out.println(conds);
+			}
 			displayTree(ifs.getIfNotOk());
 		}
 	}
 
 	/**
 	 * Get the root of the execution level tree
+	 * 
 	 * @return the root of the tree
 	 */
 	public TLTNode getTree() {
 		return this.root;
 	}
-	
-	
+
 	/**
 	 * Set the root of the execution level tree
 	 */
 	public void setTree(TLTNode root) {
-		this.root=root;
+		this.root = root;
 	}
-	
-
 
 	/**
 	 * Creates a new Task.
@@ -275,7 +263,6 @@
 		this.mapOfTaskParameter.put(aParameter.getName(), aParameter);
 	}
 
-
 	/**
 	 * Returns a list of tasknames referenced bye this task
 	 * 
@@ -285,221 +272,104 @@
 		return this.listOfReference;
 	}
 
-    public List<Observer> getAllObservers() {
-        if(this.getTree() != null) {
-            return this.getTree().getAllObservers();
-        }
-        return new ArrayList<Observer>();
-    }
-
-    // MimeType's equals() doesn't work as expected
-    // so it's easier to store the String representation
-    // of mime types :-/
-    public List<String> getSupportedMimeTypes() {
-        List<String> res = new ArrayList<String>();
-        List<Observer> observers = getAllObservers();
-        for(Observer o : observers) {
-            List<MimeType> mimes = o.getSupportedMimeTypes();
-            for(MimeType m : mimes) {
-                if(!res.contains(m.toString())) {
-                    res.add(m.toString());
-                }
-            }
-        }
-        return res;
-    }
-
-	/**
-	 * Recursively expands this task and referenced ones and merges observations
-	 * and parameters.<br/> If a task A includes a task B that includes a task
-	 * C, expand will put both B and C in A's referenced tasks.
-	 * 
-	 * @param mapOfTask
-	 */
-	/*
-	public void expand(final Map<String, Task> mapOfTask) {
-		this.bExpandingOrExpanded = true;
-		final List<String> listOfOldReference = new ArrayList<String>();
-		// re-ask why there build another list of reference
-		// it's because he add reference in this.references
-		for (final String sReference : this.listOfReference) {
-			listOfOldReference.add(sReference);
-		}
-
-		for (final String sReference : listOfOldReference) {
-			final Task aTask = mapOfTask.get(sReference);
-
-			if (aTask == null) {
-				Task.logger.error("The task " + sReference
-						+ " directly referenced " + "by the task"
-						+ this.getID() + " does not seem to"
-						+ " exist... Ignoring reference");
-				continue;
-			}
-
-			this.merge(aTask);
-
-			if (!aTask.bExpandingOrExpanded) {
-				aTask.expand(mapOfTask);
-			}
-
-			for (final String sNewReference : aTask.listOfReference) {
-				if (this.listOfReference.contains(sNewReference)
-						|| this.sID.equals(sNewReference)) {
-					continue;
-				}
-				final Task aTaskCurrentRef = mapOfTask.get(sNewReference);
-				if (aTaskCurrentRef == null) {
-					Task.logger.error("The task " + sReference + " recursively"
-							+ " referenced by the task" + getID()
-							+ " does not seem to exist... Ignoring "
-							+ "reference");
-					continue;
-				}
-				this.listOfReference.add(sNewReference);
-				this.merge(aTaskCurrentRef);
-			}
+	public List<Observer> getAllObservers() {
+		if (this.getTree() != null) {
+			return this.getTree().getAllObservers();
 		}
+		return new ArrayList<Observer>();
 	}
-	*/
 
-	/**
-	 * Merges another task with this one
-	 * 
-	 * @param aNotherTask
-	 *            the task to merge
-	 */
-	/*
-	private void merge(final Task aNotherTask) {
-		this.mergeObservations(aNotherTask);
-		this.mergeParameters(aNotherTask);
-	}
-*/
-	/**
-	 * Merges observations of another task with this one
-	 * 
-	 * @param aNotherTask
-	 *            the task to merge
-	 */
-	/*
-	private void mergeObservations(final Task aNotherTask) {
-		Task.logger.trace("mergeObservations");
-		if (Task.logger.isDebugEnabled()) {
-			Task.logger.debug("Other task : " + aNotherTask + ".");
-		}
-		final Map<String, Observation> mapOfObservation = aNotherTask
-				.getMapOfObservation();
-		for (final String sObservationID : mapOfObservation.keySet()) {
-			final Observation aObservation = mapOfObservation
-					.get(sObservationID);
-			if (this.mapOfObservation.containsKey(sObservationID)) {
-				this.mapOfObservation.get(sObservationID).merge(aObservation);
-			} else {
-				this.mapOfObservation.put(sObservationID, aObservation);
-			}
-		}
-	}
-*/
-	/**
-	 * Merges parameters of another task with this one.
-	 * 
-	 * @param aNotherTask
-	 *            the other task to merge
-	 */
-	private void mergeParameters(final Task aNotherTask) {
-		final Map<String, Parameter> mapOfParameter = aNotherTask
-				.getMapOfParameter();
-		for (final String sParameterName : mapOfParameter.keySet()) {
-			final Parameter aLocalParameter = this.mapOfTaskParameter
-					.get(sParameterName);
-			final Parameter aNotherParameter = mapOfParameter
-					.get(sParameterName);
-			if (aLocalParameter != null) {
-				aLocalParameter.merge(aNotherParameter);
-			} else {
-				this.mapOfTaskParameter.put(sParameterName, aNotherParameter);
+	// MimeType's equals() doesn't work as expected
+	// so it's easier to store the String representation
+	// of mime types :-/
+	public List<String> getSupportedMimeTypes() {
+		List<String> res = new ArrayList<String>();
+		List<Observer> observers = getAllObservers();
+		for (Observer o : observers) {
+			List<MimeType> mimes = o.getSupportedMimeTypes();
+			for (MimeType m : mimes) {
+				if (!res.contains(m.toString())) {
+					res.add(m.toString());
+				}
 			}
 		}
+		return res;
 	}
-	
-	
 
 	public void mergeSubtask(final Map<String, Task> mapOfTask, Task subtask) {
 		for (TLTExec exec : subtask.getTree().getExecutionList()) {
-			if (exec.getType().equals("observation"))
+			if (exec.getType().equals("observation")) {
 				this.root.addExec(exec);
-			else if (exec.getType().equals("subtask")) {
+			} else if (exec.getType().equals("subtask")) {
 				Task newTask = mapOfTask.get(exec.getValue());
-				newTask.expandNode(mapOfTask,newTask.getTree());
-				mergeSubtask(mapOfTask,newTask);
+				newTask.expandNode(mapOfTask, newTask.getTree());
+				mergeSubtask(mapOfTask, newTask);
 			}
-				
+
 		}
 		for (TLTIf tltIf : subtask.getTree().getIfList()) {
 			this.root.addIf(tltIf);
 		}
 	}
-	
+
 	/**
 	 * 
 	 */
 	public TLTNode expandNode(final Map<String, Task> mapOfTask, TLTNode aRoot) {
 		aRoot.bExpandingOrExpanded = true;
-		
+
 		TLTNode finalRoot = new TLTNode();
-		
+
 		for (TLTExec exec : aRoot.getExecutionList()) {
 			if (exec.getType().equals("subtask")) {
-				finalRoot = mergeNode(mapOfTask, finalRoot, 
-						mapOfTask.get(exec.getValue()).getTree());
-			}
-			else if (exec.getType().equals("observation")) {
+				finalRoot = mergeNode(mapOfTask, finalRoot, mapOfTask.get(
+						exec.getValue()).getTree());
+			} else if (exec.getType().equals("observation")) {
 				finalRoot.addExec(exec);
 			}
-		}	
+		}
 
 		for (TLTIf tltIf : aRoot.getIfList()) {
 			tltIf = expandIf(mapOfTask, tltIf);
 			finalRoot.addIf(tltIf);
 		}
-		
+
 		return finalRoot;
 	}
-	
-	
-		public TLTNode mergeNode(final Map<String,Task> mapOfTask, TLTNode firstNode,
-				TLTNode secondNode) {
-			TLTNode finalNode = firstNode;
-			for (TLTExec exec : secondNode.getExecutionList()) {
-				if (exec.getType().equals("observation"))
-					finalNode.addExec(exec);
-				else if (exec.getType().equals("subtask")) {
-					TLTNode newNode = mapOfTask.get(exec.getValue()).getTree();
-					if (!mapOfTask.get(exec.getValue()).getTree().bExpandingOrExpanded) 
-						newNode = 
-							expandNode(mapOfTask,mapOfTask.get(exec.getValue()).getTree());					
-					finalNode = mergeNode(mapOfTask,finalNode,newNode);
+
+	public TLTNode mergeNode(final Map<String, Task> mapOfTask,
+			TLTNode firstNode, TLTNode secondNode) {
+		TLTNode finalNode = firstNode;
+		for (TLTExec exec : secondNode.getExecutionList()) {
+			if (exec.getType().equals("observation")) {
+				finalNode.addExec(exec);
+			} else if (exec.getType().equals("subtask")) {
+				TLTNode newNode = mapOfTask.get(exec.getValue()).getTree();
+				if (!mapOfTask.get(exec.getValue()).getTree().bExpandingOrExpanded) {
+					newNode = expandNode(mapOfTask, mapOfTask.get(
+							exec.getValue()).getTree());
 				}
+				finalNode = mergeNode(mapOfTask, finalNode, newNode);
 			}
-			for (TLTIf tltIf : secondNode.getIfList()) {
-				tltIf = expandIf(mapOfTask,tltIf);
-				finalNode.addIf(tltIf);
-			}	
-			return finalNode;
 		}
-		
-	
-		public TLTIf expandIf(final Map<String,Task> mapOfTask, TLTIf tltIf) {
-			if (!tltIf.getIfOk().bExpandingOrExpanded) {
-				TLTNode tltIfOk = expandNode(mapOfTask,tltIf.getIfOk());
-				tltIf.setIfOk(tltIfOk);
-			}
-			if (!tltIf.getIfNotOk().bExpandingOrExpanded) {
-				TLTNode tltIfNotOk = expandNode(mapOfTask,tltIf.getIfNotOk());	
-				tltIf.setIfNotOk(tltIfNotOk);
-			}
-			return tltIf;
+		for (TLTIf tltIf : secondNode.getIfList()) {
+			tltIf = expandIf(mapOfTask, tltIf);
+			finalNode.addIf(tltIf);
 		}
+		return finalNode;
+	}
+
+	public TLTIf expandIf(final Map<String, Task> mapOfTask, TLTIf tltIf) {
+		if (!tltIf.getIfOk().bExpandingOrExpanded) {
+			TLTNode tltIfOk = expandNode(mapOfTask, tltIf.getIfOk());
+			tltIf.setIfOk(tltIfOk);
+		}
+		if (!tltIf.getIfNotOk().bExpandingOrExpanded) {
+			TLTNode tltIfNotOk = expandNode(mapOfTask, tltIf.getIfNotOk());
+			tltIf.setIfNotOk(tltIfNotOk);
+		}
+		return tltIf;
+	}
 
 	/**
 	 * Adds a reference to another task
@@ -511,6 +381,7 @@
 		this.listOfReference.add(sReference);
 	}
 
+	@Override
 	public String toString() {
 		final int iStringBufferSize = 5000;
 		final String sVariableSeparator = "\n";
@@ -526,5 +397,4 @@
 		return aStringBuffer.toString();
 	}
 
-
 }

Received on Tuesday, 11 August 2009 13:43:46 UTC