Merge branch '2013.10' into 2014.11
authorPhilipp Schüle <p.schuele@metaways.de>
Wed, 1 Apr 2015 15:59:44 +0000 (17:59 +0200)
committerPhilipp Schüle <p.schuele@metaways.de>
Wed, 1 Apr 2015 15:59:44 +0000 (17:59 +0200)
1  2 
tests/tine20/Tinebase/Frontend/Json/PersistentFilterTest.php
tine20/Sales/Controller/Contract.php
tine20/Tinebase/Model/Filter/Id.php
tine20/Tinebase/Record/Abstract.php

@@@ -76,15 -56,55 +76,18 @@@ class Sales_Controller_Contract extend
       * get by id
       *
       * @param string $_id
+      * @param int $_containerId
+      * @param bool         $_getRelatedData
       * @return Tinebase_Record_RecordSet
       */
-     public function get($_id)
+     public function get($_id, $_containerId = NULL, $_getRelatedData = TRUE)
      {
-         $sharedContracts = $this->getSharedContractsContainer();
-         return parent::get($_id, $sharedContracts->getId());
+         $containerId = $_containerId !== null ? $_containerId : $this->getSharedContractsContainer();
+         
+         return parent::get($_id, $containerId, $_getRelatedData);
      }
      
 -    /**
 -     * Checks if number is unique if manual generated
 -     * @param Tinebase_Record_Interface $r
 -     * @param Boolean $update true if called un update
 -     * @throws Tinebase_Exception_Duplicate
 -     */
 -    private function _checkNumberUniquity($r, $update = false)
 -    {
 -        $filterArray = array(array('field' => 'number', 'operator' => 'equals', 'value' => $r->__get('number')));
 -        if($update) {
 -            $filterArray[] = array('field' => 'id', 'operator' => 'notin', 'value' => $r->getId());
 -        }
 -        $filter = new Sales_Model_ContractFilter($filterArray);
 -        $existing = $this->search($filter);
  
 -        if(count($existing->toArray()) > 0) {
 -            $e = new Tinebase_Exception_Duplicate(_('The number you have tried to set is already in use!'));
 -            $e->setData($existing);
 -            $e->setClientRecord($r);
 -            throw $e;
 -        }
 -        return true;
 -    }
 -    /**
 -     * Checks if number is unique if manual generated
 -     * @param Tinebase_Record_Interface $r
 -     * @throws Tinebase_Exception_Record_Validation
 -     */
 -    private function _checkNumberType($_record)
 -    {
 -        $number = $_record->__get('number');
 -        if(empty($number)) {
 -            throw new Tinebase_Exception_Record_Validation('Please use a contract number!');
 -        } elseif ((Sales_Config::getInstance()->get('contractNumberValidation', 'integer') == 'integer') && !is_numeric($number)) {
 -            throw new Tinebase_Exception_Record_Validation('Please use a decimal number as contract number!');
 -        }
 -    }
 -    
      /**
       * @see Tinebase_Controller_Record_Abstract::update()
       */
Simple merge
Simple merge