Revision: 1823
Author: [email protected]
Date: Sun Jan 10 04:58:09 2010
Log: Add another couple of refactor notes, we need a generic results object and we need to use a pre-exisitng SPARQL endpoint rather than trying to reinvent a rather complex wheel.

See ISSUE 116
http://code.google.com/p/simal/source/detail?r=1823

Modified:
/trunk/uk.ac.osswatch.simal.web/src/main/java/uk/ac/osswatch/simal/wicket/SparqlQueryPage.java

=======================================
--- /trunk/uk.ac.osswatch.simal.web/src/main/java/uk/ac/osswatch/simal/wicket/SparqlQueryPage.java Sun Jan 10 04:53:57 2010 +++ /trunk/uk.ac.osswatch.simal.web/src/main/java/uk/ac/osswatch/simal/wicket/SparqlQueryPage.java Sun Jan 10 04:58:09 2010
@@ -41,7 +41,7 @@
 /**
  * A page for querying the RDF backend using SPARQL
  *
- * @author svanderwaal
+ * @REFACTOR use an existing SPARQL endpoint provider - see ISSUE 116
  *
  */
 public class SparqlQueryPage extends BasePage {
@@ -103,6 +103,7 @@
    */
   private SparqlResult getSparqlQueryResults(String queryStr) {
     String sparqlQuery = StringEscapeUtils.unescapeXml(queryStr);
+ // @REFACTOR the UI should not be aware of the back end implementation, we need a generic result interface for queries
     SparqlResult result = null;
     try {
       ISimalRepository repo = UserApplication.getRepository();
-- 
You received this message because you are subscribed to the Google Groups 
"Simal Commits" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/simal-commits?hl=en.


Reply via email to