aboutsummaryrefslogtreecommitdiff
path: root/engine/lib/metastrings.php
diff options
context:
space:
mode:
authorBrett Profitt <brett.profitt@gmail.com>2011-04-16 17:43:18 -0400
committerBrett Profitt <brett.profitt@gmail.com>2011-04-16 17:43:18 -0400
commit628aa953ec150a1ed34680ef1d4f9c81dfc99bb1 (patch)
tree30e91cb88799a7c81be03759bbcec7d5bf255d1e /engine/lib/metastrings.php
parent309dff2bf948ab191ccb1e8d4a777e49ad498820 (diff)
downloadelgg-628aa953ec150a1ed34680ef1d4f9c81dfc99bb1.tar.gz
elgg-628aa953ec150a1ed34680ef1d4f9c81dfc99bb1.tar.bz2
Denormalizing annotation names for calculation functions.
Diffstat (limited to 'engine/lib/metastrings.php')
-rw-r--r--engine/lib/metastrings.php69
1 files changed, 50 insertions, 19 deletions
diff --git a/engine/lib/metastrings.php b/engine/lib/metastrings.php
index 604c7f765..6cdf9ee47 100644
--- a/engine/lib/metastrings.php
+++ b/engine/lib/metastrings.php
@@ -229,6 +229,8 @@ function delete_orphaned_metastrings() {
*
* metastring_type => STR metadata or annotation(s)
*
+ * denormalize => Mixed Denormalize nothing (false, default), all, name, or value.
+ *
* @return mixed
* @access private
*/
@@ -293,7 +295,8 @@ function elgg_get_metastring_based_objects($options) {
'wheres' => array(),
'joins' => array(),
- 'callback' => $callback
+ 'callback' => $callback,
+ 'denormalize' => false
);
// @todo Ignore site_guid right now because of #2910
@@ -378,7 +381,6 @@ function elgg_get_metastring_based_objects($options) {
$joins[] = "JOIN {$db_prefix}metastrings n on n_table.name_id = n.id";
$joins[] = "JOIN {$db_prefix}metastrings v on n_table.value_id = v.id";
-
// remove identical join clauses
$joins = array_unique($joins);
@@ -393,7 +395,7 @@ function elgg_get_metastring_based_objects($options) {
// metastrings
$metastring_clauses = elgg_get_metastring_sql('n_table', $options['metastring_names'],
$options['metastring_values'], null, $options['metastring_ids'],
- $options['metastring_case_sensitive']);
+ $options['metastring_case_sensitive'], $options['denormalize']);
if ($metastring_clauses) {
$wheres = array_merge($wheres, $metastring_clauses['wheres']);
@@ -405,7 +407,12 @@ function elgg_get_metastring_based_objects($options) {
$options['metastring_calculation'] = 'count';
}
- if ($options['metastring_calculation'] === ELGG_ENTITIES_NO_VALUE) {
+ if ($options['metastring_calculation']) {
+// $joins[] = "JOIN {$db_prefix}metastrings v on n_table.value_id = v.id";
+
+ $function = sanitise_string($options['metastring_calculation']);
+ $query = "SELECT {$function}(v.string) as calculation FROM {$db_prefix}$type n_table";
+ } else {
// evalutate selects
if ($options['selects']) {
$selects = '';
@@ -418,8 +425,8 @@ function elgg_get_metastring_based_objects($options) {
$query = "SELECT DISTINCT n_table.*, n.string as name,
v.string as value{$selects} FROM {$db_prefix}$type n_table";
- } else {
- $query = "SELECT {$options['metastring_calculation']}(v.string) as calculation FROM {$db_prefix}$type n_table";
+
+// $query = "SELECT DISTINCT n_table.*{$selects} FROM {$db_prefix}$type n_table";
}
// add joins
@@ -459,7 +466,7 @@ function elgg_get_metastring_based_objects($options) {
$offset = sanitise_int($options['offset']);
$query .= " LIMIT $offset, $limit";
}
-
+global $test; if ($test) { var_dump($options, $query); }
$dt = get_data($query, $options['callback']);
return $dt;
} else {
@@ -479,11 +486,13 @@ function elgg_get_metastring_based_objects($options) {
* @param array $pairs Name / value pairs. Not currently used.
* @param array $ids Metastring IDs
* @param bool $case_sensitive Should name and values be case sensitive?
+ * @param mixed $denormalize Denormalize none (false, default), all, name, value. If operators
+ * are anything but = or IN you cannot denormalize value.
*
* @return array
*/
function elgg_get_metastring_sql($table, $names = null, $values = null,
- $pairs = null, $ids = null, $case_sensitive = false) {
+ $pairs = null, $ids = null, $case_sensitive = false, $denormalize = false) {
if ((!$names && $names !== 0)
&& (!$values && $values !== 0)
@@ -506,7 +515,7 @@ function elgg_get_metastring_sql($table, $names = null, $values = null,
$access = get_access_sql_suffix($table);
$return = array (
- 'joins' => array (),
+ 'joins' => array(),
'wheres' => array()
);
@@ -515,28 +524,50 @@ function elgg_get_metastring_sql($table, $names = null, $values = null,
// get names wheres and joins
$names_where = '';
if ($names !== NULL) {
+ // @todo only really needed for metadata (vs annotations)
+// $return['joins'][] = "JOIN {$db_prefix}metastrings n on n_table.name_id = n.id";
+
if (!is_array($names)) {
$names = array($names);
}
+
+ if ($denormalize == 'all' || $denormalize == 'name') {
+ $name_ids = array();
+ foreach ($names as $name) {
+ if (!$name) {
+ $name = '0';
+ }
+
+ $name_ids[] = get_metastring_id($name, $case_sensitive);
+ }
- $sanitised_names = array();
- foreach ($names as $name) {
- // normalise to 0.
- if (!$name) {
- $name = '0';
+ $names_str = implode(',', $name_ids);
+ if ($names_str) {
+ $names_where = "(n_table.name_id IN ($names_str))";
+ }
+ } else {
+ $sanitised_names = array();
+ foreach ($names as $name) {
+ // normalise to 0.
+ if (!$name) {
+ $name = '0';
+ }
+ $sanitised_names[] = '\'' . sanitise_string($name) . '\'';
}
- $sanitised_names[] = '\'' . sanitise_string($name) . '\'';
- }
- if ($names_str = implode(',', $sanitised_names)) {
- $return['joins'][] = "JOIN {$db_prefix}metastrings msn on $table.name_id = msn.id";
- $names_where = "(msn.string IN ($names_str))";
+ $names_str = implode(',', $sanitised_names);
+ if ($names_str) {
+ $return['joins'][] = "JOIN {$db_prefix}metastrings msn on $table.name_id = msn.id";
+ $names_where = "(msn.string IN ($names_str))";
+ }
}
}
// get values wheres and joins
$values_where = '';
if ($values !== NULL) {
+// $return['joins'][] = "JOIN {$db_prefix}metastrings v on n_table.value_id = v.id";
+
if (!is_array($values)) {
$values = array($values);
}