Merge branch 'release/2.0.0' into develop

examples/2.7-query-reuse.php

+<?php
+
+require('init.php');
+htmlHeader();
+
+// create a client instance
+$client = new Solarium_Client($config);
+
+
+// first create a base query as a query class
+class PriceQuery extends Solarium_Query_Select
+{
+    protected function _init()
+    {
+        // set a query (all prices starting from 12)
+        $this->setQuery('price:[12 TO *]');
+
+        // set start and rows param (comparable to SQL limit) using fluent interface
+        $this->setStart(2)->setRows(20);
+
+        // set fields to fetch (this overrides the default setting 'all fields')
+        $this->setFields(array('id','name','price'));
+
+        // sort the results by price ascending
+        $this->addSort('price', self::SORT_ASC);
+    }
+}
+
+// the query instance easily be altered based on user input
+// try calling this page with "?start=10" added to the url.
+$query = new PriceQuery();
+if(isset($_GET['start']) && is_numeric($_GET['start'])){
+    $query->setStart($_GET['start']);
+}
+
+// alternatively you can use class inheritance to create query inheritance
+// in this example this class isn't actually used, but you can simple replace
+// the var $query with an instance of this class...
+class LowerPriceQuery extends PriceQuery{
+    protected function _init()
+    {
+        // this call makes sure we get all the settings of the parent class
+        parent::_init();
+
+        $this->setQuery('price:[5 TO *]');
+    }
+}
+
+// this executes the query and returns the result
+$resultset = $client->select($query);
+
+// display the total number of documents found by solr
+echo 'NumFound: '.$resultset->getNumFound();
+
+// show documents using the resultset iterator
+foreach ($resultset as $document) {
+
+    echo '<hr/><table>';
+
+    // the documents are also iterable, to get all fields
+    foreach($document AS $field => $value)
+    {
+        // this converts multivalue fields to a comma-separated string
+        if(is_array($value)) $value = implode(', ', $value);
+
+        echo '<tr><th>' . $field . '</th><td>' . $value . '</td></tr>';
+    }
+
+    echo '</table>';
+}
+
+htmlFooter();
\ No newline at end of file

examples/3.4-optimize.php

+<?php
+
+require('init.php');
+htmlHeader();
+
+// create a client instance
+$client = new Solarium_Client($config);
+
+// get an update query instance
+$update = $client->createUpdate();
+
+// optimize the index
+$update->addOptimize(true, false, 5);
+
+// this executes the query and returns the result
+$result = $client->update($update);
+
+echo '<b>Update query executed<b><br/>';
+echo 'Query status: ' . $result->getStatus(). '<br/>';
+echo 'Query time: ' . $result->getQueryTime();
+
+htmlFooter();
\ No newline at end of file

examples/3.5-rollback.php

+<?php
+
+require('init.php');
+htmlHeader();
+
+// create a client instance
+$client = new Solarium_Client($config);
+
+// get an update query instance
+$update = $client->createUpdate();
+
+// rollback any uncommitted changes on the Solr server
+$update->addRollback();
+
+// this executes the query and returns the result
+$result = $client->update($update);
+
+echo '<b>Update query executed<b><br/>';
+echo 'Query status: ' . $result->getStatus(). '<br/>';
+echo 'Query time: ' . $result->getQueryTime();
+
+htmlFooter();
\ No newline at end of file

examples/6.2-escaping.php

@@ -13,7 +13,8 @@ $query = $client->createSelect();
 $input = 'ATA "133';
 
 // in this case phrase escaping is used (most common) but you can also do term escaping, see the manual
-$query->setQuery('features:' . Solarium_Escape::phrase($input));
+$helper = $query->getHelper();
+$query->setQuery('features:' . $helper->escapePhrase($input));
 
 // this executes the query and returns the result
 $resultset = $client->select($query);

examples/autoload.php

+<?php
+
+require('../library/Solarium/Autoloader.php');
+Solarium_Autoloader::register();
\ No newline at end of file

examples/index.html

@@ -45,7 +45,7 @@
                     <li><a href="2.5.4-dismax.php">2.5.4 Dismax</a></li>
                 </ul>
                 <li><a href="2.6-helper-functions.php">2.6 Helper functions</a></li>
