[tracker/sparql-refactor: 5/6] SPARQL: Fix variable binding with subqueries
- From: Jürg Billeter <juergbi src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [tracker/sparql-refactor: 5/6] SPARQL: Fix variable binding with subqueries
- Date: Fri, 26 Mar 2010 15:06:40 +0000 (UTC)
commit a4afd8c1e107343aec49d027c681d70c9bf33cb1
Author: Jürg Billeter <j bitron ch>
Date: Wed Mar 24 17:16:15 2010 +0100
SPARQL: Fix variable binding with subqueries
src/libtracker-data/tracker-sparql-query.vala | 30 ++++++++++++++----------
1 files changed, 17 insertions(+), 13 deletions(-)
---
diff --git a/src/libtracker-data/tracker-sparql-query.vala b/src/libtracker-data/tracker-sparql-query.vala
index 8a13a1a..227a1cf 100644
--- a/src/libtracker-data/tracker-sparql-query.vala
+++ b/src/libtracker-data/tracker-sparql-query.vala
@@ -732,14 +732,11 @@ public class Tracker.SparqlQuery : Object {
sql.append_printf (" AS %s", variable.sql_expression);
if (subquery) {
- if (variable.binding != null) {
- throw get_error ("redefining variable `?%s'".printf (variable.name));
- }
-
- variable.binding = new VariableBinding ();
- variable.binding.data_type = type;
- variable.binding.variable = variable;
- variable.binding.sql_expression = variable.sql_expression;
+ var binding = new VariableBinding ();
+ binding.data_type = type;
+ binding.variable = variable;
+ binding.sql_expression = variable.sql_expression;
+ add_variable_binding (new StringBuilder (), binding, VariableState.BOUND);
}
}
@@ -2864,7 +2861,10 @@ public class Tracker.SparqlQuery : Object {
}
VariableBindingList? get_variable_binding_list (Variable variable) {
- var binding_list = triple_context.var_map.lookup (variable);
+ VariableBindingList binding_list = null;
+ if (triple_context != null) {
+ binding_list = triple_context.var_map.lookup (variable);
+ }
if (binding_list == null && context.in_scalar_subquery) {
// in scalar subquery: check variables of outer queries
var parent_context = context.parent_context;
@@ -2878,8 +2878,10 @@ public class Tracker.SparqlQuery : Object {
binding.type = outer_var.binding.type;
binding.sql_expression = outer_var.sql_expression;
binding_list = new VariableBindingList ();
- triple_context.variables.append (binding.variable);
- triple_context.var_map.insert (binding.variable, binding_list);
+ if (triple_context != null) {
+ triple_context.variables.append (binding.variable);
+ triple_context.var_map.insert (binding.variable, binding_list);
+ }
context.var_set.insert (binding.variable, VariableState.BOUND);
binding_list.list.append (binding);
@@ -2896,8 +2898,10 @@ public class Tracker.SparqlQuery : Object {
var binding_list = get_variable_binding_list (binding.variable);
if (binding_list == null) {
binding_list = new VariableBindingList ();
- triple_context.variables.append (binding.variable);
- triple_context.var_map.insert (binding.variable, binding_list);
+ if (triple_context != null) {
+ triple_context.variables.append (binding.variable);
+ triple_context.var_map.insert (binding.variable, binding_list);
+ }
sql.append_printf ("%s AS %s, ",
binding.sql_expression,
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]