Dev

.editorconfig

@@ -0,0 +1,16 @@
+root = true
+
+[*]
+charset = utf-8
+indent_style = space
+indent_size = 4
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true
+
+[*.{json,yml,todo,md}]
+indent_size = 2
+
+[*.md]
+trim_trailing_whitespace = false
+insert_final_newline = false

.travis.yml

@@ -2,16 +2,15 @@ language: php
 
 php:
   - 5.4
-  - 5.5
   - 5.6
   - 7.0
   - 7.1
 
 before_script:
   - composer self-update
-  - composer install --prefer-source --no-interaction
+  - composer install --prefer-dist --no-interaction
 
-script: phpunit
+script: vendor/bin/phpunit
 
 branches:
   only:

composer.json

@@ -1,19 +1,18 @@
 {
     "name": "aak74/bx-data",
-    "version": "0.3.7",
+    "version": "0.6.3",
     "description": "Классы для удобного доступа к данным в 1C-Bitrix",
     "license": "MIT",
     "keywords": ["bitrix", "1c-bitrix", "classes", "akop", "highload blocks", "битрикс", "1С-битрикс"],
-    "type": "bitrix-d7-module",
+    "type": "library",
     "authors": [
         {
             "name": "Andrew Kopylov",
             "email": "[email protected]"
         }
     ],
     "require": {
-        "php": ">=5.4.0",
-        "composer/installers": "^1.0"
+        "php": ">=5.6.0"
     },
     "require-dev": {
         "phpunit/phpunit": "4.*"

doc/classes/Element/HlElement.md

@@ -12,11 +12,13 @@ namespace App\Model;
 
 class Model extends \Akop\Element\HlElement
 {
-	protected $entityName = 'Model';
+  protected $entityName = 'Model';
 }
 ```
 
 ## Типичный getList
+
+### Код
 ```PHP
 <?php
 $models = new \App\Model\Model;
@@ -28,8 +30,7 @@ $result = $models->getList([
 ]);
 ```
 
-## Формируется запрос вида:
-
+### Формируется запрос вида:
 ```SQL
 SELECT
   `model`.`ID` AS `ID`,
@@ -41,3 +42,25 @@ LEFT JOIN `b_hlbd_auto_brand` `model_brandname_` ON `model`.`UF_BRAND` = `model_
 WHERE `model`.`UF_BRAND` = 120
 ORDER BY `model`.`UF_NAME` ASC
 ```
+
+## Выборка с ИЛИ
+### Код
+```PHP
+<?php
+$models = new \App\Model\Model;
+
+$result = $models->getList([
+  'filter' => [
+    'LOGIC' => 'OR',
+    'brandId' => 120,
+    '<id' => 30,
+  ],
+]);
+```
+
+### Формируется запрос вида:
+```SQL
+SELECT *
+FROM `b_hlbd_auto_model` `model`
+WHERE `model`.`UF_BRAND` = 120 OR `model`.`ID` < 30
+```

doc/functions/add.md

@@ -0,0 +1,16 @@
+# add
+## Описание
+```php
+add(array $params) : integer;
+```
+Добавляет строку в модель.
+Возвращает `id` вставленной записи или `false` в случае неудачи.
+
+## Пример
+```php
+add([
+  'name' => 'Renault',
+  'active' => 'Y',
+  'sort' => '200'
+]);
+```

doc/functions/all.md

@@ -13,20 +13,20 @@ getRow(array $params = [])
 ```
 Возвращает **одну строку** в соответствии с переданным параметром.
 
-## add
+## [add](add.md)
 ```php
 add(array $params)
 ```
 Добавляет строку.
 
-## update
+## [update](update.md)
 ```php
 update($primaryKey, array $params)
 ```
 Обновляетвляет строку.
 
-## upsert
+## [upsert](upsert.md)
 ```php
 upsert(array $filter, array $params)
 ```
-Обновляет строку если найдена строка в соответствии с фильтром. Если такой строки нет, то добавляет строку.
+Обновляет строку если найдена строка в соответствии с фильтром. Если такой строки нет, то добавляет строку.

doc/functions/update.md

@@ -0,0 +1,19 @@
+# update
+## Описание
+```php
+update($id, array $params) : integer;
+```
+Обновляет строку в модели.
+Возвращает `id` измененной записи.
+
+## Пример
+```php
+update(
+  1,
+  [
+    'name' => 'Renault',
+    'active' => 'Y',
+    'sort' => '200'
+  ]
+);
+```

doc/functions/upsert.md

@@ -0,0 +1,22 @@
+# upsert
+## Описание
+```php
+upsert(array $filter, array $params) : integer;
+```
+Обновляет или добавляет строку в модели.
+Ищет строку, которая соотвествует фильтру. Если строка найдена, то обновляет ее. Если строка не найдена, то добавляет запись.
+Возвращает `id` измененной (вставленной) записи.
+
+## Пример
+```php
+upsert(
+  [
+    'name' => 'Renault',
+  ],
+  [
+    'name' => 'Renault',
+    'active' => 'Y',
+    'sort' => '200'
+  ]
+);
+```

phpmd.ruleset.xml

@@ -0,0 +1,27 @@
+<?xml version="1.0"?>
+<ruleset name="My first PHPMD rule set"
+         xmlns="http://pmd.sf.net/ruleset/1.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://pmd.sf.net/ruleset/1.0.0
+                     http://pmd.sf.net/ruleset_xml_schema.xsd"
+         xsi:noNamespaceSchemaLocation="
+                     http://pmd.sf.net/ruleset_xml_schema.xsd">
+    <description>
+        My custom rule set that checks my code...
+    </description>
+
+    <!-- Import the entire unused code rule set -->
+    <rule ref="rulesets/unusedcode.xml" />
+    <rule ref="rulesets/naming.xml">
+        <exclude name="ShortVariable" />
+    </rule>
+    <rule ref="rulesets/unusedcode.xml" />
+    <rule ref="rulesets/design.xml" />
+    <rule ref="rulesets/controversial.xml" />
+
+    <rule ref="rulesets/codesize.xml/CyclomaticComplexity" />
+
+    <rule ref="rulesets/cleancode.xml">
+        <exclude name="StaticAccess" />
+    </rule>
+</ruleset>

src/Element/AbstractElement.php

@@ -10,11 +10,16 @@ class AbstractElement implements ElementInterface
     use ParamTrait;
 
     protected $fieldsBase = [];
+    protected $dateFormat = 'd.m.Y';
+    protected $dateTimeFormat = 'd.m.Y H:i:s';
+    protected $dates = [];
     protected $fields = ["ID"];
     protected $reversedFields = [];
     protected $compressedFields = [];
     protected $primaryKey = "ID";
     protected $isAssoc = true;
+    protected $fieldsStripTags = [];
+    protected $fieldsIgnore = [];
 
     private $errorMesage = '';
     private $lastOperation = false;
@@ -133,7 +138,14 @@ public function upsert(array $filter, array $params)
             "filter" => $filter,
         ));
 
-        $primaryKey = $item[$this->primaryKey] || $item[strtolower($this->primaryKey)];
+        $primaryKey = null;
+        if ($item[$this->primaryKey]) {
+            $primaryKey = $item[$this->primaryKey];
+        }
+        if ($item[strtolower($this->primaryKey)]) {
+            $primaryKey = $item[strtolower($this->primaryKey)];
+        }
+
         if ($item && $primaryKey) {
             $this->update($primaryKey, $params);
             return $primaryKey;
@@ -200,16 +212,6 @@ private function compressField($fieldName, $fieldValue)
             : $fieldValue;
     }
 
-/*
-    private function uncompressFields(array $fields)
-    {
-        foreach ($fields as $fieldName => $fieldValue) {
-            $result[$fieldName] = $this->uncompressField($fieldName, $fieldValue);
-        }
-        return $result;
-    }
-*/
-
     private function uncompressField($fieldName, $fieldValue)
     {
         return (in_array($fieldName, $this->compressedFields))
@@ -234,25 +236,77 @@ protected function setErrorMessage($message)
     }
 
     /**
-     * Возвращает набор данных с переименованными полями
-     * Если поля были сжаты, то разжимает их
+     * Возвращает обработанный набор данных
+     *  - Переименовывает поля
+     *  - Если поля были сжаты, то разжимает их
+     *  - Если поля указаны в массиве дат, то значение преобразуются в строку
      * @param  [array] $item набор данных
      * @return [array]
      */
-    protected function getRenamed($item)
+    protected function getProcessed($item)
     {
         if (!empty($this->reversedFields)) {
             foreach ($item as $key => $value) {
                 $fieldName = (isset($this->reversedFields[$key]))
                     ? $this->reversedFields[$key]
                     : $key;
 
-                $result[$fieldName] = $this->uncompressField($fieldName, $value);
+                $value = $this->uncompressField($fieldName, $value);
+                $value = $this->convertDateFromDB($fieldName, $value);
+                $result[$fieldName] = $this->stripTags($fieldName, $value);
             }
         }
         return $result;
     }
 
+    /**
+     * Удалят html теги
+     */
+    private function stripTags($fieldName, $value)
+    {
+        return (!empty($value) && in_array($fieldName, $this->fieldsStripTags))
+            ? strip_tags($value)
+            : $value;
+    }
+
+    /**
+     * Если поле указано в массиве дат, то возвращает вместо объекта строку
+     * Для того, чтобы избежать это преобразование достаточно не заполнять массив дат
+     */
+    private function convertDateFromDB($fieldName, $value)
+    {
+        // \Akop\Util::pre([$fieldName, $value, $this->dates, in_array($fieldName, $this->dates)], 'convertDateFromDB');
+        return (!empty($value) && in_array($fieldName, $this->dates))
+            ? $value->toString()
+            : $value;
+    }
+
+    /** 
+     * Преобразует строку в объект дата, пригодный для сохранения в БД
+     */
+    private function convertDateToDB($fieldName, $value)
+    {
+        if (in_array($fieldName, $this->dates) && !empty($value)) {
+            if ($this->isTimeProbably($value)) {
+                $dt = new \Bitrix\Main\Type\DateTime($value, $this->dateTimeFormat);
+                return $dt->format('Y-m-d H:i:s');
+            }
+            $dt = new \Bitrix\Main\Type\DateTime($value, $this->dateFormat);
+            return $dt->format('Y-m-d');
+        }
+        return $value;
+    }
+
+    /**
+     * Является ли строка временем (предположительно)
+     * так как используется очень ограниченно, то такое условие вполне достаточно
+     * При особых DateFormat следует переопределить функцию
+     */
+    protected function isTimeProbably($value)
+    {
+        return (strlen($value) > 10);
+    }
+
     protected function updateValueForReverse($value)
     {
         return $value;
@@ -277,4 +331,10 @@ private function reverseFields()
             }
         }
     }
+
+    protected function getLastId()
+    {
+        global $DB;
+        return $DB->LastID();
+    }    
 }