From df95b1257ff0201b44aec092cdd4d70018beb14f Mon Sep 17 00:00:00 2001
From: "Michael J. Rubinsky" <mrubinsk@horde.org>
Date: Mon, 19 Jul 2010 12:41:30 -0400
Subject: [PATCH] toDriverCharset() needs to be public, rename it as such.

Share objects need to be able to call this method.
---
 framework/Share/lib/Horde/Share.php                  | 11 +++++++++++
 framework/Share/lib/Horde/Share/Object/Sql.php       |  2 +-
 framework/Share/lib/Horde/Share/Sql.php              |  7 +++----
 framework/Share/lib/Horde/Share/Sql/Hierarchical.php |  2 +-
 4 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/framework/Share/lib/Horde/Share.php b/framework/Share/lib/Horde/Share.php
index 74e48869a..16aa12bb4 100644
--- a/framework/Share/lib/Horde/Share.php
+++ b/framework/Share/lib/Horde/Share.php
@@ -416,6 +416,17 @@ class Horde_Share
     }
 
     /**
+     * Convert TO the storage driver's charset. Individual share objects should
+     * implement this method if needed.
+     *
+     * @param array $data  Data to be converted.
+     */
+    public function toDriverCharset($data)
+    {
+        // noop
+    }
+    
+    /**
      * Utility function to be used with uasort() for sorting arrays of
      * Horde_Share objects.
      *
diff --git a/framework/Share/lib/Horde/Share/Object/Sql.php b/framework/Share/lib/Horde/Share/Object/Sql.php
index 6301af001..5b648ad8d 100644
--- a/framework/Share/lib/Horde/Share/Object/Sql.php
+++ b/framework/Share/lib/Horde/Share/Object/Sql.php
@@ -107,7 +107,7 @@ class Horde_Share_Object_Sql extends Horde_Share_Object
         $fields = array();
         $params = array();
 
-        foreach ($this->_shareOb->_toDriverCharset($this->data) as $key => $value) {
+        foreach ($this->_shareOb->toDriverCharset($this->data) as $key => $value) {
             if ($key != 'share_id' && $key != 'perm' && $key != 'share_flags') {
                 $fields[] = $key;
                 $params[] = $value;
diff --git a/framework/Share/lib/Horde/Share/Sql.php b/framework/Share/lib/Horde/Share/Sql.php
index 0c10c142c..52a6a5c6e 100644
--- a/framework/Share/lib/Horde/Share/Sql.php
+++ b/framework/Share/lib/Horde/Share/Sql.php
@@ -749,7 +749,7 @@ class Horde_Share_Sql extends Horde_Share
         }
 
         $attributes = $this->_toDriverKeys($attributes);
-        $attributes = $this->_toDriverCharset($attributes);
+        $attributes = $this->toDriverCharset($attributes);
 
         if (is_array($attributes)) {
             // Build attribute/key filter.
@@ -896,10 +896,9 @@ class Horde_Share_Sql extends Horde_Share
     /**
      * Utility function to convert TO the SQL server's charset.
      *
-     * @TODO: This needs to be public since it's called by the share object.
-     * Look at making this outright public or maybe moving it to the object.
+     * @see Horde_Share#toDriverCharset
      */
-    public function _toDriverCharset($data)
+    public function toDriverCharset($data)
     {
         if (!is_array($data)) {
             return $data;
diff --git a/framework/Share/lib/Horde/Share/Sql/Hierarchical.php b/framework/Share/lib/Horde/Share/Sql/Hierarchical.php
index 08fd380dd..05c2717a3 100644
--- a/framework/Share/lib/Horde/Share/Sql/Hierarchical.php
+++ b/framework/Share/lib/Horde/Share/Sql/Hierarchical.php
@@ -236,7 +236,7 @@ class Horde_Share_Sql_Hierarchical extends Horde_Share_Sql
         $attributes = $this->_toDriverKeys($attributes);
 
         /* ...and to driver charset */
-        $attributes = $this->_toDriverCharset($attributes);
+        $attributes = $this->toDriverCharset($attributes);
 
         if (is_array($attributes)) {
             // Build attribute/key filter.
-- 
2.11.0