-
+                <li><a href="2.7-query-reuse.php">2.7 Query re-use</a></li>
             </ul>
 
             <li>3. Update query</li>
@@ -53,6 +53,8 @@
                 <li><a href="3.1-add-docs.php">3.1 Add docs</a></li>
                 <li><a href="3.2-delete-by-query.php">3.2 Delete by query</a></li>
                 <li><a href="3.3-delete-by-id.php">3.3 Delete by ID</a></li>
+                <li><a href="3.4-optimize.php">3.4 Optimize index</a></li>
+                <li><a href="3.5-rollback.php">3.5 Rollback</a></li>
             </ul>
 
             <li>4. Usage modes</li>

examples/init.php

@@ -3,8 +3,7 @@
 error_reporting(E_ALL);
 ini_set('display_errors', true);
 
-require('../library/Solarium/Autoloader.php');
-Solarium_Autoloader::register();
+require('autoload.php');
 
 if (file_exists('config.php')) {
     require('config.php');

library/Solarium/Client.php

-
 <?php
 /**
  * Copyright 2011 Bas de Nooijer. All rights reserved.

library/Solarium/Document/ReadWrite.php

@@ -124,7 +124,7 @@ class Solarium_Document_ReadWrite extends Solarium_Document_ReadOnly
      */
     public function setField($key, $value, $boost = null)
     {
-        if ($value == null) {
+        if ($value === null) {
             $this->removeField($key);
         } else {
             $this->_fields[$key] = $value;

library/Solarium/Escape.php

-<?php
-/**
- * Copyright 2011 Bas de Nooijer. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are met:
- *
- * 1. Redistributions of source code must retain the above copyright notice,
- *    this list of conditions and the following disclaimer.
- *
- * 2. Redistributions in binary form must reproduce the above copyright notice,
- *    this listof conditions and the following disclaimer in the documentation
- *    and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDER AND CONTRIBUTORS "AS IS"
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
- * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
- * POSSIBILITY OF SUCH DAMAGE.
- *
- * The views and conclusions contained in the software and documentation are
- * those of the authors and should not be interpreted as representing official
- * policies, either expressed or implied, of the copyright holder.
- *
- * @copyright Copyright 2011 Bas de Nooijer <solarium@raspberry.nl>
- * @license http://github.com/basdenooijer/solarium/raw/master/COPYING
- * @link http://www.solarium-project.org/
- *
- * @package Solarium
- */
-
-/**
- * Escape data for usage in a Solr query
- *
- * Any (user) input for a query can be passed to one of the escape methods to
- * prevent any issues with special characters.
- *
- * Do mind that you cannot build a complete query first and then pass it to
- * this method, the whole query will be escaped. You need to escape only the
- * 'content' of your query.
- *
- * @package Solarium
- */
-class Solarium_Escape
-{
-
-    /**
-     * Escape a term
-     *
-     * A term is a single word.
-     * All characters that have a special meaning in a Solr query are escaped.
-     *
-     * If you want to use the input as a phrase please use the {@link phrase()}
-     * method, because a phrase requires much less escaping.\
-     *
-     * @link http://lucene.apache.org/java/docs/queryparsersyntax.html#Escaping%20Special%20Characters
-     *
-     * @param string $input
-     * @return string
-     */
-    static public function term($input)
-    {
-        $pattern = '/(\+|-|&&|\|\||!|\(|\)|\{|}|\[|]|\^|"|~|\*|\?|:|\\\)/';
-
-        return preg_replace($pattern, '\\\$1', $input);
-    }
-
-    /**
-     * Escape a phrase
-     *
-     * A phrase is a group of words.
-     * Special characters will be escaped and the phrase will be surrounded by
-     * double quotes to group the input into a single phrase. So don't put
-     * quotes around the input.
-     *
-     * @param string $input
-     * @return string
-     */
-    static public function phrase($input)
-    {
-        return '"' . preg_replace('/("|\\\)/', '\\\$1', $input) . '"';
-    }
-
-    
-
-}
\ No newline at end of file

library/Solarium/Query.php

@@ -45,6 +45,13 @@
 abstract class Solarium_Query extends Solarium_Configurable
 {
 
+    /**
+     * Helper instance
+     *
+     * @var Solarium_Query_Helper
+     */
+    protected $_helper;
+    
     /**
      * Get type for this query
      *
@@ -101,4 +108,20 @@ abstract class Solarium_Query extends Solarium_Configurable
         return $this->getOption('resultclass');
     }
 
+    /**
+     * Get a helper instance
+     *
+     * Uses lazy loading: the helper is instantiated on first use
+     *
+     * @return Solarium_Query_Helper
+     */
+    public function getHelper()
+    {
+        if (null === $this->_helper) {
+            $this->_helper = new Solarium_Query_Helper;
+        }
+
+        return $this->_helper;
+    }
+
 }
\ No newline at end of file

library/Solarium/Query/Select/Helper.php → library/Solarium/Query/Helper.php

@@ -37,16 +37,57 @@
  */
 
 /**
- * Select query helper
+ * Query helper
  *
  * Generates small snippets for use in queries, filterqueries and sorting
  *
  * @package Solarium
  * @subpackage Query
  */
-class Solarium_Query_Select_Helper
+class Solarium_Query_Helper
 {
 
+    /**
+     * Escape a term
+     *
+     * A term is a single word.
+     * All characters that have a special meaning in a Solr query are escaped.
+     *
+     * If you want to use the input as a phrase please use the {@link phrase()}
+     * method, because a phrase requires much less escaping.\
+     *
+     * @link http://lucene.apache.org/java/docs/queryparsersyntax.html#Escaping%20Special%20Characters
+     *
+     * @param string $input
+     * @return string
+     */
+    public function escapeTerm($input)
+    {
+        $pattern = '/(\+|-|&&|\|\||!|\(|\)|\{|}|\[|]|\^|"|~|\*|\?|:|\\\)/';
+
+        return preg_replace($pattern, '\\\$1', $input);
+    }
+
+    /**
+     * Escape a phrase
+     *
+     * A phrase is a group of words.
+     * Special characters will be escaped and the phrase will be surrounded by
+     * double quotes to group the input into a single phrase. So don't put
+     * quotes around the input.
+     *
+     * Do mind that you cannot build a complete query first and then pass it to
+     * this method, the whole query will be escaped. You need to escape only the
+     * 'content' of your query.
+     *
+     * @param string $input
+     * @return string
+     */
+    public function escapePhrase($input)
+    {
+        return '"' . preg_replace('/("|\\\)/', '\\\$1', $input) . '"';
+    }
+
     /**
      * Render a range query
      *

library/Solarium/Query/Select.php

@@ -144,13 +144,6 @@ class Solarium_Query_Select extends Solarium_Query
      */
     protected $_components = array();
 
-    /**
-     * Helper instance
-     *
-     * @var Solarium_Query_Select_Helper
-     */
-    protected $_helper;
-
     /**
      * Initialize options
      *
@@ -757,19 +750,4 @@ class Solarium_Query_Select extends Solarium_Query
         return $this->getComponent(Solarium_Query_Select::COMPONENT_HIGHLIGHTING, true);
     }
 
-    /**
-     * Get a helper instance
-     *
-     * Uses lazy loading: the helper is instantiated on first use
-     *
-     * @return Solarium_Query_Select_Helper
-     */
-    public function getHelper()
-    {
-        if (null === $this->_helper) {
-            $this->_helper = new Solarium_Query_Select_Helper;
-        }
-
-        return $this->_helper;
-    }
 }
\ No newline at end of file

tests/Solarium/Document/ReadWriteTest.php

@@ -128,6 +128,20 @@ class Solarium_Document_ReadWriteTest extends PHPUnit_Framework_TestCase
         );
     }
 
+    public function testSetFieldWithFalsyValue()
+    {
+        $falsy_value = '';
+        $this->_doc->setField('name', $falsy_value);
+ 
+        $expectedFields = $this->_fields;
+        $expectedFields['name'] = $falsy_value;
+ 
+        $this->assertEquals(
+            $expectedFields,
+            $this->_doc->getFields()
+        );
+    }
+
     public function testRemoveField()
     {
         $this->_doc->removeField('name');

tests/Solarium/EscapeTest.php

-<?php
-/**
- * Copyright 2011 Bas de Nooijer. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are met:
- *
- * 1. Redistributions of source code must retain the above copyright notice,
- *    this list of conditions and the following disclaimer.
- *
- * 2. Redistributions in binary form must reproduce the above copyright notice,
- *    this listof conditions and the following disclaimer in the documentation
- *    and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDER AND CONTRIBUTORS "AS IS"
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
- * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
- * POSSIBILITY OF SUCH DAMAGE.
- *
- * The views and conclusions contained in the software and documentation are
- * those of the authors and should not be interpreted as representing official
- * policies, either expressed or implied, of the copyright holder.
- */
-
-class Solarium_EscapeTest extends PHPUnit_Framework_TestCase
-{
-
-    public function testTerm()
-    {
-        $this->assertEquals(
-            'a\\+b',
-            Solarium_Escape::term('a+b')
-        );
-    }
-
-    public function testTermNoEscape()
-    {
-        $this->assertEquals(
-            'abc',
-            Solarium_Escape::term('abc')
-        );
-    }
-
-    public function testPhrase()
-    {
-        $this->assertEquals(
-            '"a+\\"b"',
-            Solarium_Escape::phrase('a+"b')
-        );
-    }
-
-    public function testPhraseNoEscape()
-    {
-        $this->assertEquals(
-            '"a+b"',
-            Solarium_Escape::phrase('a+b')
-        );
-    }
-    
-}
\ No newline at end of file

tests/Solarium/Query/Select/HelperTest.php → tests/Solarium/Query/HelperTest.php

@@ -29,13 +29,13 @@
  * policies, either expressed or implied, of the copyright holder.
  */
 
-class Solarium_Query_Select_HelperTest extends PHPUnit_Framework_TestCase
+class Solarium_Query_HelperTest extends PHPUnit_Framework_TestCase
 {
     protected $_helper;
 
     public function setUp()
     {
-        $this->_helper = new Solarium_Query_Select_Helper;
+        $this->_helper = new Solarium_Query_Helper;
     }
 
     public function testRangeQueryInclusive()
@@ -120,4 +120,36 @@ class Solarium_Query_Select_HelperTest extends PHPUnit_Framework_TestCase
         );
     }
 
+    public function testEscapeTerm()
+    {
+        $this->assertEquals(
+            'a\\+b',
+            $this->_helper->escapeTerm('a+b')
+        );
+    }
+
+    public function testEscapeTermNoEscape()
+    {
+        $this->assertEquals(
+            'abc',
+            $this->_helper->escapeTerm('abc')
+        );
+    }
+
+    public function testEscapePhrase()
+    {
+        $this->assertEquals(
+            '"a+\\"b"',
+            $this->_helper->escapePhrase('a+"b')
+        );
+    }
+
+    public function testEscapePhraseNoEscape()
+    {
+        $this->assertEquals(
+            '"a+b"',
+            $this->_helper->escapePhrase('a+b')
+        );
+    }
+
 }

tests/Solarium/Query/SelectTest.php

@@ -479,16 +479,6 @@ class Solarium_Query_SelectTest extends PHPUnit_Framework_TestCase
         );
     }
     
-    public function testGetHelper()
-    {
-        $helper = $this->_query->getHelper();
-
-        $this->assertEquals(
-            'Solarium_Query_Select_Helper',
-            get_class($helper)
-        );
-    }
-
     public function testRegisterComponentType()
     {
         $components = $this->_query->getComponentTypes();

tests/Solarium/QueryTest.php

@@ -46,6 +46,17 @@ class Solarium_QueryTest extends PHPUnit_Framework_TestCase
         $this->assertEquals('myResultClass', $query->getResultClass());
     }
 
+    public function testGetHelper()
+    {
+        $query = new TestQuery;
+        $helper = $query->getHelper();
+
+        $this->assertEquals(
+            'Solarium_Query_Helper',
+            get_class($helper)
+        );
+    }
+    
 }
 
 class TestQuery extends Solarium_Query