test composer req

.docker/config/Dockerfile

@@ -0,0 +1,64 @@
+FROM php:5.5-cli
+
+#
+# Tools & libraries
+#
+RUN apt-get update \
+    && apt-get install -y --no-install-recommends \
+        vim \
+        git \
+        zip \
+        unzip \
+        zlib1g-dev \
+        libicu-dev \
+    && apt-get clean \
+    && rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/*
+
+#
+# PHP extensions
+#
+RUN docker-php-ext-install \
+    zip \
+    intl \
+    mbstring
+
+#
+# PHP configuration:
+# - default configuration
+# - timezone
+#
+COPY php.ini /usr/local/etc/php/php.ini
+ARG TIMEZONE
+RUN echo "\n""date.timezone = $TIMEZONE""\n" >> /usr/local/etc/php/php.ini
+
+#
+# Xdebug
+#
+RUN pecl install xdebug \
+    && docker-php-ext-enable xdebug
+COPY xdebug.ini /usr/local/etc/php/conf.d/docker-php-ext-xdebug.ini
+
+#
+# Phing
+#
+RUN pear channel-discover pear.phing.info \
+    && pear install [--alldeps] phing/phing
+
+#
+# Composer + https://packagist.org/packages/hirak/prestissimo package
+#
+RUN php -r "copy('https://getcomposer.org/installer', 'composer-setup.php');" \
+    && php -r "if (hash_file('SHA384', 'composer-setup.php') === \
+    '544e09ee996cdf60ece3804abc52599c22b1f40f4323403c44d44fdfdd586475ca9813a858088ffbc1f233e9b180f061') { echo \
+    'Installer verified'; } else { echo 'Installer corrupt'; unlink('composer-setup.php'); } echo PHP_EOL;" \
+    && php composer-setup.php --install-dir=/usr/local/bin --filename=composer \
+    && php -r "unlink('composer-setup.php');" \
+    && composer global require --no-plugins --no-scripts hirak/prestissimo \
+    && rm -rf /root/.composer/cache/*
+
+#
+# Bash
+#
+RUN sed -i 's/^# export/export/g' /root/.bashrc \
+    && sed -i 's/^# alias/alias/g' /root/.bashrc \
+    && echo "\n"'export PATH=/project/vendor/bin:$PATH'"\n" >> /root/.bashrc

.env

@@ -0,0 +1 @@
+TIMEZONE=Europe/Warsaw

.env.dist

@@ -1,15 +0,0 @@
-# -----------------------------------------------------------------------------
-### Docker
-# -----------------------------------------------------------------------------
-
-#
-# All containers
-#
-DOCKER_CONTAINER_OWNER=meritoo
-DOCKER_CONTAINER_PROJECT=common-library
-
-#
-# PHP configuration:
-# - timezone
-#
-TIMEZONE=Europe/Warsaw

.gitignore

@@ -1,58 +1,51 @@
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
-# Environment-related parameters
-# ------------------------------------------------------------------------------
-.env
-
-
-# ------------------------------------------------------------------------------
 ### Vendors
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 /vendor/
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### Composer
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 /composer.lock
 /composer.phar
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### Phing
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
-/phing/properties
+/.phing/properties
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### PHPUnit
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 /phpunit.xml
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
-### PHP Coding Standards Fixer
+### PHP Coding Standards Fixer generated files
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
-/.php_cs
 /.php_cs.cache
 
 
-# -----------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### Build files
-# -----------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
-/build/
+/.build/
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### Generated databases
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
-/data/tmp
+/.data/tmp
 *.sql
 *.sqlite
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### Compiled source
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 *.com
 *.class
 *.dll
@@ -61,22 +54,23 @@
 *.so
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### Shell scripts
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 /*.sh
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### JetBrains
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 /.idea
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### NetBeans template
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 nbproject/private/
+build/
 nbbuild/
 dist/
 nbdist/
@@ -84,9 +78,9 @@ nbactions.xml
 .nb-gradle/
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### OSX template
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 .DS_Store
 .AppleDouble
 .LSOverride
@@ -113,9 +107,9 @@ Temporary Items
 .apdisk
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### Linux template
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 *~
 
 # temporary files which can be created if a process still has a handle open of a deleted file
@@ -128,9 +122,9 @@ Temporary Items
 .Trash-*
 
 
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 ### Windows template
-# ------------------------------------------------------------------------------
+# ----------------------------------------------------------------------------------------------------------------------
 # Windows image file caches
 Thumbs.db
 ehthumbs.db

.phing/app.xml

@@ -2,12 +2,12 @@
 <project name="Meritoo Package" basedir="." default="build:main" phingVersion="2.14.0">
     <!-- Properties -->
     <if>
-        <available file="phing/properties" property="custom.properties.available"/>
+        <available file=".phing/properties" property="custom.properties.available"/>
         <then>
-            <property file="phing/properties"/>
+            <property file=".phing/properties" />
         </then>
         <else>
-            <property file="phing/properties.dist"/>
+            <property file=".phing/properties.dist" />
         </else>
     </if>
 
@@ -21,9 +21,9 @@
             depends="app:composer, app:vendors, app:checkout"
             description="Prepares app to build." />
 
-    <!-- Updates Composer and validates composer.* files -->
+    <!-- Check / update composer -->
-    <target name="app:composer" description="Updates Composer and validates composer.* files">
+    <target name="app:composer" description="Checks / updates composer">
-        <echo msg="Updating Composer and validating composer.* files..." />
+        <echo msg="Checking / updating composer..." />
 
         <if>
             <available file="composer.phar" />
@@ -42,14 +42,7 @@
                 </if>
             </else>
         </if>
-
-        <!-- Update Composer -->
         <composer command="selfupdate" />
-
-        <!-- Validate Composer -->
-        <composer command="validate">
-            <arg line="--no-check-all --strict" />
-        </composer>
     </target>
 
     <!-- Project Install/update vendors -->
@@ -75,8 +68,6 @@
             <then>
                 <composer php="${composer.php}" composer="${composer.path}" command="install">
                     <arg value="--optimize-autoloader" />
-                    <arg value="--prefer-dist" />
-                    <arg value="--classmap-authoritative" />
                 </composer>
             </then>
             <else>

.phing/properties.dist

@@ -31,7 +31,7 @@ assets.installWithSymlink = true
 # The cache:clear command should always be called with the --no-warmup option. Warmup should be done via the cache:warmup command.
 # https://github.com/symfony/symfony/blob/master/UPGRADE-3.3.md#frameworkbundle
 #
-# Meritoo <github@meritoo.pl>
+# Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
 # 2017-06-06
 #
 cache.clearWithWarmup = false
@@ -65,7 +65,7 @@ composer.validate = false
 
 # System directories
 #
-dir.data = ${project.basedir}/data
+dir.data = ${project.basedir}/.data
 dir.src = ${project.basedir}/src
 dir.tests = ${project.basedir}/tests
 
@@ -73,7 +73,7 @@ dir.tests = ${project.basedir}/tests
 # Build directories
 # --------------------------------------------------------------------------------
 
-dir.build = ${project.basedir}/build
+dir.build = ${project.basedir}/.build
 dir.reports = ${dir.build}/logs
 dir.reports.pdepend = ${dir.reports}/pdepend
 dir.reports.coverage = ${dir.reports}/phpunit_coverage
@@ -81,7 +81,7 @@ dir.reports.coverage = ${dir.reports}/phpunit_coverage
 # Disabled, because unnecessary right now
 # phpdocumentor/phpdocumentor cannot be installed via Composer
 #
-# Meritoo <github@meritoo.pl>
+# Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
 # 2017-02-22
 #
 #dir.docs = ${dir.build}/docs
@@ -91,15 +91,40 @@ dir.reports.coverage = ${dir.reports}/phpunit_coverage
 # Data directories
 # --------------------------------------------------------------------------------
 
+dir.data.tests = ${dir.data}/tests
 dir.data.temporary = ${dir.data}/tmp
 
 # --------------------------------------------------------------------------------
 # Testing
 # --------------------------------------------------------------------------------
 
-# Path of the framework used to run unit tests
+# Test database path
 #
-tests.framework.path = ./vendor/bin/phpunit --verbose --no-coverage --testdox
+tests.database = ${dir.data.temporary}/database.sqlite
+
+#
+# Disabled, because unnecessary right now
+# PHPUnit is installed and loaded by Composer
+#
+# Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
+# 2017-02-22
+#
+# Run PHPUnit using exec task instead of phpunitTask
+#phpunit.useExec = false
+
+#
+# Disabled, because unnecessary right now
+# We want generate code coverage always
+#
+# Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
+# 2017-02-22
+#
+# Collect coverage data during tests
+#phpunit.withCoverage = true
+
+# Path of the PHPUnit (https://phpunit.de)
+#
+phpUnit.path = ./vendor/bin/phpunit
 
 # Path of the PHP Coding Standards Fixer (http://cs.sensiolabs.org)
 #

.phing/tests.xml

@@ -4,19 +4,19 @@
     The AutoloaderTask is required to load binaries installed by Composer.
     The "autoloaderpath" attribute of this task is not required, because it's default value is: vendor/autoload.php.
 
-    Meritoo <github@meritoo.pl>
+    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
     2017-02-23
     -->
     <autoloader />
 
     <!-- Properties -->
     <if>
-        <available file="phing/properties" property="custom.properties.available"/>
+        <available file=".phing/properties" property="custom.properties.available"/>
         <then>
-            <property file="phing/properties"/>
+            <property file=".phing/properties" />
         </then>
         <else>
-            <property file="phing/properties.dist"/>
+            <property file=".phing/properties.dist" />
         </else>
     </if>
 
@@ -48,7 +48,7 @@
     a) phpdocumentor/phpdocumentor v2.9.0 requires symfony/validator ~2.2
     b) symfony/validator ~2.2 causes to remove symfony/symfony 3.*
 
-    Meritoo <github@meritoo.pl>
+    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
     2017-02-22
     -->
 
@@ -73,7 +73,7 @@
     a) phpdocumentor/phpdocumentor v2.9.0 requires symfony/validator ~2.2
     b) symfony/validator ~2.2 causes to remove symfony/symfony 3.*
 
-    Meritoo <github@meritoo.pl>
+    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
     2017-02-22
     -->
     <!--<target name="build:doc"-->
@@ -87,7 +87,7 @@
 
     <!-- Test target -->
     <target name="build:test"
-            depends="test:unit"
+            depends="test:phpunit"
             description="Executes all tests" />
 
     <!-- Project build clean -->
@@ -103,8 +103,8 @@
         <!--<mkdir dir="${dir.docs}" />-->
         <!--<mkdir dir="${dir.docs.phpdoc2}" />-->
         <mkdir dir="${dir.reports}" />
+        <mkdir dir="${dir.reports.coverage}" />
         <mkdir dir="${dir.reports.pdepend}" />
-        <mkdir dir="${dir.reports.coverage}"/>
     </target>
 
     <!-- PHPDocumentor2 API documentation target -->
@@ -113,7 +113,7 @@
     a) phpdocumentor/phpdocumentor v2.9.0 requires symfony/validator ~2.2
     b) symfony/validator ~2.2 causes to remove symfony/symfony 3.*
 
-    Meritoo <github@meritoo.pl>
+    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
     2017-02-22
 
     <target name="doc:phpdoc2" description="Generates API documentations">
@@ -142,7 +142,7 @@
     via Composer the Symfony2 standard is not included / available in this package. In this case the PHP Coding
     Standards Fixer (http://cs.sensiolabs.org) is used.
 
-    Meritoo <github@meritoo.pl>
+    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
     2017-02-22
     -->
     <target name="check:cs" description="Checks coding standard">
@@ -166,7 +166,7 @@
         <!--
         Previous / old version
 
-        Meritoo <github@meritoo.pl>
+        Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
         2017-02-22
 
         <exec command="phpcpd \-\-log-pmd=${dir.reports}/pmd-cpd.xml ${dir.src}" />
@@ -204,7 +204,7 @@
         <!--
         Previous / old version
 
-        Meritoo <github@meritoo.pl>
+        Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
         2017-02-22
 
         <exec command="phploc \-\-log-csv=${dir.reports}/phploc.csv ${dir.src}" />
@@ -212,9 +212,94 @@
     </target>
 
     <!-- Unit tests -->
-    <target name="test:unit" description="Executes unit tests">
+    <target name="test:phpunit" description="Executes PHPUnit tests">
+        <!-- Check test database -->
+        <if>
+            <not>
+                <available file="${dir.data.tests}" type="dir" property="dir.data.tests.available" />
+            </not>
+            <then>
+                <mkdir dir="${dir.data.tests}" />
+            </then>
+        </if>
+        <if>
+            <not>
+                <available file="${tests.database}" property="tests.database.available" />
+            </not>
+            <then>
+                <touch file="${tests.database}" />
+            </then>
+        </if>
         <echo msg="Running unit tests..." />
-        <exec command="${tests.framework.path}" passthru="true"/>
+        <coverage-setup database="${dir.reports.coverage}/coverage.db">
+            <fileset refid="sourcecode" />
+        </coverage-setup>
+        <exec command="${phpUnit.path} --verbose --configuration ${project.basedir}/phpunit.xml.dist" passthru="true" />
+        <!--
+        I have to use ExecTask to run PHPUnit instead of PHPUnitTask, because tests are not running if PHPUnitTask is
+        used (don't know why):
+        Total tests run: 0, Failures: 0, Errors: 0, Incomplete: 0, Skipped: 0
+
+        Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
+        2017-02-23
+
+        <phpunit configuration="${project.basedir}/phpunit.xml.dist" printsummary="true">
+            <formatter type="xml" todir="${dir.reports}" outfile="phpunit.xml" />
+            <formatter type="plain" todir="${dir.reports}" outfile="phpunit.txt" />
+            <formatter type="clover" todir="${dir.reports.coverage}" />
+            <formatter type="summary" todir="${dir.reports}" outfile="phpunit_summary.txt" />
+        </phpunit>
+        -->
+
+        <!--
+        Previous / old version
+
+        Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
+        2017-02-22
+
+        <if>
+            <istrue value="${phpunit.useExec}" />
+            <then>
+                <if>
+                    <istrue value="${phpunit.withCoverage}" />
+                    <then>
+                        <echo message="Running tests with code coverage..." />
+                        <exec command="phpunit \-\-log-junit ${dir.reports}/phpunit.xml \-\-coverage-clover ${dir.reports.coverage}/clover-coverage.xml \-\-coverage-crap4j ${dir.reports.coverage}/crap4j-coverage.xml \-\-coverage-html ${dir.reports.coverage}/ -c ${project.basedir} \-\-colors" passthru="true" checkreturn="true" />
+                    </then>
+                    <else>
+                        <echo message="Running tests without code coverage..." />
+                        <exec command="phpunit \-\-log-junit ${dir.reports}/phpunit.xml -c ${project.basedir} \-\-colors" passthru="true" checkreturn="true" />
+                    </else>
+                </if>
+            </then>
+            <else>
+                <if>
+                    <istrue value="${phpunit.withCoverage}" />
+                    <then>
+                        <echo message="Running tests with code coverage..." />
+                        <coverage-setup database="${dir.reports.coverage}/coverage.db">
+                            <fileset refid="sourcecode" />
+                        </coverage-setup>
+                        <phpunit printsummary="true" codecoverage="true">
+                            <formatter type="xml" todir="${dir.reports}" outfile="phpunit.xml" />
+                            <formatter type="plain" todir="${dir.reports}" outfile="phpunit.txt" />
+                            <formatter type="clover" todir="${dir.reports.coverage}" />
+                            <formatter type="summary" todir="${dir.reports}" outfile="phpunit_summary.txt" />
+                        </phpunit>
+                    </then>
+                    <else>
+                        <echo message="Running tests without code coverage..." />
+                        <phpunit printsummary="true">
+                            <formatter todir="${dir.reports}" type="xml" outfile="phpunit.xml" />
+                            <batchtest>
+                                <fileset refid="tests" />
+                            </batchtest>
+                        </phpunit>
+                    </else>
+                </if>
+            </else>
+        </if>
+        -->
     </target>
 
     <!-- Checkout and finalization -->

.php_cs.dist

@@ -1,6 +1,13 @@
 <?php
 
 $finder = PhpCsFixer\Finder::create()
+    /*
+     * Do not verify:
+     * - all DependencyInjection/Configuration classes: the Configuration.php files
+     * - autoloader from /app directory: autoload.php
+     */
+    ->notPath('/DependencyInjection\/Configuration\.php/')
+    ->notPath('/autoload\.php/')
     ->in([
         __DIR__ . '/src',
         __DIR__ . '/tests',
@@ -9,19 +16,15 @@ $finder = PhpCsFixer\Finder::create()
 return PhpCsFixer\Config::create()
     ->setRules([
         '@Symfony'               => true,
+        'phpdoc_summary'         => false,
+        'phpdoc_separation'      => false,
+        'phpdoc_align'           => false,
+        'cast_spaces'            => false,
         'binary_operator_spaces' => [
             'align_double_arrow' => true,
         ],
-        'blank_line_before_return'            => false,
-        'cast_spaces'                         => false,
         'concat_space'           => [
             'spacing' => 'one',
         ],
-        'phpdoc_add_missing_param_annotation' => true,
-        'phpdoc_align'                        => false,
-        'phpdoc_order'                        => true,
-        'phpdoc_separation'                   => false,
-        'phpdoc_summary'                      => false,
-        'trim_array_spaces'                   => false,
     ])
     ->setFinder($finder);

.styleci.yml

@@ -0,0 +1,8 @@
+preset: symfony
+
+disabled:
+    - phpdoc_annotation_without_dot
+    - cast_spaces
+    - concat_without_spaces
+    - blank_line_before_return
+    - trim_array_spaces

.travis.yml

@@ -4,14 +4,9 @@ php:
   - 5.6
   - 7.0
   - 7.1
-  - 7.2
-
-before_install:
-  - sudo locale-gen de_DE.UTF-8 es_ES.UTF-8 en_GB.UTF-8 en_US.UTF-8 fr_FR.UTF-8 it_IT.UTF-8 pl_PL.UTF-8 ru_RU.UTF-8
-  - composer global require hirak/prestissimo
 
 install:
-- travis_wait 30 composer install
+  - composer install
 
 script:
   - php ./vendor/bin/phpunit

CHANGELOG.md

@@ -1,43 +0,0 @@
-# Meritoo Common Library
-
-Common and useful classes, methods, exceptions etc.
-
-# 0.1.2
-
-1. Documentation > Value Objects
-2. Docker > improve performance
-3. Utilities > Reflection > setPropertyValue() method > sets value of given property
-
-# 0.1.1
-
-1. TravisCI > run using PHP 7.2 too
-2. ValueObject > class Version > represents version of software
-3. Move version of this package to `VERSION` file (from `composer.json` file)
-
-# 0.1.0
-
-1. Composer > support/require PHP 5.6+ (instead of 5.5.9+)
-2. Docker > rename `php-cli` service to `php`
-3. Exceptions > create instance of exception using static `create()` method (instead of constructor)
-4. Documentation > Exceptions
-
-# 0.0.21
-
-1. Composer > require ext-pcre
-2. Arrays > minor refactoring
-3. Update @author and @copyright in classes' descriptions
-
-# 0.0.20
-
-1. Collection > add() method > treat empty string as not provided index (same as null)
-
-# 0.0.19
-
-1. Add this changelog
-2. Reorganize documentation & update [Readme](README.md)
-3. Docker: use project-related binaries globally
-4. StyleCI & PHP Coding Standards Fixer: update configuration
-5. Documentation > Docker > add paragraph for PHP Coding Standards Fixer
-6. Coding standard > fix automatically
-7. StyleCI configuration > fix bug "The provided fixer 'binary_operator_spaces' cannot be enabled again because it was already enabled"
-8. StyleCI > disable & remove

README.md

@@ -1,29 +1,114 @@
 # Meritoo Common Library
-
 Common and useful classes, methods, exceptions etc.
 
-[![Travis](https://img.shields.io/travis/rust-lang/rust.svg?style=flat-square)](https://travis-ci.org/meritoo/common-library) [![Packagist](https://img.shields.io/packagist/v/meritoo/common-library.svg?style=flat-square)](https://packagist.org/packages/meritoo/common-library) [![license](https://img.shields.io/github/license/meritoo/common-library.svg?style=flat-square)](https://github.com/meritoo/common-library) [![GitHub commits](https://img.shields.io/github/commits-since/meritoo/common-library/0.0.1.svg?style=flat-square)](https://github.com/meritoo/common-library) [![Coverage Status](https://coveralls.io/repos/github/meritoo/common-library/badge.svg?branch=master)](https://coveralls.io/github/meritoo/common-library?branch=master)
+[![Travis](https://img.shields.io/travis/rust-lang/rust.svg?style=flat-square)](https://travis-ci.org/meritoo/common-library) [![Packagist](https://img.shields.io/packagist/v/meritoo/common-library.svg?style=flat-square)](https://packagist.org/packages/meritoo/common-library) [![StyleCI](https://styleci.io/repos/101790028/shield?branch=master)](https://styleci.io/repos/101790028) [![license](https://img.shields.io/github/license/meritoo/common-library.svg?style=flat-square)](https://github.com/meritoo/common-library) [![GitHub commits](https://img.shields.io/github/commits-since/meritoo/common-library/0.0.1.svg?style=flat-square)](https://github.com/meritoo/common-library) [![Coverage Status](https://coveralls.io/repos/github/meritoo/common-library/badge.svg?branch=master)](https://coveralls.io/github/meritoo/common-library?branch=master)
 
-# Installation
+## Installation
 
 Run [Composer](https://getcomposer.org) to install this package in your project:
 
 ```bash
-composer require meritoo/common-library
+$ composer require wiosna/common-library
 ```
 
-> [How to install Composer?](https://getcomposer.org/download)
+> How to install Composer: https://getcomposer.org/download
 
-# Usage
+## Rebuilding project and tests running
 
-1. [Base test case (with common methods and data providers)](docs/Base-test-case.md)
+```bash
-2. [Collection of elements](docs/Collection-of-elements.md)
+$ docker-compose up -d
-3. [Exceptions](docs/Static-methods.md)
+$ docker-compose exec php-cli phing
-4. [Static methods](docs/Static-methods.md)
+```
-5. [Value Objects](docs/Value-Objects.md)
 
-# Development
+> What is Docker? https://www.docker.com/what-docker
 
-More information [you can find here](docs/Development.md)
+## Static methods
+
+This package contains a lot of class with static methods, so usage is not so complicated. Just run the static method who would you like to use. Example:
+
+```php
+use Meritoo\Common\Utilities\Arrays;
+
+$firstElement = Arrays::getFirstElement(['lorem', 'ipsum']);
+var_dump($firstElement); // string(5) "lorem"
+```
+
+## Base test case with common methods and data providers
+
+Located here: `Meritoo\Common\Test\Base\BaseTestCase`. Just extend the `BaseTestCase` class and use it like in `Meritoo\Common\Test\Utilities\DateTest` class:
+
+```php
+class DateTest extends BaseTestCase
+{
+    /**
+     * @param mixed $value Empty value, e.g. ""
+     * @dataProvider provideEmptyValue
+     */
+    public function testGetDateTimeEmptyValue($value)
+    {
+        self::assertFalse(Date::getDateTime($value));
+    }
+
+	(...)
+}
+```
+
+or in `Meritoo\Common\Test\Utilities\MimeTypesTest` class:
+
+```php
+class MimeTypesTest extends BaseTestCase
+{
+	(...)
+
+    /**
+     * @param bool $mimeType The mime type, e.g. "video/mpeg"
+     * @dataProvider provideBooleanValue
+     */
+    public function testGetExtensionBooleanMimeType($mimeType)
+    {
+        self::assertEquals('', MimeTypes::getExtension($mimeType));
+    }
+
+	(...)
+}
+```
+
+## Collection of elements
+
+Located here: `Meritoo\Common\Collection\Collection`. It's a set of some elements, e.g. objects. It's iterable and countable. Provides very useful methods. Some of them:
+- `getFirst()` - returns the first element in the collection
+- `getLast()` - returns the last element in the collection
+- `isEmpty()` - returns information if collection is empty
+- `add($element, $index = null)` - adds given element (at the end of collection)
+- `addMultiple($elements, $useIndexes = false)` - adds given elements (at the end of collection)
+- `prepend($element)` - prepends given element (adds given element at the beginning of collection)
+- `remove($element)` - removes given element
+
+Examples of usage below.
+
+#### An empty collection
+
+```php
+use Meritoo\Common\Collection\Collection;
+
+$emptyCollection = new Collection();
+var_dump($emptyCollection->isEmpty()); // bool(true)
+```
+
+#### Simple collection
+
+```php
+use Meritoo\Common\Collection\Collection;
+
+$elements = [
+    'lorem',
+    'ipsum',
+    123 => 'dolor',
+    345 => 'sit',
+];
+
+$simpleCollection = new Collection($elements);
+var_dump($simpleCollection->has('dolor')); // bool(true)
+```
 
 Enjoy!

VERSION

@@ -1 +0,0 @@
-0.1.2

build.xml

@@ -2,12 +2,12 @@
 <project name="Meritoo Package" basedir="." default="build:main" phingVersion="2.14.0">
     <!-- Properties -->
     <if>
-        <available file="phing/properties" property="custom.properties.available"/>
+        <available file=".phing/properties" property="custom.properties.available"/>
         <then>
-            <property file="phing/properties"/>
+            <property file=".phing/properties" />
         </then>
         <else>
-            <property file="phing/properties.dist"/>
+            <property file=".phing/properties.dist" />
         </else>
     </if>
 
@@ -18,24 +18,24 @@
 
     <!-- Build app -->
     <target name="build:app" description="Prepares app to build and tests">
-        <phing phingfile="phing/app.xml" haltonfailure="true"/>
+        <phing phingfile=".phing/app.xml" haltonfailure="true" />
     </target>
 
     <!-- Build tests -->
     <target name="build:tests" description="Runs all tests, checks and creates docs">
-        <phing phingfile="phing/tests.xml" haltonfailure="true"/>
+        <phing phingfile=".phing/tests.xml" haltonfailure="true" />
 
         <!--
         Conditional running of tests.
         Disabled, because not required.
 
-        Meritoo <github@meritoo.pl>
+        Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
         2017-02-22
 
         <if>
             <equals arg1="${env}" arg2="test" />
             <then>
-                <phing phingfile="phing/tests.xml" haltonfailure="true" />
+                <phing phingfile=".phing/tests.xml" haltonfailure="true" />
             </then>
             <else>
                 <echo message="[Skipped] Running tests, checks and creating docs, because it's a not 'test' environment..." />

composer.json

@@ -1,7 +1,8 @@
 {
-    "name": "meritoo/common-library",
+    "name": "wiosna-dev/common-library",
     "description": "Useful classes, methods, extensions etc.",
     "license": "MIT",
+    "version": "0.0.17",
     "authors": [
         {
             "name": "Meritoo.pl",
@@ -10,10 +11,10 @@
         }
     ],
     "require": {
-        "php": ">=5.6",
+        "php": ">=5.5.9",
-        "ext-pcre": "*",
         "doctrine/orm": "^2.5",
-        "gedmo/doctrine-extensions": "^2.4"
+        "gedmo/doctrine-extensions": "^2.4",
+        "symfony/http-foundation": "^3.3"
     },
     "require-dev": {
         "friendsofphp/php-cs-fixer": "^2.2",

docker-compose.yml

@@ -1,20 +1,15 @@
 version: '3'
 
 services:
-    php:
+    php-cli:
-        image: ${DOCKER_CONTAINER_OWNER}/${DOCKER_CONTAINER_PROJECT}-php
+        image: meritoo/common-library
-        container_name: ${DOCKER_CONTAINER_OWNER}-${DOCKER_CONTAINER_PROJECT}-php
+        container_name: meritoo-common-library
+        working_dir: /project
         entrypoint: php
         command: -S 0.0.0.0:9999
         build:
-            context: ./docker/config
+            context: ./.docker/config
             args:
-            - TIMEZONE=${TIMEZONE}
+                - TIMEZONE=$TIMEZONE
         volumes:
-        - .:/project:cached
+            - .:/project
-    composer:
-        image: ${DOCKER_CONTAINER_OWNER}/${DOCKER_CONTAINER_PROJECT}-php
-        container_name: ${DOCKER_CONTAINER_OWNER}-${DOCKER_CONTAINER_PROJECT}-composer
-        entrypoint: composer
-        volumes:
-        - .:/project:cached

docker/config/Dockerfile

@@ -1,131 +0,0 @@
-FROM php:5.6-cli
-MAINTAINER Meritoo <github@meritoo.pl>
-
-#
-# Tools & libraries
-#
-RUN apt-get update \
-    && apt-get install -y --no-install-recommends \
-        vim \
-        git \
-        unzip \
-        libicu-dev \
-        locales \
-    && apt-get clean \
-    && rm -rf \
-        /var/lib/apt/lists/* \
-        /tmp/* \
-        /var/tmp/*
-
-#
-# Generating locales:
-# - de_DE
-# - es_ES
-# - en_GB
-# - en_US
-# - fr_FR
-# - it_IT
-# - pl_PL
-# - ru_RU
-#
-RUN sed -i 's/^# de_DE/de_DE/g; \
-            s/^# es_ES/es_ES/g; \
-            s/^# en_GB/en_GB/g; \
-            s/^# en_US/en_US/g; \
-            s/^# fr_FR/fr_FR/g; \
-            s/^# it_IT/it_IT/g; \
-            s/^# pl_PL/pl_PL/g; \
-            s/^# ru_RU/ru_RU/g;' /etc/locale.gen \
-    && locale-gen
-
-#
-# Set default language
-#
-# Required to avoid problem with using strange language by error messages.
-# Example: "chmod(): Aucun fichier ou dossier de ce type"
-#
-ENV LANGUAGE=en_US.UTF-8
-
-#
-# PHP extensions
-#
-RUN docker-php-ext-install \
-    intl \
-    mbstring
-
-#
-# PHP extensions (PECL):
-# - Xdebug
-#
-RUN pecl install \
-        xdebug-2.5.5 \
-    && docker-php-ext-enable \
-        xdebug
-
-COPY xdebug.ini /usr/local/etc/php/conf.d/docker-php-ext-xdebug.ini
-
-#
-# PHP configuration:
-# - default configuration
-# - timezone
-#
-COPY php.ini /usr/local/etc/php/php.ini
-ARG TIMEZONE
-RUN ln -snf /usr/share/zoneinfo/${TIMEZONE} /etc/localtime \
-    && echo ${TIMEZONE} > /etc/timezone \
-    && printf '[PHP]\ndate.timezone = "%s"\n' ${TIMEZONE} > /usr/local/etc/php/conf.d/tzone.ini \
-    && "date"
-#RUN echo "\n""date.timezone = $TIMEZONE""\n" >> /usr/local/etc/php/php.ini
-
-#
-# Phing
-#
-RUN pear channel-discover pear.phing.info \
-    && pear install [--alldeps] phing/phing
-
-#
-# Composer - environment variables:
-# - disable warning about running commands as root/super user
-# - disable automatic clearing of sudo sessions
-#
-# More:
-# https://getcomposer.org/doc/03-cli.md#composer-allow-superuser
-#
-ENV COMPOSER_ALLOW_SUPERUSER 1
-
-#
-# Composer + https://packagist.org/packages/hirak/prestissimo package
-#
-RUN php -r "copy('https://getcomposer.org/installer', 'composer-setup.php');" \
-    && php -r "if (hash_file('SHA384', 'composer-setup.php') === \
-        '544e09ee996cdf60ece3804abc52599c22b1f40f4323403c44d44fdfdd586475ca9813a858088ffbc1f233e9b180f061') { echo \
-        'Installer verified'; } else { echo 'Installer corrupt'; unlink('composer-setup.php'); } echo PHP_EOL;" \
-    && php composer-setup.php --install-dir=/usr/local/bin --filename=composer \
-    && php -r "unlink('composer-setup.php');" \
-    && composer global require \
-        --no-plugins \
-        --no-scripts \
-        --no-progress \
-        --no-suggest \
-        --no-interaction \
-        --prefer-dist \
-        --optimize-autoloader \
-        --classmap-authoritative \
-        hirak/prestissimo \
-    && rm -rf ~/.composer/cache/* \
-    && composer clear-cache \
-    && composer --version
-
-#
-# Bash
-#
-RUN sed -i 's/^# export/export/g; \
-            s/^# alias/alias/g;' ~/.bashrc \
-    && echo 'COLUMNS=200'"\n" >> ~/.bashrc
-
-#
-# Use project-related binaries globally
-#
-ENV PATH="/project/vendor/bin:${PATH}"
-
-WORKDIR /project

docs/Base-test-case.md

@@ -1,53 +0,0 @@
-# Meritoo Common Library
-
-Common and useful classes, methods, exceptions etc.
-
-# Base test case (with common methods and data providers)
-
-Located here: `Meritoo\Common\Test\Base\BaseTestCase`. Just extend the `BaseTestCase` class and use it like in `Meritoo\Common\Test\Utilities\DateTest` class:
-
-```php
-class DateTest extends BaseTestCase
-{
-    /**
-     * @param mixed $value Empty value, e.g. ""
-     * @dataProvider provideEmptyValue
-     */
-    public function testGetDateTimeEmptyValue($value)
-    {
-        self::assertFalse(Date::getDateTime($value));
-    }
-
-	(...)
-}
-```
-
-or in `Meritoo\Common\Test\Utilities\MimeTypesTest` class:
-
-```php
-class MimeTypesTest extends BaseTestCase
-{
-	(...)
-
-    /**
-     * @param bool $mimeType The mime type, e.g. "video/mpeg"
-     * @dataProvider provideBooleanValue
-     */
-    public function testGetExtensionBooleanMimeType($mimeType)
-    {
-        self::assertEquals('', MimeTypes::getExtension($mimeType));
-    }
-
-	(...)
-}
-```
-
-# More
-
-1. [**Base test case (with common methods and data providers)**](Base-test-case.md)
-2. [Collection of elements](Collection-of-elements.md)
-3. [Exceptions](Exceptions.md)
-4. [Static methods](Static-methods.md)
-5. [Value Objects](Value-Objects.md)
-
-[‹ Back to `Readme`](../README.md)

docs/Collection-of-elements.md

@@ -1,51 +0,0 @@
-# Meritoo Common Library
-
-Common and useful classes, methods, exceptions etc.
-
-# Collection of elements
-
-Located here: `Meritoo\Common\Collection\Collection`. It's a set of some elements, e.g. objects. It's iterable and countable. Provides very useful methods. Some of them:
-- `getFirst()` - returns the first element in the collection
-- `getLast()` - returns the last element in the collection
-- `isEmpty()` - returns information if collection is empty
-- `add($element, $index = null)` - adds given element (at the end of collection)
-- `addMultiple($elements, $useIndexes = false)` - adds given elements (at the end of collection)
-- `prepend($element)` - prepends given element (adds given element at the beginning of collection)
-- `remove($element)` - removes given element
-
-Examples of usage below.
-
-### An empty collection
-
-```php
-use Meritoo\Common\Collection\Collection;
-
-$emptyCollection = new Collection();
-var_dump($emptyCollection->isEmpty()); // bool(true)
-```
-
-### Simple collection
-
-```php
-use Meritoo\Common\Collection\Collection;
-
-$elements = [
-    'lorem',
-    'ipsum',
-    123 => 'dolor',
-    345 => 'sit',
-];
-
-$simpleCollection = new Collection($elements);
-var_dump($simpleCollection->has('dolor')); // bool(true)
-```
-
-# More
-
-1. [Base test case (with common methods and data providers)](Base-test-case.md)
-2. [**Collection of elements**](Collection-of-elements.md)
-3. [Exceptions](Exceptions.md)
-4. [Static methods](Static-methods.md)
-5. [Value Objects](Value-Objects.md)
-
-[‹ Back to `Readme`](../README.md)

docs/Development.md

@@ -1,119 +0,0 @@
-# Meritoo Common Library
-
-Development-related information
-
-# Requirements
-
-* [Docker](https://www.docker.com)
-* Your favourite IDE :)
-
-# Getting started
-
-1. Build, create and start Docker's containers by running command:
-
-    ```bash
-    docker-compose up -d
-    ```
-
-2. Install packages by running command:
-
-    ```bash
-    docker-compose run composer install
-    ```
-
-> [What is Docker?](https://www.docker.com/what-docker)
-
-# Composer
-
-Available as `composer` service. You can run any Composer's command using the `composer` service:
-
-```bash
-docker-compose run composer <command>
-```
-
-Examples below.
-
-##### Install packages
-
-```bash
-docker-compose run composer install
-```
-
-##### Update packages
-
-```bash
-docker-compose run composer update
-```
-
-##### Add package
-
-```bash
-docker-compose run composer require <vendor>/<package>
-```
-
-##### Remove package
-
-```bash
-docker-compose run composer remove <vendor>/<package>
-```
-
-# Coding Standards Fixer
-
-Fix coding standard by running command:
-
-```bash
-docker-compose exec php php-cs-fixer fix
-```
-
-Omit cache and run the Fixer from scratch by running command:
-
-```bash
-docker-compose exec php rm .php_cs.cache && docker-compose exec php php-cs-fixer fix
-```
-
-> [Want more?](https://cs.sensiolabs.org)
-
-# Tests
-
-### Prerequisites
-
-Install required packages by running command: `docker-compose run composer install`.
-
-### Running tests
-
-#### Simply & quick, without code coverage
-
-Tests are running using Docker and `php` service defined in `docker-compose.yml`. Example:
-
-```bash
-docker-compose exec php phpunit --no-coverage
-```
-
-You can also run them in container. In this case you have to run 2 commands:
-1. Enter container:
-
-    ```bash
-	docker-compose exec php bash
-    ```
-
-2. Run tests:
-
-    ```bash
-    phpunit --no-coverage
-    ```
-
-#### With code coverage
-
-```bash
-docker-compose exec php phpunit
-```
-
-# Other
-
-Rebuild project and run tests by running command:
-
-```bash
-docker-compose exec php phing
-```
-
-[&lsaquo; Back to `Readme`](../README.md)

docs/Exceptions.md

@@ -1,62 +0,0 @@
-# Meritoo Common Library
-
-Common and useful classes, methods, exceptions etc.
-
-# Exceptions
-
-### Create instance of exception
-
-This package contains a lot of exceptions. Each of them contains static method `create()` with proper arguments that is used to create instance of the exception. Example:
-
-```php
-use Meritoo\Common\Exception\Bundle\IncorrectBundleNameException;
-throw IncorrectBundleNameException::create('RisusIpsum');
-```
-
-### Base exception for unknown type of something
-
-##### Short description
-
-It's a `Meritoo\Common\Exception\Base\UnknownTypeException` class. Related to `Meritoo\Common\Type\Base\BaseType` class that represents type of something, e.g. type of button, order.
-
-##### Usage
-
-You can extend `Meritoo\Common\Exception\Base\UnknownTypeException` class and create your own static method, e.g. `createException()`, which will be used create instance of the exception. Inside the `createException()` method you can call `parent::create()` method.
-
-##### Example
-
-```php
-<?php
-
-namespace Your\Package\Exception\Type;
-
-use Meritoo\Common\Exception\Base\UnknownTypeException;
-use Your\Package\Type\SimpleType;
-
-class UnknownSimpleTypeException extends UnknownTypeException
-{
-    /**
-     * Creates exception
-     *
-     * @param string $unknownType Unknown and simple type
-     * @return UnknownSimpleTypeException
-     */
-    public static function createException($unknownType)
-    {
-        /* @var UnknownSimpleTypeException $exception */
-        $exception = parent::create($unknownType, new SimpleType(), 'my simple type of something');
-
-        return $exception;
-    }
-}
-```
-
-# More
-
-1. [Base test case (with common methods and data providers)](Base-test-case.md)
-2. [Collection of elements](Collection-of-elements.md)
-3. [**Exceptions**](Exceptions.md)
-4. [Static methods](Static-methods.md)
-5. [Value Objects](Value-Objects.md)
-
-[&lsaquo; Back to `Readme`](../README.md)

docs/Static-methods.md

@@ -1,24 +0,0 @@
-# Meritoo Common Library
-
-Common and useful classes, methods, exceptions etc.
-
-# Static methods
-
-This package contains a lot of class with static methods, so usage is not so complicated. Just run the static method who would you like to use. Example:
-
-```php
-use Meritoo\Common\Utilities\Arrays;
-
-$firstElement = Arrays::getFirstElement(['lorem', 'ipsum']);
-var_dump($firstElement); // string(5) "lorem"
-```
-
-# More
-
-1. [Base test case (with common methods and data providers)](Base-test-case.md)
-2. [Collection of elements](Collection-of-elements.md)
-3. [Exceptions](Exceptions.md)
-4. [**Static methods**](Static-methods.md)
-5. [Value Objects](Value-Objects.md)
-
-[‹ Back to `Readme`](../README.md)

docs/Value-Objects.md

@@ -1,53 +0,0 @@
-# Meritoo Common Library
-
-Common and useful classes, methods, exceptions etc.
-
-# Value Objects
-
-Located in `Meritoo\Common\ValueObject` namespace.
-
-### Version
-
-##### Namespace
-
-`Meritoo\Common\ValueObject\Version`
-
-##### Info
-
-Represents version of software. Contains 3 properties:
-1. `$majorPart` - the "major" part of version
-2. `$minorPart` - the "minor" part of version
-3. `$patchPart` - the "patch" part of version
-
-##### New instance
-
-New instance can be created using:
-
-1. Constructor:
-
-	```php
-    new Version(1, 0, 2);
-    ```
-
-2. Static methods:
-	1. `fromArray()` - creates new instance using given version as array
-
-	```php
-	Version::fromArray([1, 0, 2]);
-	```
-
-	2. `fromString()` - creates new instance using given version as string:
-
-	```php
-    Version::fromString('1.0.2');
-    ```
-
-# More
-
-1. [Base test case (with common methods and data providers)](Base-test-case.md)
-2. [Collection of elements](Collection-of-elements.md)
-3. [Exceptions](Exceptions.md)
-4. [Static methods](Static-methods.md)
-5. [**Value Objects**](Value-Objects.md)
-
-[‹ Back to `Readme`](../README.md)

phpunit.xml.dist

@@ -1,34 +1,16 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
-<phpunit
+<phpunit backupGlobals="false"
-        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-        xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/4.8/phpunit.xsd"
-        backupGlobals="true"
          backupStaticAttributes="false"
-        bootstrap="./vendor/autoload.php"
-        cacheTokens="false"
          colors="true"
          convertErrorsToExceptions="true"
          convertNoticesToExceptions="true"
          convertWarningsToExceptions="true"
-        forceCoversAnnotation="false"
-        mapTestClassNameToCoveredClassName="false"
          processIsolation="false"
-        stopOnError="false"
          stopOnFailure="false"
-        stopOnIncomplete="false"
+         syntaxCheck="true"
-        stopOnSkipped="false"
+         bootstrap="./vendor/autoload.php"
-        stopOnRisky="false"
-        testSuiteLoaderClass="PHPUnit_Runner_StandardTestSuiteLoader"
-        timeoutForSmallTests="1"
-        timeoutForMediumTests="10"
-        timeoutForLargeTests="60"
-        verbose="true"
 >
-    <php>
-        <ini name="error_reporting" value="-1"/>
-    </php>
-
     <testsuites>
         <testsuite name="Meritoo Package - Main Test Suite">
             <directory>./tests/</directory>
@@ -48,6 +30,6 @@
     </groups>
 
     <logging>
-        <log type="coverage-html" target="./build/logs/phpunit_coverage/html"/>
+        <log type="coverage-html" target="./.build/logs/phpunit_coverage/html" />
     </logging>
 </phpunit>

src/Collection/Collection.php

@@ -18,8 +18,8 @@ use Meritoo\Common\Utilities\Arrays;
  * Collection of elements.
  * It's a set of some elements, e.g. objects.
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Collection implements Countable, ArrayAccess, IteratorAggregate
 {
@@ -109,7 +109,7 @@ class Collection implements Countable, ArrayAccess, IteratorAggregate
      */
     public function add($element, $index = null)
     {
-        if (null === $index || '' === $index) {
+        if (null === $index) {
             $this->elements[] = $element;
         } else {
             $this->elements[$index] = $element;

src/Exception/Base/UnknownTypeException.php

@@ -15,28 +15,27 @@ use Meritoo\Common\Utilities\Arrays;
 /**
  * An exception used while type of something is unknown
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 abstract class UnknownTypeException extends Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      *
      * @param string|int $unknownType  The unknown type of something (value of constant)
      * @param BaseType   $typeInstance An instance of class that contains type of the something
      * @param string     $typeName     Name of the something
-     * @return UnknownTypeException
      */
-    public static function create($unknownType, BaseType $typeInstance, $typeName)
+    public function __construct($unknownType, BaseType $typeInstance, $typeName)
     {
+        $allTypes = $typeInstance->getAll();
+        $types = Arrays::values2string($allTypes, '', ', ');
+
         $template = 'The \'%s\' type of %s is unknown. Probably doesn\'t exist or there is a typo. You should use one'
             . ' of these types: %s.';
 
-        $allTypes = $typeInstance->getAll();
-        $types = Arrays::values2string($allTypes, '', ', ');
         $message = sprintf(sprintf($template, $unknownType, $typeName, $types));
-
+        parent::__construct($message);
-        return new static($message);
     }
 }

src/Exception/Bundle/IncorrectBundleNameException.php

@@ -1,36 +0,0 @@
-<?php
-
-/**
- * (c) Meritoo.pl, http://www.meritoo.pl
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
-namespace Meritoo\Common\Exception\Bundle;
-
-use Exception;
-
-/**
- * An exception used while name of bundle is incorrect
- *
- * @author    Meritoo <github@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
- */
-class IncorrectBundleNameException extends Exception
-{
-    /**
-     * Creates exception
-     *
-     * @param string $bundleName Incorrect name of bundle
-     * @return IncorrectBundleNameException
-     */
-    public static function create($bundleName)
-    {
-        $template = 'Name of bundle \'%s\' is incorrect. It should start with big letter and end with "Bundle". Is'
-            . ' there everything ok?';
-
-        $message = sprintf($template, $bundleName);
-
-        return new static($message);
-    }
-}

src/Exception/Date/UnknownDatePartTypeException.php

@@ -0,0 +1,32 @@
+<?php
+
+/**
+ * (c) Meritoo.pl, http://www.meritoo.pl
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Meritoo\Common\Exception\Date;
+
+use Meritoo\Common\Exception\Base\UnknownTypeException;
+use Meritoo\Common\Type\DatePartType;
+
+/**
+ * An exception used while type of date part, e.g. "year", is unknown
+ *
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
+ * @copyright Meritoo.pl
+ */
+class UnknownDatePartTypeException extends UnknownTypeException
+{
+    /**
+     * Class constructor
+     *
+     * @param string $unknownDatePart Type of date part, e.g. "year". One of DatePartType class constants.
+     * @param string $value           Incorrect value
+     */
+    public function __construct($unknownDatePart, $value)
+    {
+        parent::__construct($unknownDatePart, new DatePartType(), sprintf('date part (with value %s)', $value));
+    }
+}

src/Exception/File/EmptyFileException.php

@@ -11,22 +11,21 @@ namespace Meritoo\Common\Exception\File;
 /**
  * An exception used while file with given path is empty (has no content)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class EmptyFileException extends \Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      *
      * @param string $emptyFilePath Path of the empty file
-     * @return EmptyFileException
      */
-    public static function create($emptyFilePath)
+    public function __construct($emptyFilePath)
     {
         $template = 'File with path \'%s\' is empty (has no content). Did you provide path of proper file?';
         $message = sprintf($template, $emptyFilePath);
 
-        return new static($message);
+        parent::__construct($message);
     }
 }

src/Exception/File/EmptyFilePathException.php

@@ -11,16 +11,16 @@ namespace Meritoo\Common\Exception\File;
 /**
  * An exception used while path of given file is empty
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class EmptyFilePathException extends \Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      */
-    public static function create()
+    public function __construct()
     {
-        return new static('Path of the file is empty. Did you provide path of proper file?');
+        parent::__construct('Path of the file is empty. Did you provide path of proper file?');
     }
 }

src/Exception/File/NotExistingFileException.php

@@ -11,22 +11,21 @@ namespace Meritoo\Common\Exception\File;
 /**
  * An exception used while file with given path does not exist
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class NotExistingFileException extends \Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      *
      * @param string $notExistingFilePath Path of not existing (or not readable) file
-     * @return NotExistingFileException
      */
-    public static function create($notExistingFilePath)
+    public function __construct($notExistingFilePath)
     {
         $template = 'File with path \'%s\' does not exist (or is not readable). Did you provide path of proper file?';
         $message = sprintf($template, $notExistingFilePath);
 
-        return new static($message);
+        parent::__construct($message);
     }
 }

src/Exception/Method/DisabledMethodException.php

@@ -13,19 +13,18 @@ use Exception;
 /**
  * An exception used while method cannot be called, because is disabled
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class DisabledMethodException extends Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      *
      * @param string $disabledMethod    Name of the disabled method
      * @param string $alternativeMethod (optional) Name of the alternative method
-     * @return DisabledMethodException
      */
-    public static function create($disabledMethod, $alternativeMethod = '')
+    public function __construct($disabledMethod, $alternativeMethod = '')
     {
         $template = 'Method %s() cannot be called, because is disabled.';
 
@@ -34,7 +33,6 @@ class DisabledMethodException extends Exception
         }
 
         $message = sprintf($template, $disabledMethod, $alternativeMethod);
-
+        parent::__construct($message);
-        return new static($message);
     }
 }

src/Exception/Reflection/CannotResolveClassNameException.php

@@ -13,21 +13,20 @@ use Exception;
 /**
  * An exception used while name of class or trait cannot be resolved
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class CannotResolveClassNameException extends Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      *
      * @param array|object|string $source   Source of the class's / trait's name. It can be an array of objects,
      *                                      namespaces, object or namespace.
      * @param bool                $forClass (optional) If is set to true, message of this exception for class is
      *                                      prepared. Otherwise - for trait.
-     * @return CannotResolveClassNameException
      */
-    public static function create($source, $forClass = true)
+    public function __construct($source, $forClass = true)
     {
         $forWho = 'trait';
         $value = '';
@@ -43,6 +42,6 @@ class CannotResolveClassNameException extends Exception
         $template = 'Name of %s from given \'%s\'%s cannot be resolved. Is there everything ok?';
         $message = sprintf($template, $forWho, gettype($source), $value);
 
-        return new static($message);
+        parent::__construct($message);
     }
 }

src/Exception/Reflection/MissingChildClassesException.php

@@ -14,19 +14,18 @@ use Meritoo\Common\Utilities\Reflection;
 /**
  * An exception used while given class has no child classes
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class MissingChildClassesException extends Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      *
      * @param array|object|string $parentClass Class that hasn't child classes, but it should. An array of objects,
      *                                         strings, object or string.
-     * @return MissingChildClassesException
      */
-    public static function create($parentClass)
+    public function __construct($parentClass)
     {
         $template = 'The \'%s\' class requires one child class at least who will extend her (maybe is an abstract'
             . ' class), but the child classes are missing. Did you forget to extend this class?';
@@ -34,6 +33,6 @@ class MissingChildClassesException extends Exception
         $parentClassName = Reflection::getClassName($parentClass);
         $message = sprintf($template, $parentClassName);
 
-        return new static($message);
+        parent::__construct($message);
     }
 }

src/Exception/Reflection/NotExistingPropertyException.php

@@ -1,33 +0,0 @@
-<?php
-
-/**
- * (c) Meritoo.pl, http://www.meritoo.pl
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
-namespace Meritoo\Common\Exception\Reflection;
-
-/**
- * An exception used while property does not exist in instance of class
- *
- * @author    Meritoo <github@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
- */
-class NotExistingPropertyException extends \Exception
-{
-    /**
-     * Creates exception
-     *
-     * @param mixed  $object   Object that should contains given property
-     * @param string $property Name of the property
-     * @return NotExistingPropertyException
-     */
-    public static function create($object, $property)
-    {
-        $template = 'Property \'%s\' does not exist in instance of class \'%s\'. Did you use proper name of property?';
-        $message = sprintf($template, $property, get_class($object));
-
-        return new static($message);
-    }
-}

src/Exception/Reflection/TooManyChildClassesException.php

@@ -14,20 +14,19 @@ use Meritoo\Common\Utilities\Reflection;
 /**
  * An exception used while given class has more than one child class
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class TooManyChildClassesException extends Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      *
      * @param array|object|string $parentClass  Class that has more than one child class, but it shouldn't. An array
      *                                          of objects, strings, object or string.
      * @param array               $childClasses Child classes
-     * @return TooManyChildClassesException
      */
-    public static function create($parentClass, array $childClasses)
+    public function __construct($parentClass, array $childClasses)
     {
         $template = "The '%s' class requires one child class at most who will extend her, but more than one child"
             . " class was found:\n- %s\n\nWhy did you create more than one classes that extend '%s' class?";
@@ -35,6 +34,6 @@ class TooManyChildClassesException extends Exception
         $parentClassName = Reflection::getClassName($parentClass);
         $message = sprintf($template, $parentClassName, implode("\n- ", $childClasses), $parentClassName);
 
-        return new static($message);
+        parent::__construct($message);
     }
 }

src/Exception/Regex/IncorrectColorHexLengthException.php

@@ -11,24 +11,22 @@ namespace Meritoo\Common\Exception\Regex;
 /**
  * An exception used while length of given hexadecimal value of color is incorrect
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class IncorrectColorHexLengthException extends \Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      *
      * @param string $color Incorrect hexadecimal value of color
-     * @return IncorrectColorHexLengthException
      */
-    public static function create($color)
+    public function __construct($color)
     {
         $template = 'Length of hexadecimal value of color \'%s\' is incorrect. It\'s %d, but it should be 3 or 6.'
             . ' Is there everything ok?';
 
         $message = sprintf($template, $color, strlen($color));
-
+        parent::__construct($message);
-        return new static($message);
     }
 }

src/Exception/Regex/InvalidColorHexValueException.php

@@ -11,21 +11,19 @@ namespace Meritoo\Common\Exception\Regex;
 /**
  * An exception used while given hexadecimal value of color is invalid
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class InvalidColorHexValueException extends \Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      *
      * @param string $color Invalid hexadecimal value of color
-     * @return InvalidColorHexValueException
      */
-    public static function create($color)
+    public function __construct($color)
     {
         $message = sprintf('Hexadecimal value of color \'%s\' is invalid. Is there everything ok?', $color);
-
+        parent::__construct($message);
-        return new static($message);
     }
 }

src/Exception/Regex/InvalidHtmlAttributesException.php

@@ -1,31 +0,0 @@
-<?php
-
-/**
- * (c) Meritoo.pl, http://www.meritoo.pl
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
-namespace Meritoo\Common\Exception\Regex;
-
-/**
- * An exception used while html attributes are invalid
- *
- * @author    Meritoo <github@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
- */
-class InvalidHtmlAttributesException extends \Exception
-{
-    /**
-     * Creates exception
-     *
-     * @param string $htmlAttributes Invalid html attributes
-     * @return InvalidHtmlAttributesException
-     */
-    public static function create($htmlAttributes)
-    {
-        $message = sprintf('HTML attributes \'%s\' are invalid. Is there everything ok?', $htmlAttributes);
-
-        return new static($message);
-    }
-}

src/Exception/Regex/InvalidUrlException.php

@@ -1,31 +1,23 @@
 <?php
 
-/**
- * (c) Meritoo.pl, http://www.meritoo.pl
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
 namespace Meritoo\Common\Exception\Regex;
 
 /**
  * An exception used while url is invalid
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class InvalidUrlException extends \Exception
 {
     /**
-     * Creates exception
+     * Class constructor
      *
      * @param string $url Invalid url
-     * @return InvalidUrlException
      */
-    public static function create($url)
+    public function __construct($url)
     {
         $message = sprintf('Url \'%s\' is invalid. Is there everything ok?', $url);
-
+        parent::__construct($message);
-        return new static($message);
     }
 }

src/Exception/Type/UnknownDatePartTypeException.php

@@ -1,36 +0,0 @@
-<?php
-
-/**
- * (c) Meritoo.pl, http://www.meritoo.pl
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
-namespace Meritoo\Common\Exception\Type;
-
-use Meritoo\Common\Exception\Base\UnknownTypeException;
-use Meritoo\Common\Type\DatePartType;
-
-/**
- * An exception used while type of date part, e.g. "year", is unknown
- *
- * @author    Meritoo <github@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
- */
-class UnknownDatePartTypeException extends UnknownTypeException
-{
-    /**
-     * Creates exception
-     *
-     * @param string $unknownDatePart Unknown type of date part
-     * @param string $value           Incorrect value
-     * @return UnknownDatePartTypeException
-     */
-    public static function createException($unknownDatePart, $value)
-    {
-        /* @var UnknownDatePartTypeException $exception */
-        $exception = parent::create($unknownDatePart, new DatePartType(), sprintf('date part (with value %s)', $value));
-
-        return $exception;
-    }
-}

src/Exception/Type/UnknownOopVisibilityTypeException.php

@@ -1,11 +1,5 @@
 <?php
 
-/**
- * (c) Meritoo.pl, http://www.meritoo.pl
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
 namespace Meritoo\Common\Exception\Type;
 
 use Meritoo\Common\Exception\Base\UnknownTypeException;
@@ -14,22 +8,16 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * An exception used while the visibility of a property, a method or (as of PHP 7.1.0) a constant is unknown
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class UnknownOopVisibilityTypeException extends UnknownTypeException
 {
     /**
-     * Creates exception
+     * {@inheritdoc}
-     *
-     * @param string $unknownType Unknown visibility of a property, a method or (as of PHP 7.1.0) a constant
-     * @return UnknownOopVisibilityTypeException
      */
-    public static function createException($unknownType)
+    public function __construct($unknownType)
     {
-        /* @var UnknownOopVisibilityTypeException $exception */
+        parent::__construct($unknownType, new OopVisibilityType(), 'OOP-related visibility');
-        $exception = parent::create($unknownType, new OopVisibilityType(), 'OOP-related visibility');
-
-        return $exception;
     }
 }

src/Test/Base/BaseTestCase.php

@@ -14,8 +14,8 @@ use PHPUnit\Framework\TestCase;
 /**
  * Base test case with common methods and data providers
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 abstract class BaseTestCase extends TestCase
 {

src/Test/Base/BaseTypeTestCase.php

@@ -13,8 +13,8 @@ use Meritoo\Common\Traits\Test\Base\BaseTypeTestCaseTrait;
 /**
  * Base test case for the type of something
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 abstract class BaseTypeTestCase extends BaseTestCase
 {

src/Traits/Test/Base/BaseTestCaseTrait.php

@@ -14,16 +14,14 @@ use Meritoo\Common\Exception\Type\UnknownOopVisibilityTypeException;
 use Meritoo\Common\Type\OopVisibilityType;
 use Meritoo\Common\Utilities\Miscellaneous;
 use ReflectionClass;
-use ReflectionException;
 use ReflectionMethod;
-use stdClass;
 
 /**
  * BaseTestCaseTrait
  * Created on 2017-11-02
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 trait BaseTestCaseTrait
 {
@@ -32,7 +30,7 @@ trait BaseTestCaseTrait
      *
      * @var string
      */
-    private static $testsDataDirPath = 'data/tests';
+    private static $testsDataDirPath = '.data/tests';
 
     /**
      * Provides an empty value
@@ -105,29 +103,9 @@ trait BaseTestCaseTrait
         yield['surprise/me/one/more/time.txt'];
     }
 
-    /**
-     * Provides non scalar value, e.g. [] or null
-     *
-     * @return Generator
-     */
-    public function provideNonScalarValue()
-    {
-        yield[
-            [],
-        ];
-
-        yield[
-            null,
-        ];
-
-        yield[
-            new stdClass(),
-        ];
-    }
-
     /**
      * Returns path of file used by tests.
-     * It should be placed in /data/tests directory of this project.
+     * It should be placed in /.data/tests directory of this project.
      *
      * @param string $fileName      Name of file
      * @param string $directoryPath (optional) Path of directory containing the file
@@ -159,7 +137,6 @@ trait BaseTestCaseTrait
      * @param int                     $requiredArgumentsCount (optional) Expected count/amount of required arguments
      *                                                        of the verified method
      * @throws UnknownOopVisibilityTypeException
-     * @throws ReflectionException
      *
      * Attention. 2nd argument, the $method, may be:
      * - string - name of the method
@@ -216,7 +193,6 @@ trait BaseTestCaseTrait
      * @param int    $argumentsCount         (optional) Expected count/amount of arguments of the verified method
      * @param int    $requiredArgumentsCount (optional) Expected count/amount of required arguments of the verified
      *                                       method
-     * @throws ReflectionException
      * @throws UnknownOopVisibilityTypeException
      */
     protected static function assertConstructorVisibilityAndArguments(
@@ -238,7 +214,6 @@ trait BaseTestCaseTrait
      * Asserts that class with given namespace has no constructor
      *
      * @param string $classNamespace Namespace of class that contains constructor to verify
-     * @throws ReflectionException
      */
     protected static function assertHasNoConstructor($classNamespace)
     {

src/Traits/Test/Base/BaseTypeTestCaseTrait.php

@@ -14,8 +14,8 @@ use Meritoo\Common\Type\Base\BaseType;
 /**
  * Trait for the base test case for the type of something
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 trait BaseTypeTestCaseTrait
 {

src/Type/Base/BaseType.php

@@ -14,8 +14,8 @@ use Meritoo\Common\Utilities\Reflection;
  * Base / abstract type of something, e.g. type of button, order, date etc.
  * Child class should contain constants - each of them represent one type.
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 abstract class BaseType
 {

src/Type/DatePartType.php

@@ -13,8 +13,8 @@ use Meritoo\Common\Type\Base\BaseType;
 /**
  * Type of date part, e.g. "year"
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class DatePartType extends BaseType
 {

src/Type/OopVisibilityType.php

@@ -7,8 +7,8 @@ use Meritoo\Common\Type\Base\BaseType;
 /**
  * The visibility of a property, a method or (as of PHP 7.1.0) a constant
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  *
  * @see       http://php.net/manual/en/language.oop5.visibility.php
  */

src/Utilities/Arrays.php

@@ -11,8 +11,8 @@ namespace Meritoo\Common\Utilities;
 /**
  * Useful arrays methods
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Arrays
 {
@@ -48,6 +48,23 @@ class Arrays
 
                 if (is_array($value)) {
                     $effect .= self::values2string($value, $arrayColumnKey, $separator);
+                    /*
+                     * Previous version. Doesn't work with array containing arrays, e.g.:
+                     * array(
+                     *      1 => array(
+                     *          'item1',
+                     *          'item2'
+                     *      ),
+                     *      2 => array(
+                     *          'item3',
+                     *          'item4'
+                     *      )
+                     * )
+                     *
+                      if(isset($value[$arrayColumnKey])){
+                      $effect .= $value[$arrayColumnKey];
+                      }
+                     */
                 } else {
                     if (empty($arrayColumnKey)) {
                         $effect .= $value;
@@ -513,10 +530,6 @@ class Arrays
      */
     public static function removeElement(array $array, $item)
     {
-        /*
-         * No elements or the element does not exist?
-         * Nothing to do
-         */
         if (empty($array) || !in_array($item, $array)) {
             return false;
         }
@@ -613,10 +626,6 @@ class Arrays
      */
     public static function setKeysAsValues(array $array, $ignoreDuplicatedValues = true)
     {
-        /*
-         * No elements?
-         * Nothing to do
-         */
         if (empty($array)) {
             return [];
         }
@@ -673,7 +682,7 @@ class Arrays
             return null;
         }
 
-        $effect = &$array;
+        $effect = $array;
         ksort($effect, $sortFlags);
 
         foreach ($effect as &$value) {
@@ -822,20 +831,13 @@ class Arrays
      */
     public static function getLastElementsPaths(array $array, $separator = '.', $parentPath = '', $stopIfMatchedBy = '')
     {
-        /*
+        $paths = [];
-         * No elements?
-         * Nothing to do
-         */
-        if (empty($array)) {
-            return [];
-        }
 
+        if (!empty($array)) {
             if (!empty($stopIfMatchedBy)) {
                 $stopIfMatchedBy = self::makeArray($stopIfMatchedBy);
             }
 
-        $paths = [];
-
             foreach ($array as $key => $value) {
                 $path = $key;
                 $stopRecursion = false;
@@ -884,6 +886,7 @@ class Arrays
                     $paths += $recursivePaths;
                 }
             }
+        }
 
         return $paths;
     }
@@ -1080,10 +1083,6 @@ class Arrays
      */
     public static function getAllValuesOfKey(array $array, $key)
     {
-        /*
-         * No elements?
-         * Nothing to do
-         */
         if (empty($array)) {
             return null;
         }

src/Utilities/Bundle.php

@@ -8,23 +8,20 @@
 
 namespace Meritoo\Common\Utilities;
 
-use Meritoo\Common\Exception\Bundle\IncorrectBundleNameException;
-
 /**
  * Useful methods for bundle
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Bundle
 {
     /**
-     * Returns path of given bundle to view / template with given extension
+     * Returns path to view / template of given bundle
      *
      * @param string $viewPath   Path of the view / template, e.g. "MyDirectory/my-template"
-     * @param string $bundleName Full name of the bundle, e.g. "MyExtraBundle"
+     * @param string $bundleName Name of the bundle, e.g. "MyExtraBundle"
-     * @param string $extension  (optional) Extension of the view / template (default: "html.twig")
+     * @param string $extension  (optional) Extension of the view / template
-     * @throws IncorrectBundleNameException
      * @return string|null
      */
     public static function getBundleViewPath($viewPath, $bundleName, $extension = 'html.twig')
@@ -37,13 +34,6 @@ class Bundle
             return null;
         }
 
-        /*
-         * Given name of bundle is invalid?
-         */
-        if (!Regex::isValidBundleName($bundleName)) {
-            throw IncorrectBundleNameException::create($bundleName);
-        }
-
         /*
          * Path of the view / template doesn't end with given extension?
          */
@@ -51,39 +41,6 @@ class Bundle
             $viewPath = sprintf('%s.%s', $viewPath, $extension);
         }
 
-        /*
+        return sprintf('%s:%s', $bundleName, $viewPath);
-         * Prepare short name of bundle and path of view / template with "/" (instead of ":")
-         */
-        $shortBundleName = static::getShortBundleName($bundleName);
-        $viewPath = str_replace(':', '/', $viewPath);
-
-        return sprintf('@%s/%s', $shortBundleName, $viewPath);
-    }
-
-    /**
-     * Returns short name of bundle (without "Bundle")
-     *
-     * @param string $fullBundleName Full name of the bundle, e.g. "MyExtraBundle"
-     * @throws IncorrectBundleNameException
-     * @return string|null
-     */
-    public static function getShortBundleName($fullBundleName)
-    {
-        /*
-         * Given name of bundle is invalid?
-         */
-        if (!Regex::isValidBundleName($fullBundleName)) {
-            if (!is_string($fullBundleName)) {
-                $fullBundleName = gettype($fullBundleName);
-            }
-
-            throw new IncorrectBundleNameException($fullBundleName);
-        }
-
-        $matches = [];
-        $pattern = Regex::getBundleNamePattern();
-        preg_match($pattern, $fullBundleName, $matches);
-
-        return $matches[1];
     }
 }

src/Utilities/Composer.php

@@ -13,8 +13,8 @@ use stdClass;
 /**
  * Useful Composer-related methods (only static functions)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Composer
 {

src/Utilities/Date.php

@@ -10,16 +10,14 @@ namespace Meritoo\Common\Utilities;
 
 use DateInterval;
 use DateTime;
-use Exception;
+use Meritoo\Common\Exception\Date\UnknownDatePartTypeException;
-use Meritoo\Common\Exception\Type\UnknownDatePartTypeException;
 use Meritoo\Common\Type\DatePartType;
-use Meritoo\Common\Type\DatePeriod;
 
 /**
  * Useful date methods
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Date
 {
@@ -68,19 +66,13 @@ class Date
      * The dates are returned in an array with indexes 'start' and 'end'.
      *
      * @param int $period The period, type of period. One of DatePeriod class constants, e.g. DatePeriod::LAST_WEEK.
-     * @throws Exception
+     * @return DatePeriod
-     * @return null|DatePeriod
      */
     public static function getDatesForPeriod($period)
     {
-        /*
+        $datePeriod = null;
-         * Type of period is incorrect?
-         * Nothing to do
-         */
-        if (!(new DatePeriod())->isCorrectType($period)) {
-            return null;
-        }
 
+        if (DatePeriod::isCorrectPeriod($period)) {
             $dateStart = null;
             $dateEnd = null;
 
@@ -156,18 +148,15 @@ class Date
                     break;
             }
 
-        /*
+            if (null !== $dateStart && null !== $dateEnd) {
-         * Start or end date is unknown?
-         * Nothing to do
-         */
-        if (null === $dateStart || null === $dateEnd) {
-            return null;
-        }
-
                 $dateStart->setTime(0, 0, 0);
                 $dateEnd->setTime(23, 59, 59);
 
-        return new DatePeriod($dateStart, $dateEnd);
+                $datePeriod = new DatePeriod($dateStart, $dateEnd);
+            }
+        }
+
+        return $datePeriod;
     }
 
     /**
@@ -220,7 +209,6 @@ class Date
     /**
      * Returns current day of week
      *
-     * @throws UnknownDatePartTypeException
      * @return int
      */
     public static function getCurrentDayOfWeek()
@@ -242,8 +230,8 @@ class Date
      * @param int $month The month value
      * @param int $day   The day value
      *
-     * @throws UnknownDatePartTypeException
      * @return int
+     * @throws UnknownDatePartTypeException
      */
     public static function getDayOfWeek($year, $month, $day)
     {
@@ -255,21 +243,21 @@ class Date
          * Oops, incorrect year
          */
         if ($year <= 0) {
-            throw UnknownDatePartTypeException::createException(DatePartType::YEAR, $year);
+            throw new UnknownDatePartTypeException(DatePartType::YEAR, $year);
         }
 
         /*
          * Oops, incorrect month
          */
         if ($month < 1 || $month > 12) {
-            throw UnknownDatePartTypeException::createException(DatePartType::MONTH, $month);
+            throw new UnknownDatePartTypeException(DatePartType::MONTH, $month);
         }
 
         /*
          * Oops, incorrect day
          */
         if ($day < 1 || $day > 31) {
-            throw UnknownDatePartTypeException::createException(DatePartType::DAY, $day);
+            throw new UnknownDatePartTypeException(DatePartType::DAY, $day);
         }
 
         if ($month < 3) {
@@ -487,7 +475,6 @@ class Date
      * @param string   $intervalTemplate (optional) Template used to build date interval. It should contain "%d" as the
      *                                   placeholder which is replaced with a number that represents each iteration.
      *                                   Default: interval for days.
-     * @throws Exception
      * @return array
      */
     public static function getDatesCollection(DateTime $startDate, $datesCount, $intervalTemplate = 'P%dD')
@@ -533,7 +520,6 @@ class Date
      * @param int      $end              (optional) End of random partition
      * @param string   $intervalTemplate (optional) Template used to build date interval. The placeholder is replaced
      *                                   with next, iterated value.
-     * @throws Exception
      * @return DateTime
      */
     public static function getRandomDate(DateTime $startDate = null, $start = 1, $end = 100, $intervalTemplate = 'P%sD')

src/Utilities/DatePeriod.php

@@ -6,20 +6,18 @@
  * file that was distributed with this source code.
  */
 
-namespace Meritoo\Common\Type;
+namespace Meritoo\Common\Utilities;
 
 use DateTime;
-use Meritoo\Common\Type\Base\BaseType;
-use Meritoo\Common\Utilities\Date;
 
 /**
  * A date's period.
  * Contains start and end date of the period.
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
-class DatePeriod extends BaseType
+class DatePeriod
 {
     /**
      * The period constant: last month
@@ -110,6 +108,17 @@ class DatePeriod extends BaseType
         $this->endDate = $endDate;
     }
 
+    /**
+     * Returns information if given period is correct
+     *
+     * @param int $period The period to verify
+     * @return bool
+     */
+    public static function isCorrectPeriod($period)
+    {
+        return in_array($period, Reflection::getConstants(__CLASS__));
+    }
+
     /**
      * Returns formatted one of the period's date: start date or end date
      *

src/Utilities/GeneratorUtility.php

@@ -13,8 +13,8 @@ use Generator;
 /**
  * Useful methods for the Generator class (only static functions)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class GeneratorUtility
 {

src/Utilities/Locale.php

@@ -11,8 +11,8 @@ namespace Meritoo\Common\Utilities;
 /**
  * Useful locale methods
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Locale
 {
@@ -23,7 +23,7 @@ class Locale
      *                             setting. It's the same constant as required by setlocale() function.
      * @param string $languageCode Language code, in ISO 639-1 format. Short form of the locale, e.g. "fr".
      * @param string $countryCode  (optional) Country code, in ISO 3166-1 alpha-2 format, e.g. "FR"
-     * @return false|string
+     * @return bool
      *
      * Available categories (values of $category argument):
      * - LC_ALL for all of the below
@@ -57,29 +57,9 @@ class Locale
         }
 
         $localeLongForm = self::getLongForm($languageCode, $countryCode);
+        setlocale($category, $localeLongForm);
 
-        return setlocale($category, $localeLongForm);
+        return true;
-    }
-
-    /**
-     * Returns locale for given category
-     *
-     * @param int $category Named constant specifying the category of the functions affected by the locale setting.
-     *                      It's the same constant as required by setlocale() function.
-     * @return string
-     *
-     * Available categories (values of $category argument):
-     * - LC_ALL for all of the below
-     * - LC_COLLATE for string comparison, see strcoll()
-     * - LC_CTYPE for character classification and conversion, for example strtoupper()
-     * - LC_MONETARY for localeconv()
-     * - LC_NUMERIC for decimal separator (See also localeconv())
-     * - LC_TIME for date and time formatting with strftime()
-     * - LC_MESSAGES for system responses (available if PHP was compiled with libintl)
-     */
-    public static function getLocale($category)
-    {
-        return setlocale($category, '0');
     }
 
     /**

src/Utilities/MimeTypes.php

@@ -11,8 +11,8 @@ namespace Meritoo\Common\Utilities;
 /**
  * Useful methods for mime types of files
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class MimeTypes
 {
@@ -759,8 +759,9 @@ class MimeTypes
      * Returns mime type of given file
      *
      * @param string $filePath Path of the file to check
-     * @throws \RuntimeException
      * @return string
+     *
+     * @throws \RuntimeException
      */
     public static function getMimeType($filePath)
     {

src/Utilities/Miscellaneous.php

@@ -9,15 +9,14 @@
 namespace Meritoo\Common\Utilities;
 
 use Gedmo\Sluggable\Util\Urlizer;
-use Meritoo\Common\Exception\Regex\IncorrectColorHexLengthException;
+use Symfony\Component\HttpFoundation\Cookie;
-use Meritoo\Common\Exception\Regex\InvalidColorHexValueException;
 use Transliterator;
 
 /**
  * Miscellaneous methods (only static functions)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Miscellaneous
 {
@@ -627,10 +626,10 @@ class Miscellaneous
      * Breaks long text
      *
      * @param string $text                   The text to check and break
-     * @param int    $perLine                (optional) Characters count per line. Default: 100.
+     * @param int    $perLine                (optional) Characters count per line
-     * @param string $separator              (optional) Separator that is placed between lines. Default: "<br>".
+     * @param string $separator              (optional) Separator that is placed beetwen lines
-     * @param string $encoding               (optional) Character encoding. Used by mb_substr(). Default: "UTF-8".
+     * @param string $encoding               (optional) Character encoding. Used by mb_substr().
-     * @param int    $proportionalAberration (optional) Proportional aberration for chars (percent value). Default: 20.
+     * @param int    $proportionalAberration (optional) Proportional aberration for chars (percent value)
      * @return string
      */
     public static function breakLongText(
@@ -717,23 +716,15 @@ class Miscellaneous
      *
      * @param string $directoryPath Directory path
      * @param bool   $contentOnly   (optional) If is set to true, only content of the directory is removed, not
-     *                              directory itself. Otherwise - directory is removed too (default behaviour).
+     *                              directory. Otherwise - directory is removed too.
-     * @return bool|null
+     * @return bool
      */
     public static function removeDirectory($directoryPath, $contentOnly = false)
     {
-        /*
-         * Directory does not exist?
-         * Nothing to do
-         */
         if (!file_exists($directoryPath)) {
-            return null;
+            return true;
         }
 
-        /*
-         * It's not a directory?
-         * Let's treat it like file
-         */
         if (!is_dir($directoryPath)) {
             return unlink($directoryPath);
         }
@@ -748,9 +739,6 @@ class Miscellaneous
             }
         }
 
-        /*
-         * Directory should be removed too?
-         */
         if (!$contentOnly) {
             return rmdir($directoryPath);
         }
@@ -1204,6 +1192,138 @@ class Miscellaneous
         return $value;
     }
 
+    /**
+     * Returns a CURL response with parsed HTTP headers as array with "headers", "cookies" and "content" keys
+     *
+     * The headers and cookies are parsed and returned as an array, and an array of Cookie objects. Returned array looks
+     * like this example:
+     * <code>
+     * [
+     *      'headers' => [
+     *          'Content-Type' => 'text/html; charset=UTF-8',
+     *          ...
+     *      ],
+     *      'cookies' => [
+     *          new Symfony\Component\HttpFoundation\Cookie(),
+     *          new Symfony\Component\HttpFoundation\Cookie(),
+     *          ...
+     *      ],
+     *      'content' => '<html>...</html>'
+     * ]
+     * </code>
+     *
+     * If you want to attach HTTP headers into response content by CURL you need to set "CURLOPT_HEADER" option
+     * to "true". To read exact length of HTTP headers from CURL you can use "curl_getinfo()" function
+     * and read "CURLINFO_HEADER_SIZE" option.
+     *
+     * @param string $response   the full content of response, including HTTP headers
+     * @param int    $headerSize The length of HTTP headers in content
+     * @return array
+     */
+    public static function getCurlResponseWithHeaders($response, $headerSize)
+    {
+        $headerContent = mb_substr($response, 0, $headerSize);
+        $content = mb_substr($response, $headerSize);
+        $headers = [];
+        $cookies = [];
+
+        /*
+         * Let's transform headers content into two arrays: headers and cookies
+         */
+        foreach (explode("\r\n", $headerContent) as $i => $line) {
+            /*
+             * First line is only HTTP status and is unneeded so skip it
+             */
+            if (0 === $i) {
+                continue;
+            }
+
+            if (Regex::contains($line, ':')) {
+                list($key, $value) = explode(': ', $line);
+
+                /*
+                 * If the header is a "set-cookie" let's save it to "cookies" array
+                 */
+                if ('Set-Cookie' === $key) {
+                    $cookieParameters = explode(';', $value);
+
+                    $name = '';
+                    $value = '';
+                    $expire = 0;
+                    $path = '/';
+                    $domain = null;
+                    $secure = false;
+                    $httpOnly = true;
+
+                    foreach ($cookieParameters as $j => $parameter) {
+                        $param = explode('=', $parameter);
+
+                        /*
+                         * First parameter will be always a cookie name and it's value. It is not needed to run
+                         * further actions for them, so save the values and move to next parameter.
+                         */
+                        if (0 === $j) {
+                            $name = trim($param[0]);
+                            $value = trim($param[1]);
+                            continue;
+                        }
+
+                        /*
+                         * Now there would be the rest of cookie parameters, names of params are sent different way so
+                         * I need to lowercase the names and remove unneeded spaces.
+                         */
+                        $paramName = mb_strtolower(trim($param[0]));
+                        $paramValue = true;
+
+                        /*
+                         * Some parameters don't have value e.g. "secure", but the value for them if they're specified
+                         * is "true". Otherwise - just read a value for parameter if exists.
+                         */
+                        if (array_key_exists(1, $param)) {
+                            $paramValue = trim($param[1]);
+                        }
+
+                        switch ($paramName) {
+                            case 'expires':
+                                $expire = $paramValue;
+                                break;
+                            case 'path':
+                                $path = $paramValue;
+                                break;
+                            case 'domain':
+                                $domain = $paramValue;
+                                break;
+                            case 'secure':
+                                $secure = $paramValue;
+                                break;
+                            case 'httponly':
+                                $httpOnly = $paramValue;
+                                break;
+                        }
+                    }
+
+                    /*
+                     * Create new Cookie object and add it to "cookies" array.
+                     * I must skip to next header as cookies shouldn't be saved in "headers" array.
+                     */
+                    $cookies[] = new Cookie($name, $value, $expire, $path, $domain, $secure, $httpOnly);
+                    continue;
+                }
+
+                /*
+                 * Save response header which is not a cookie
+                 */
+                $headers[$key] = $value;
+            }
+        }
+
+        return [
+            'headers' => $headers,
+            'cookies' => $cookies,
+            'content' => $content,
+        ];
+    }
+
     /**
      * Adds missing the "0" characters to given number until given length is reached
      *
@@ -1312,8 +1432,6 @@ class Miscellaneous
      * Returns inverted value of color for given color
      *
      * @param string $color Hexadecimal value of color to invert (with or without hash), e.g. "dd244c" or "#22a5fe"
-     * @throws IncorrectColorHexLengthException
-     * @throws InvalidColorHexValueException
      * @return string
      */
     public static function getInvertedColor($color)

src/Utilities/QueryBuilderUtility.php

@@ -10,7 +10,6 @@ namespace Meritoo\Common\Utilities;
 
 use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\ORM\EntityManager;
-use Doctrine\ORM\OptimisticLockException;
 use Doctrine\ORM\Query\Expr\Join;
 use Doctrine\ORM\Query\Parameter;
 use Doctrine\ORM\QueryBuilder;
@@ -18,8 +17,8 @@ use Doctrine\ORM\QueryBuilder;
 /**
  * Useful methods for query builder (the Doctrine's QueryBuilder class)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class QueryBuilderUtility
 {
@@ -34,10 +33,6 @@ class QueryBuilderUtility
     {
         $aliases = $queryBuilder->getRootAliases();
 
-        /*
-         * No aliases?
-         * Nothing to do
-         */
         if (empty($aliases)) {
             return null;
         }
@@ -47,8 +42,7 @@ class QueryBuilderUtility
 
     /**
      * Returns alias of given property joined in given query builder
-     *
+     * If the join does not exist, null is returned.
-     * If there are no joins or the join does not exist, null is returned.
      * It's also information if given property is already joined in given query builder.
      *
      * @param QueryBuilder $queryBuilder The query builder to verify
@@ -59,10 +53,6 @@ class QueryBuilderUtility
     {
         $joins = $queryBuilder->getDQLPart('join');
 
-        /*
-         * No joins?
-         * Nothing to do
-         */
         if (empty($joins)) {
             return null;
         }
@@ -109,18 +99,7 @@ class QueryBuilderUtility
      */
     public static function setCriteria(QueryBuilder $queryBuilder, array $criteria = [], $alias = '')
     {
-        /*
+        if (!empty($criteria)) {
-         * No criteria used in WHERE clause?
-         * Nothing to do
-         */
-        if (empty($criteria)) {
-            return $queryBuilder;
-        }
-
-        /*
-         * No alias provided?
-         * Let's use root alias
-         */
             if (empty($alias)) {
                 $alias = self::getRootAlias($queryBuilder);
             }
@@ -147,6 +126,7 @@ class QueryBuilderUtility
 
                 $queryBuilder = $queryBuilder->andWhere($predicate);
             }
+        }
 
         return $queryBuilder;
     }
@@ -156,15 +136,14 @@ class QueryBuilderUtility
      *
      * @param EntityManager         $entityManager The entity manager
      * @param array|ArrayCollection $entities      The entities to delete
-     * @param bool                  $flushDeleted  (optional) If is set to true, flushes the deleted objects (default
+     * @param bool                  $flushDeleted  (optional) If is set to true, flushes the deleted objects.
-     *                                             behaviour). Otherwise - not.
+     *                                             Otherwise - not.
-     * @throws OptimisticLockException
      * @return bool
      */
     public static function deleteEntities(EntityManager $entityManager, $entities, $flushDeleted = true)
     {
         /*
-         * No entities provided?
+         * No entities found?
          * Nothing to do
          */
         if (empty($entities)) {
@@ -190,20 +169,13 @@ class QueryBuilderUtility
      * Attention. Existing parameters will be overridden.
      *
      * @param QueryBuilder          $queryBuilder The query builder
-     * @param array|ArrayCollection $parameters   Parameters to add. Collection of Doctrine\ORM\Query\Parameter
+     * @param array|ArrayCollection $parameters   Parameters to add. Collection of instances of
-     *                                            instances or an array with key-value pairs.
+     *                                            Doctrine\ORM\Query\Parameter class or an array with key-value pairs.
      * @return QueryBuilder
      */
     public static function addParameters(QueryBuilder $queryBuilder, $parameters)
     {
-        /*
+        if (!empty($parameters)) {
-         * No parameters?
-         * Nothing to do
-         */
-        if (empty($parameters)) {
-            return $queryBuilder;
-        }
-
             foreach ($parameters as $key => $parameter) {
                 $name = $key;
                 $value = $parameter;
@@ -215,6 +187,7 @@ class QueryBuilderUtility
 
                 $queryBuilder->setParameter($name, $value);
             }
+        }
 
         return $queryBuilder;
     }

src/Utilities/Reflection.php

@@ -13,14 +13,18 @@ use Doctrine\Common\Util\Inflector;
 use Meritoo\Common\Collection\Collection;
 use Meritoo\Common\Exception\Reflection\CannotResolveClassNameException;
 use Meritoo\Common\Exception\Reflection\MissingChildClassesException;
-use Meritoo\Common\Exception\Reflection\NotExistingPropertyException;
 use Meritoo\Common\Exception\Reflection\TooManyChildClassesException;
+use ReflectionClass;
+use ReflectionException;
+use ReflectionMethod;
+use ReflectionObject;
+use ReflectionProperty;
 
 /**
  * Useful reflection methods
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Reflection
 {
@@ -36,14 +40,14 @@ class Reflection
     {
         $effect = [];
 
-        $reflection = new \ReflectionClass($class);
+        $reflection = new ReflectionClass($class);
         $methods = $reflection->getMethods();
 
         if (!empty($methods)) {
             $className = self::getClassName($class);
 
             foreach ($methods as $method) {
-                if ($method instanceof \ReflectionMethod) {
+                if ($method instanceof ReflectionMethod) {
                     if ($withoutInheritance && $className !== $method->class) {
                         continue;
                     }
@@ -64,14 +68,14 @@ class Reflection
      */
     public static function getConstants($class)
     {
-        $reflection = new \ReflectionClass($class);
+        $reflection = new ReflectionClass($class);
 
         return $reflection->getConstants();
     }
 
     /**
-     * Returns maximum integer value of constant of given class / object.
+     * Returns maximum constant from all constants of given class / object.
-     * Constants whose values are integers are considered only.
+     * Values of constants should be integers.
      *
      * @param object|string $class The object or name of object's class
      * @return int|null
@@ -104,7 +108,7 @@ class Reflection
      */
     public static function hasMethod($class, $method)
     {
-        $reflection = new \ReflectionClass($class);
+        $reflection = new ReflectionClass($class);
 
         return $reflection->hasMethod($method);
     }
@@ -118,7 +122,7 @@ class Reflection
      */
     public static function hasProperty($class, $property)
     {
-        $reflection = new \ReflectionClass($class);
+        $reflection = new ReflectionClass($class);
 
         return $reflection->hasProperty($property);
     }
@@ -132,7 +136,7 @@ class Reflection
      */
     public static function hasConstant($class, $constant)
     {
-        $reflection = new \ReflectionClass($class);
+        $reflection = new ReflectionClass($class);
 
         return $reflection->hasConstant($constant);
     }
@@ -146,7 +150,7 @@ class Reflection
      */
     public static function getConstantValue($class, $constant)
     {
-        $reflection = new \ReflectionClass($class);
+        $reflection = new ReflectionClass($class);
 
         if (self::hasConstant($class, $constant)) {
             return $reflection->getConstant($constant);
@@ -185,16 +189,16 @@ class Reflection
              * Let's dig more and get proper value
              *
              * Required to avoid bug:
-             * \ReflectionObject::__construct() expects parameter 1 to be object, null given
+             * ReflectionObject::__construct() expects parameter 1 to be object, null given
              * (...)
-             * 4. at \ReflectionObject->__construct (null)
+             * 4. at ReflectionObject->__construct (null)
              * 5. at Reflection ::getPropertyValue (null, 'name', true)
              * 6. at ListService->getItemValue (object(Deal), 'project.name', '0')
              *
              * while using "project.name" as property - $project has $name property ($project exists in the Deal class)
              * and the $project equals null
              *
-             * Meritoo <github@meritoo.pl>
+             * Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
              * 2016-11-07
              */
             if (null !== $object) {
@@ -212,17 +216,17 @@ class Reflection
              * Use \ReflectionObject class
              */
             try {
-                $reflectionProperty = new \ReflectionProperty($className, $property);
+                $reflectionProperty = new ReflectionProperty($className, $property);
                 $value = $reflectionProperty->getValue($object);
-            } catch (\ReflectionException $exception) {
+            } catch (ReflectionException $exception) {
                 /*
                  * 2nd try:
                  * Look for the get / has / is methods
                  */
-                $class = new \ReflectionObject($object);
+                $class = new ReflectionObject($object);
                 $valueFound = false;
 
-                if ($force || $class->hasProperty($property)) {
+                if ($class->hasProperty($property) || $force) {
                     $property = Inflector::classify($property);
 
                     $getterPrefixes = [
@@ -235,7 +239,7 @@ class Reflection
                         $getterName = sprintf('%s%s', $prefix, $property);
 
                         if ($class->hasMethod($getterName)) {
-                            $method = new \ReflectionMethod($object, $getterName);
+                            $method = new ReflectionMethod($object, $getterName);
 
                             /*
                              * Getter is not accessible publicly?
@@ -382,13 +386,13 @@ class Reflection
     {
         $fullClassName = self::getClassName($source);
 
-        if (null === $fullClassName || '' === $fullClassName) {
+        if (empty($fullClassName)) {
             return '';
         }
 
         $className = self::getClassName($source, true);
 
-        if ($className === $fullClassName) {
+        if ($className == $fullClassName) {
             return $className;
         }
 
@@ -407,7 +411,7 @@ class Reflection
         $className = self::getClassName($source);
         $interfaces = class_implements($className);
 
-        return in_array($interface, $interfaces, true);
+        return in_array($interface, $interfaces);
     }
 
     /**
@@ -424,8 +428,8 @@ class Reflection
 
         $parents = class_parents($childClassName);
 
-        if (is_array($parents) && 0 < count($parents)) {
+        if (is_array($parents)) {
-            return in_array($parentClassName, $parents, true);
+            return in_array($parentClassName, $parents);
         }
 
         return false;
@@ -435,22 +439,22 @@ class Reflection
      * Returns given object properties
      *
      * @param array|object|string $source         An array of objects, namespaces, object or namespace
-     * @param int                 $filter         (optional) Filter of properties. Uses \ReflectionProperty class
+     * @param int                 $filter         (optional) Filter of properties. Uses ReflectionProperty class
      *                                            constants. By default all properties are returned.
      * @param bool                $includeParents (optional) If is set to true, properties of parent classes are
      *                                            included (recursively). Otherwise - not.
-     * @return array|\ReflectionProperty
+     * @return array|ReflectionProperty
      */
     public static function getProperties($source, $filter = null, $includeParents = false)
     {
         $className = self::getClassName($source);
-        $reflection = new \ReflectionClass($className);
+        $reflection = new ReflectionClass($className);
 
         if (null === $filter) {
-            $filter = \ReflectionProperty::IS_PRIVATE
+            $filter = ReflectionProperty::IS_PRIVATE
-                + \ReflectionProperty::IS_PROTECTED
+                + ReflectionProperty::IS_PROTECTED
-                + \ReflectionProperty::IS_PUBLIC
+                + ReflectionProperty::IS_PUBLIC
-                + \ReflectionProperty::IS_STATIC;
+                + ReflectionProperty::IS_STATIC;
         }
 
         $properties = $reflection->getProperties($filter);
@@ -472,12 +476,12 @@ class Reflection
      * Returns a parent class or false if there is no parent class
      *
      * @param array|object|string $source An array of objects, namespaces, object or namespace
-     * @return \ReflectionClass|bool
+     * @return ReflectionClass|bool
      */
     public static function getParentClass($source)
     {
         $className = self::getClassName($source);
-        $reflection = new \ReflectionClass($className);
+        $reflection = new ReflectionClass($className);
 
         return $reflection->getParentClass();
     }
@@ -488,8 +492,8 @@ class Reflection
      *
      * @param array|object|string $class Class who child classes should be returned. An array of objects, strings,
      *                                   object or string.
-     * @throws CannotResolveClassNameException
      * @return array|null
+     * @throws CannotResolveClassNameException
      */
     public static function getChildClasses($class)
     {
@@ -509,7 +513,7 @@ class Reflection
          * Oops, cannot resolve class
          */
         if (null === $className) {
-            throw CannotResolveClassNameException::create($class);
+            throw new CannotResolveClassNameException($class);
         }
 
         $childClasses = [];
@@ -526,7 +530,7 @@ class Reflection
                  */
                 $realClass = ClassUtils::getRealClass($oneClass);
 
-                if (in_array($realClass, $childClasses, true)) {
+                if (in_array($realClass, $childClasses)) {
                     continue;
                 }
 
@@ -543,9 +547,10 @@ class Reflection
      *
      * @param array|object|string $parentClass Class who child class should be returned. An array of objects,
      *                                         namespaces, object or namespace.
+     * @return mixed
+     *
      * @throws MissingChildClassesException
      * @throws TooManyChildClassesException
-     * @return mixed
      */
     public static function getOneChildClass($parentClass)
     {
@@ -556,7 +561,7 @@ class Reflection
          * Oops, the base / parent class hasn't child class
          */
         if (empty($childClasses)) {
-            throw MissingChildClassesException::create($parentClass);
+            throw new MissingChildClassesException($parentClass);
         }
 
         /*
@@ -564,20 +569,20 @@ class Reflection
          * Oops, the base / parent class has too many child classes
          */
         if (count($childClasses) > 1) {
-            throw TooManyChildClassesException::create($parentClass, $childClasses);
+            throw new TooManyChildClassesException($parentClass, $childClasses);
         }
 
         return trim($childClasses[0]);
     }
 
     /**
-     * Returns property, the \ReflectionProperty instance, of given object
+     * Returns property, the ReflectionProperty instance, of given object
      *
      * @param array|object|string $class    An array of objects, namespaces, object or namespace
      * @param string              $property Name of the property
-     * @param int                 $filter   (optional) Filter of properties. Uses \ReflectionProperty class constants.
+     * @param int                 $filter   (optional) Filter of properties. Uses ReflectionProperty class constants.
      *                                      By default all properties are allowed / processed.
-     * @return null|\ReflectionProperty
+     * @return null|ReflectionProperty
      */
     public static function getProperty($class, $property, $filter = null)
     {
@@ -585,9 +590,9 @@ class Reflection
         $properties = self::getProperties($className, $filter);
 
         if (!empty($properties)) {
-            /* @var $reflectionProperty \ReflectionProperty */
+            /* @var $reflectionProperty ReflectionProperty */
             foreach ($properties as $reflectionProperty) {
-                if ($reflectionProperty->getName() === $property) {
+                if ($reflectionProperty->getName() == $property) {
                     return $reflectionProperty;
                 }
             }
@@ -603,8 +608,8 @@ class Reflection
      * @param array|string        $trait         An array of strings or string
      * @param bool                $verifyParents If is set to true, parent classes are verified if they use given
      *                                           trait. Otherwise - not.
-     * @throws CannotResolveClassNameException
      * @return bool|null
+     * @throws CannotResolveClassNameException
      */
     public static function usesTrait($class, $trait, $verifyParents = false)
     {
@@ -614,21 +619,21 @@ class Reflection
         /*
          * Oops, cannot resolve class
          */
-        if (null === $className || '' === $className) {
+        if (empty($className)) {
-            throw CannotResolveClassNameException::create($class);
+            throw new CannotResolveClassNameException($class);
         }
 
         /*
          * Oops, cannot resolve trait
          */
-        if (null === $traitName || '' === $traitName) {
+        if (empty($traitName)) {
             throw new CannotResolveClassNameException($class, false);
         }
 
-        $reflection = new \ReflectionClass($className);
+        $reflection = new ReflectionClass($className);
         $traitsNames = $reflection->getTraitNames();
 
-        $uses = in_array($traitName, $traitsNames, true);
+        $uses = in_array($traitName, $traitsNames);
 
         if (!$uses && $verifyParents) {
             $parentClassName = self::getParentClassName($className);
@@ -651,7 +656,7 @@ class Reflection
     public static function getParentClassName($class)
     {
         $className = self::getClassName($class);
-        $reflection = new \ReflectionClass($className);
+        $reflection = new ReflectionClass($className);
         $parentClass = $reflection->getParentClass();
 
         if (null === $parentClass || false === $parentClass) {
@@ -660,36 +665,4 @@ class Reflection
 
         return $parentClass->getName();
     }
-
-    /**
-     * Sets value of given property
-     *
-     * @param mixed  $object   Object that should contains given property
-     * @param string $property Name of the property
-     * @param mixed  $value    Value of the property
-     * @throws NotExistingPropertyException
-     */
-    public static function setPropertyValue($object, $property, $value)
-    {
-        $reflectionProperty = self::getProperty($object, $property);
-
-        /*
-         * Oops, property does not exist
-         */
-        if (null === $reflectionProperty) {
-            throw NotExistingPropertyException::create($object, $property);
-        }
-
-        $notAccessible = $reflectionProperty->isPrivate() || $reflectionProperty->isProtected();
-
-        if ($notAccessible) {
-            $reflectionProperty->setAccessible(true);
-        }
-
-        $reflectionProperty->setValue($object, $value);
-
-        if ($notAccessible) {
-            $reflectionProperty->setAccessible(false);
-        }
-    }
 }

src/Utilities/Regex.php

@@ -14,8 +14,8 @@ use Meritoo\Common\Exception\Regex\InvalidColorHexValueException;
 /**
  * Useful regular expressions methods
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Regex
 {
@@ -25,21 +25,18 @@ class Regex
      * @var array
      */
     private static $patterns = [
-        'email'            => '/^[\w-.]{2,}@[\w-]+\.[\w]{2,}+$/',
+        'email'            => '/[\w-]{2,}@[\w-]+\.[\w]{2,}+/',
         'phone'            => '/^\+?[0-9 ]+$/',
         'camelCasePart'    => '/([a-z]|[A-Z]){1}[a-z]*/',
         'urlProtocol'      => '/^([a-z]+:\/\/)',
         'urlDomain'        => '([\da-z\.-]+)\.([a-z\.]{2,6})(\/)?([\w\.\-]*)?(\?)?([\w \.\-\/=&]*)\/?$/i',
         'letterOrDigit'    => '/[a-zA-Z0-9]+/',
         'htmlEntity'       => '/&[a-z0-9]+;/',
-        'htmlAttribute'    => '/([\w-]+)="([\w -]+)"/',
         'fileName'         => '/.+\.\w+$/',
         'isQuoted'         => '/^[\'"]{1}.+[\'"]{1}$/',
         'windowsBasedPath' => '/^[A-Z]{1}:\\\.*$/',
         'money'            => '/^[-+]?\d+([\.,]{1}\d*)?$/',
         'color'            => '/^[a-f0-9]{6}$/i',
-        'bundleName'       => '/^(([A-Z]{1}[a-z0-9]+)((?2))*)(Bundle)$/',
-        'binaryValue'      => '/[^\x20-\x7E\t\r\n]/',
     ];
 
     /**
@@ -59,53 +56,20 @@ class Regex
      */
     public static function isValidEmail($email)
     {
-        /*
-         * Not a string?
-         * Nothing to do
-         */
-        if (!is_string($email)) {
-            return false;
-        }
-
         $pattern = self::getEmailPattern();
 
         return (bool)preg_match($pattern, $email);
     }
 
     /**
-     * Returns information if given tax ID is valid (in Poland it's named "NIP")
+     * Returns information if given tax ID (in polish: NIP) is valid
      *
-     * @param string $taxIdString Tax ID (NIP) string
+     * @param string $taxidString Tax ID (NIP) string
      * @return bool
      */
-    public static function isValidTaxId($taxIdString)
+    public static function isValidTaxid($taxidString)
     {
-        /*
+        if (!empty($taxidString)) {
-         * Not a string?
-         * Nothing to do
-         */
-        if (!is_string($taxIdString)) {
-            return false;
-        }
-
-        /*
-         * Empty/Unknown value?
-         * Nothing to do
-         */
-        if (empty($taxIdString)) {
-            return false;
-        }
-
-        $taxId = preg_replace('/[\s-]/', '', $taxIdString);
-
-        /*
-         * Tax ID is not 10 characters length OR is not numeric?
-         * Nothing to do
-         */
-        if (10 !== strlen($taxId) || !is_numeric($taxId)) {
-            return false;
-        }
-
             $weights = [
                 6,
                 5,
@@ -117,20 +81,19 @@ class Regex
                 6,
                 7,
             ];
-
+            $taxid = preg_replace('/[\s-]/', '', $taxidString);
             $sum = 0;
 
+            if (10 == strlen($taxid) && is_numeric($taxid)) {
                 for ($x = 0; $x <= 8; ++$x) {
-            $sum += $taxId[$x] * $weights[$x];
+                    $sum += $taxid[$x] * $weights[$x];
                 }
 
-        /*
+                if ((($sum % 11) % 10) == $taxid[9]) {
-         * Last number it's not a remainder from dividing per 11?
-         * Nothing to do
-         */
-        if ($sum % 11 == $taxId[9]) {
                     return true;
                 }
+            }
+        }
 
         return false;
     }
@@ -145,14 +108,6 @@ class Regex
      */
     public static function isValidUrl($url, $requireProtocol = false)
     {
-        /*
-         * Not a string?
-         * Nothing to do
-         */
-        if (!is_string($url)) {
-            return false;
-        }
-
         $pattern = self::getUrlPattern($requireProtocol);
 
         return (bool)preg_match($pattern, $url);
@@ -166,143 +121,117 @@ class Regex
      */
     public static function isValidPhoneNumber($phoneNumber)
     {
-        /*
-         * Not a string?
-         * Nothing to do
-         */
-        if (!is_string($phoneNumber)) {
-            return false;
-        }
-
         $pattern = self::getPhoneNumberPattern();
 
-        return (bool)preg_match($pattern, trim($phoneNumber));
+        return (bool)preg_match($pattern, $phoneNumber);
     }
 
     /**
-     * Returns array values that match given pattern (or values that keys match the pattern)
+     * Returns array values that matches given pattern (or values that keys matches)
      *
      * @param string $pattern       Pattern to match
-     * @param array  $array         The array (scalar values only)
+     * @param array  $dataArray     The array
-     * @param bool   $itsKeyPattern (optional) If is set to true, keys will be checked if they match pattern.
+     * @param bool   $itsKeyPattern (optional) If is set to true, keys are checks if they match pattern. Otherwise -
-     *                              Otherwise - values will be checked (default behaviour).
+     *                              values are checks.
      * @return array
      */
-    public static function getArrayValuesByPattern($pattern, array $array, $itsKeyPattern = false)
+    public static function getArrayValuesByPattern($pattern, $dataArray, $itsKeyPattern = false)
     {
-        /*
-         * No elements?
-         * Nothing to do
-         */
-        if (empty($array)) {
-            return [];
-        }
-
         if ($itsKeyPattern) {
             $effect = [];
 
-            foreach ($array as $key => $value) {
+            if (!empty($dataArray)) {
-                if ((bool)preg_match($pattern, $key)) {
+                $matches = [];
+
+                foreach ($dataArray as $key => $value) {
+                    if (preg_match($pattern, $key, $matches)) {
                         $effect[$key] = $value;
                     }
                 }
+            }
 
             return $effect;
         }
 
-        return preg_grep($pattern, $array);
+        return preg_grep($pattern, $dataArray);
     }
 
     /**
      * Filters array by given expression and column
      *
-     * Expression can be simple compare expression, like " == 2", or regular expression.
+     * Expression can be simple compare expression, like ' == 2', or regular expression.
      * Returns filtered array.
      *
-     * @param array  $array                The 2-dimensional array that should be filtered
+     * @param array  $array                The array that should be filtered
      * @param string $arrayColumnKey       Column name
-     * @param string $filterExpression     Simple filter expression (e.g. "== 2" or "!= \'home\'") or regular
+     * @param string $filterExpression     Filter expression, e.g. '== 2' or '!= \'home\''
-     *                                     expression (e.g. "/\d+/" or "/[a-z]+[,;]{2,}/")
+     * @param bool   $itsRegularExpression (optional) If is set to true, means that filter expression is a
-     * @param bool   $itsRegularExpression (optional) If is set to true, means that filter expression is a regular
+     *                                     regular expression
-     *                                     expression. Otherwise - not (default behaviour).
      * @return array
      */
     public static function arrayFilter($array, $arrayColumnKey, $filterExpression, $itsRegularExpression = false)
     {
-        /*
+        $effect = [];
-         * No elements?
-         * Nothing to do
-         */
-        if (empty($array)) {
-            return [];
-        }
 
+        if (!empty($array)) {
             $effect = $array;
 
             foreach ($effect as $key => &$item) {
-            if (!isset($item[$arrayColumnKey])) {
+                if (isset($item[$arrayColumnKey])) {
-                continue;
-            }
-
                     $value = $item[$arrayColumnKey];
 
                     if ($itsRegularExpression) {
-                $matchesCount = preg_match($filterExpression, $value);
+                        $matches = [];
+                        $pattern = '|' . $filterExpression . '|';
+                        $matchesCount = preg_match($pattern, $value, $matches);
+
                         $remove = 0 == $matchesCount;
                     } else {
-                if (is_string($value)) {
+                        if ('' == $value) {
-                    $value = sprintf('\'%s\'', $value);
+                            $value = '\'\'';
-                } elseif (is_bool($value)) {
+                        } elseif (is_string($value)) {
-                    if (true === $value) {
+                            $value = '\'' . $value . '\'';
-                        $value = 'true';
-                    } else {
-                        $value = 'false';
-                    }
                         }
 
-                eval(sprintf('$isEqual = %s%s;', $value, $filterExpression));
+                        eval('$isTrue = ' . $value . $filterExpression . ';');
 
-                /* @var bool $isEqual */
+                        /* @var bool $isTrue */
-                $remove = !$isEqual;
+                        $remove = !$isTrue;
                     }
 
                     if ($remove) {
                         unset($effect[$key]);
                     }
                 }
+            }
+        }
 
         return $effect;
     }
 
     /**
-     * Performs regular expression match with many given patterns.
+     * Perform regular expression match with many given patterns.
      * Returns information if given $subject matches one or all given $patterns.
      *
      * @param array|string $patterns     The patterns to match
      * @param string       $subject      The string to check
      * @param bool         $mustAllMatch (optional) If is set to true, $subject must match all $patterns. Otherwise -
-     *                                   not (default behaviour).
+     *                                   not.
      * @return bool
      */
     public static function pregMultiMatch($patterns, $subject, $mustAllMatch = false)
     {
-        /*
-         * No patterns?
-         * Nothing to do
-         */
-        if (empty($patterns)) {
-            return false;
-        }
-
         $effect = false;
         $patterns = Arrays::makeArray($patterns);
 
+        if (!empty($patterns)) {
             if ($mustAllMatch) {
                 $effect = true;
             }
 
             foreach ($patterns as $pattern) {
-            $matched = (bool)preg_match_all($pattern, $subject);
+                $matches = [];
+                $matched = (bool)preg_match_all($pattern, $subject, $matches);
 
                 if ($mustAllMatch) {
                     $effect = $effect && $matched;
@@ -313,6 +242,7 @@ class Regex
                     }
                 }
             }
+        }
 
         return $effect;
     }
@@ -746,14 +676,6 @@ class Regex
      */
     public static function isValidMoneyValue($value)
     {
-        /*
-         * Not a scalar value?
-         * Nothing to do
-         */
-        if (!is_scalar($value)) {
-            return false;
-        }
-
         $pattern = self::getMoneyPattern();
 
         return (bool)preg_match($pattern, $value);
@@ -766,54 +688,32 @@ class Regex
      * @param string $color          Color to verify
      * @param bool   $throwException (optional) If is set to true, throws an exception if given color is invalid
      *                               (default behaviour). Otherwise - not.
+     * @return string|bool
+     *
      * @throws IncorrectColorHexLengthException
      * @throws InvalidColorHexValueException
-     * @return string|bool
      */
     public static function getValidColorHexValue($color, $throwException = true)
     {
-        /*
-         * Not a scalar value?
-         * Nothing to do
-         */
-        if (!is_scalar($color)) {
-            return false;
-        }
-
         $color = Miscellaneous::replace($color, '/#/', '');
         $length = strlen($color);
 
-        /*
+        if (3 === $length) {
-         * Color is not 3 or 6 characters long?
+            $color = Miscellaneous::replace($color, '/(.)(.)(.)/', '$1$1$2$2$3$3');
-         * Nothing to do
+        } else {
-         */
+            if (6 !== $length) {
-        if (3 !== $length && 6 !== $length) {
                 if ($throwException) {
                     throw new IncorrectColorHexLengthException($color);
                 }
 
                 return false;
             }
-
-        /*
-         * Color is 3 characters long?
-         * Let's make it 6 characters long
-         */
-        if (3 === $length) {
-            $color = Miscellaneous::replace($color, '/(.)(.)(.)/', '$1$1$2$2$3$3');
         }
 
         $pattern = self::$patterns['color'];
         $match = (bool)preg_match($pattern, $color);
 
-        /*
+        if (!$match) {
-         * It's valid color
-         * Nothing to do more
-         */
-        if ($match) {
-            return strtolower($color);
-        }
-
             if ($throwException) {
                 throw new InvalidColorHexValueException($color);
             }
@@ -821,107 +721,6 @@ class Regex
             return false;
         }
 
-    /**
+        return strtolower($color);
-     * Returns information if given name of bundle is valid
-     *
-     * @param string $bundleName Full name of bundle to verify, e.g. "MyExtraBundle"
-     * @return bool
-     */
-    public static function isValidBundleName($bundleName)
-    {
-        /*
-         * Not a string?
-         * Nothing to do
-         */
-        if (!is_string($bundleName)) {
-            return false;
-        }
-
-        $pattern = self::getBundleNamePattern();
-
-        return (bool)preg_match($pattern, $bundleName);
-    }
-
-    /**
-     * Returns pattern used to validate / verify name of bundle
-     *
-     * @return string
-     */
-    public static function getBundleNamePattern()
-    {
-        return self::$patterns['bundleName'];
-    }
-
-    /**
-     * Returns pattern used to validate / verify html attribute
-     *
-     * @return string
-     */
-    public static function getHtmlAttributePattern()
-    {
-        return self::$patterns['htmlAttribute'];
-    }
-
-    /**
-     * Returns information if given html attribute is valid
-     *
-     * @param string $htmlAttribute The html attribute to verify
-     * @return bool
-     */
-    public static function isValidHtmlAttribute($htmlAttribute)
-    {
-        /*
-         * Not a string?
-         * Nothing to do
-         */
-        if (!is_string($htmlAttribute)) {
-            return false;
-        }
-
-        $pattern = self::getHtmlAttributePattern();
-
-        return (bool)preg_match($pattern, $htmlAttribute);
-    }
-
-    /**
-     * Returns information if given html attributes are valid
-     *
-     * @param string $htmlAttributes The html attributes to verify
-     * @return bool
-     */
-    public static function areValidHtmlAttributes($htmlAttributes)
-    {
-        /*
-         * Not a string?
-         * Nothing to do
-         */
-        if (!is_string($htmlAttributes)) {
-            return false;
-        }
-
-        $pattern = self::getHtmlAttributePattern();
-
-        return (bool)preg_match_all($pattern, $htmlAttributes);
-    }
-
-    /**
-     * Returns information if given value is a binary value
-     *
-     * @param string $value Value to verify
-     * @return bool
-     */
-    public static function isBinaryValue($value)
-    {
-        /*
-         * Not a string?
-         * Nothing to do
-         */
-        if (!is_string($value)) {
-            return false;
-        }
-
-        $pattern = self::$patterns['binaryValue'];
-
-        return (bool)preg_match($pattern, $value);
     }
 }

src/Utilities/Repository.php

@@ -14,150 +14,74 @@ use Doctrine\ORM\QueryBuilder;
 /**
  * Useful methods for repository
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Repository
 {
-    /**
-     * Name of key responsible for sorting/position of an item in array
-     *
-     * @var string
-     */
-    const POSITION_KEY = 'position';
-
     /**
      * Replenishes positions of given items
      *
-     * @param array $items  Objects who have "getPosition()" and "setPosition()" methods or arrays
+     * @param array $items  The items
-     * @param bool  $asLast (optional) If is set to true, items are placed at the end (default behaviour). Otherwise -
+     * @param bool  $asLast (optional) If is set to true, items are placed at the end. Otherwise - at the top.
-     *                      at top.
      * @param bool  $force  (optional) If is set to true, positions are set even there is no extreme position.
-     *                      Otherwise - if extreme position is unknown (is null) replenishment is stopped / skipped
+     *                      Otherwise - if extreme position is not found (is null) replenishment is stopped / skipped.
-     *                      (default behaviour).
      */
-    public static function replenishPositions(array &$items, $asLast = true, $force = false)
+    public static function replenishPositions($items, $asLast = true, $force = false)
     {
         $position = self::getExtremePosition($items, $asLast);
 
-        /*
-         * Extreme position is unknown, but it's required?
-         * Use 0 as default/start value
-         */
         if (null === $position && $force) {
             $position = 0;
         }
 
-        /*
+        if (null !== $position && !empty($items)) {
-         * Extreme position is unknown or there are no items to sort?
+            foreach ($items as $item) {
-         * Nothing to do
+                if (method_exists($item, 'getPosition')) {
-         */
+                    if (null === $item->getPosition()) {
-        if (null === $position || empty($items)) {
-            return;
-        }
-
-        foreach ($items as &$item) {
-            /*
-             * The item is not sortable?
-             */
-            if (!self::isSortable($item)) {
-                continue;
-            }
-
-            /*
-             * Position has been set?
-             * Nothing to do
-             */
-            if (self::isSorted($item)) {
-                continue;
-            }
-
-            /*
-             * Calculate position
-             */
                         if ($asLast) {
                             ++$position;
                         } else {
                             --$position;
                         }
 
-            /*
+                        if (method_exists($item, 'setPosition')) {
-             * It's an object?
-             * Use proper method to set position
-             */
-            if (is_object($item)) {
                             $item->setPosition($position);
-                continue;
                         }
-
+                    }
-            /*
+                }
-             * It's an array
+            }
-             * Use proper key to set position
-             */
-            $item[static::POSITION_KEY] = $position;
         }
     }
 
     /**
      * Returns extreme position (max or min) of given items
      *
-     * @param array $items Objects who have "getPosition()" and "setPosition()" methods or arrays
+     * @param array $items The items
      * @param bool  $max   (optional) If is set to true, maximum value is returned. Otherwise - minimum.
      * @return int
      */
-    public static function getExtremePosition(array $items, $max = true)
+    public static function getExtremePosition($items, $max = true)
     {
-        /*
-         * No items?
-         * Nothing to do
-         */
-        if (empty($items)) {
-            return null;
-        }
-
         $extreme = null;
 
+        if (!empty($items)) {
             foreach ($items as $item) {
-            /*
+                if (Reflection::hasMethod($item, 'getPosition')) {
-             * The item is not sortable?
-             */
-            if (!self::isSortable($item)) {
-                continue;
-            }
-
-            $position = null;
-
-            /*
-             * Let's grab the position
-             */
-            if (is_object($item)) {
                     $position = $item->getPosition();
-            } elseif (array_key_exists(static::POSITION_KEY, $item)) {
-                $position = $item[static::POSITION_KEY];
-            }
 
-            /*
-             * Maximum value is expected?
-             */
                     if ($max) {
-                /*
+                        if ($position > $extreme) {
-                 * Position was found and it's larger than previously found position (the extreme position)?
-                 */
-                if (null === $extreme || (null !== $position && $position > $extreme)) {
                             $extreme = $position;
                         }
-
+                    } else {
-                continue;
+                        if ($position < $extreme) {
-            }
-
-            /*
-             * Minimum value is expected here.
-             * Position was found and it's smaller than previously found position (the extreme position)?
-             */
-            if (null === $extreme || (null !== $position && $position < $extreme)) {
                             $extreme = $position;
                         }
                     }
+                }
+            }
+        }
 
         return $extreme;
     }
@@ -182,54 +106,4 @@ class Repository
             ->createQueryBuilder($alias)
             ->orderBy(sprintf('%s.%s', $alias, $property), $direction);
     }
-
-    /**
-     * Returns information if given item is sortable
-     *
-     * Sortable means it's an:
-     * - array
-     * or
-     * - object and has getPosition() and setPosition()
-     *
-     * @param mixed $item An item to verify (object who has "getPosition()" and "setPosition()" methods or an array)
-     * @return bool
-     */
-    private static function isSortable($item)
-    {
-        return is_array($item)
-            ||
-            (
-                is_object($item)
-                &&
-                Reflection::hasMethod($item, 'getPosition')
-                &&
-                Reflection::hasMethod($item, 'setPosition')
-            );
-    }
-
-    /**
-     * Returns information if given item is sorted (position has been set)
-     *
-     * @param mixed $item An item to verify (object who has "getPosition()" and "setPosition()" methods or an array)
-     * @return bool
-     */
-    private static function isSorted($item)
-    {
-        /*
-         * Given item is not sortable?
-         */
-        if (!self::isSortable($item)) {
-            return false;
-        }
-
-        /*
-         * It's an object or it's an array
-         * and position has been set?
-         */
-
-        return
-            (is_object($item) && null !== $item->getPosition())
-            ||
-            (is_array($item) && isset($item[static::POSITION_KEY]));
-    }
 }

src/Utilities/Uri.php

@@ -11,8 +11,8 @@ namespace Meritoo\Common\Utilities;
 /**
  * Useful uri methods (only static functions)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Uri
 {
@@ -24,25 +24,13 @@ class Uri
      */
     public static function getFullUri($withoutHost = false)
     {
-        $requestedUrl = Miscellaneous::getSafelyGlobalVariable(INPUT_SERVER, 'REQUEST_URI');
+        $effect = Miscellaneous::getSafelyGlobalVariable(INPUT_SERVER, 'REQUEST_URI');
 
-        /*
-         * Unknown requested url?
-         * Nothing to do
-         */
-        if (empty($requestedUrl)) {
-            return '';
-        }
-
-        /*
-         * Without host / server name?
-         * All is done
-         */
         if ($withoutHost) {
-            return $requestedUrl;
+            return $effect;
         }
 
-        return self::getServerNameOrIp(true) . $requestedUrl;
+        return self::getServerNameOrIp(true) . $effect;
     }
 
     /**
@@ -53,25 +41,13 @@ class Uri
      */
     public static function getServerNameOrIp($withProtocol = false)
     {
-        $host = Miscellaneous::getSafelyGlobalVariable(INPUT_SERVER, 'HTTP_HOST');
+        $protocol = '';
 
-        /*
-         * Unknown host / server?
-         * Nothing to do
-         */
-        if (empty($host)) {
-            return '';
-        }
-
-        /*
-         * With protocol?
-         * Let's include the protocol
-         */
         if ($withProtocol) {
-            return sprintf('%s://%s', self::getProtocolName(), $host);
+            $protocol .= self::getProtocolName() . '://';
         }
 
-        return $host;
+        return $protocol . Miscellaneous::getSafelyGlobalVariable(INPUT_SERVER, 'HTTP_HOST');
     }
 
     /**
@@ -81,6 +57,8 @@ class Uri
      */
     public static function getProtocolName()
     {
+        $effect = '';
+
         $matches = [];
         $protocolData = Miscellaneous::getSafelyGlobalVariable(INPUT_SERVER, 'SERVER_PROTOCOL'); // e.g. HTTP/1.1
         $matchCount = preg_match('|(.+)\/(.+)|', $protocolData, $matches);
@@ -90,14 +68,11 @@ class Uri
          * $matches[2] - protocol version, e.g. 1.1
          */
 
-        /*
+        if ($matchCount > 0) {
-         * Oops, cannot match protocol
+            $effect = strtolower($matches[1]);
-         */
-        if (0 == $matchCount) {
-            return '';
         }
 
-        return strtolower($matches[1]);
+        return $effect;
     }
 
     /**
@@ -107,7 +82,13 @@ class Uri
      */
     public static function getRefererUri()
     {
-        return Miscellaneous::getSafelyGlobalVariable(INPUT_SERVER, 'HTTP_REFERER');
+        $effect = '';
+
+        if (filter_has_var(INPUT_SERVER, 'HTTP_REFERER')) {
+            $effect = Miscellaneous::getSafelyGlobalVariable(INPUT_SERVER, 'HTTP_REFERER');
+        }
+
+        return $effect;
     }
 
     /**
@@ -233,35 +214,10 @@ class Uri
      */
     public static function isExternalUrl($url)
     {
-        /*
-         * Unknown url or it's just slash?
-         * Nothing to do
-         */
-        if (empty($url) || '/' === $url) {
-            return false;
-        }
-
         $currentUrl = self::getServerNameOrIp(true);
         $url = self::replenishProtocol($url);
 
-        /*
+        return !Regex::contains($currentUrl, $url);
-         * Let's prepare pattern of current url
-         */
-        $search = [
-            ':',
-            '/',
-            '.',
-        ];
-
-        $replace = [
-            '\:',
-            '\/',
-            '\.',
-        ];
-
-        $currentUrlPattern = str_replace($search, $replace, $currentUrl);
-
-        return !Regex::contains($url, $currentUrlPattern);
     }
 
     /**
@@ -317,14 +273,6 @@ class Uri
      */
     public static function getSecuredUrl($url, $user = '', $password = '')
     {
-        /*
-         * Url is not provided?
-         * Nothing to do
-         */
-        if (empty($url)) {
-            return '';
-        }
-
         $protocol = self::getProtocolName();
         $host = self::getServerNameOrIp();
 

src/Utilities/Xml.php

@@ -15,8 +15,8 @@ use SimpleXMLElement;
 /**
  * Useful XML-related methods (only static functions)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class Xml
 {

src/ValueObject/Version.php

@@ -1,179 +0,0 @@
-<?php
-
-/**
- * (c) Meritoo.pl, http://www.meritoo.pl
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
-namespace Meritoo\Common\ValueObject;
-
-/**
- * Version of software
- *
- * @author    Meritoo <github@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
- */
-class Version
-{
-    /**
-     * The "major" part.
-     * Incremented when you make incompatible API changes.
-     *
-     * @var int
-     */
-    private $majorPart;
-
-    /**
-     * The "minor" part.
-     * Incremented when you add functionality in a backwards-compatible manner.
-     *
-     * @var int
-     */
-    private $minorPart;
-
-    /**
-     * The "patch" part.
-     * Incremented when you make backwards-compatible bug fixes.
-     *
-     * @var int
-     */
-    private $patchPart;
-
-    /**
-     * Class constructor
-     *
-     * @param int $majorPart The "major" part. Incremented when you make incompatible API changes.
-     * @param int $minorPart The "minor" part. Incremented when you add functionality in a backwards-compatible manner.
-     * @param int $patchPart The "patch" part. Incremented when you make backwards-compatible bug fixes.
-     */
-    public function __construct($majorPart, $minorPart, $patchPart)
-    {
-        $this->majorPart = $majorPart;
-        $this->minorPart = $minorPart;
-        $this->patchPart = $patchPart;
-    }
-
-    /**
-     * Returns the "major" part.
-     * Incremented when you make incompatible API changes.
-     *
-     * @return int
-     */
-    public function getMajorPart()
-    {
-        return $this->majorPart;
-    }
-
-    /**
-     * Returns the "minor" part.
-     * Incremented when you add functionality in a backwards-compatible manner.
-     *
-     * @return int
-     */
-    public function getMinorPart()
-    {
-        return $this->minorPart;
-    }
-
-    /**
-     * Returns the "patch" part.
-     * Incremented when you make backwards-compatible bug fixes.
-     *
-     * @return int
-     */
-    public function getPatchPart()
-    {
-        return $this->patchPart;
-    }
-
-    /**
-     * Returns string representation of instance of this class
-     *
-     * @return string
-     */
-    public function __toString()
-    {
-        return sprintf('%d.%d.%d', $this->getMajorPart(), $this->getMinorPart(), $this->getPatchPart());
-    }
-
-    /**
-     * Returns new instance based on given version as string.
-     * Given version should contain 3 dot-separated integers, 1 per each part ("major", "minor" and "patch").
-     *
-     * Examples:
-     * "1.0.2";
-     * "10.4.0";
-     *
-     * @param string $version The version
-     * @return Version|null
-     */
-    public static function fromString($version)
-    {
-        $version = trim($version);
-
-        /*
-         * No version provided?
-         * Nothing to do
-         */
-        if (empty($version)) {
-            return null;
-        }
-
-        $matches = [];
-        $pattern = '/^(\d+)\.(\d+)\.(\d+)$/'; // e.g. "1.0.2"
-        $matched = preg_match($pattern, $version, $matches);
-
-        /*
-         * Incorrect version?
-         * Nothing to do
-         */
-        if (0 === $matched || false === $matched) {
-            return null;
-        }
-
-        $majorPart = (int)$matches[1];
-        $minorPart = (int)$matches[2];
-        $patchPart = (int)$matches[3];
-
-        return new static($majorPart, $minorPart, $patchPart);
-    }
-
-    /**
-     * Returns new instance based on given version as array.
-     * Given version should contain 3 integers, 1 per each part ("major", "minor" and "patch").
-     *
-     * Examples:
-     * [1, 0, 2];
-     * [10, 4, 0];
-     *
-     * @param array $version The version
-     * @return Version|null
-     */
-    public static function fromArray(array $version)
-    {
-        /*
-         * No version provided?
-         * Nothing to do
-         */
-        if (empty($version)) {
-            return null;
-        }
-
-        $count = count($version);
-
-        /*
-         * Incorrect version?
-         * Nothing to do
-         */
-        if (3 !== $count) {
-            return null;
-        }
-
-        $majorPart = (int)$version[0];
-        $minorPart = (int)$version[1];
-        $patchPart = (int)$version[2];
-
-        return new static($majorPart, $minorPart, $patchPart);
-    }
-}

tests/Collection/CollectionTest.php

@@ -9,7 +9,6 @@
 namespace Meritoo\Common\Test\Collection;
 
 use ArrayIterator;
-use Generator;
 use Meritoo\Common\Collection\Collection;
 use Meritoo\Common\Test\Base\BaseTestCase;
 use Meritoo\Common\Type\OopVisibilityType;
@@ -17,8 +16,8 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of the collection of elements
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class CollectionTest extends BaseTestCase
 {
@@ -130,39 +129,22 @@ class CollectionTest extends BaseTestCase
         static::assertInstanceOf(ArrayIterator::class, $this->simpleCollection->getIterator());
     }
 
-    /**
+    public function testAdd()
-     * @param mixed      $element       The element to add
-     * @param int        $expectedCount Expected count of elements in collection
-     * @param int        $expectedIndex Expected index of added element in collection
-     * @param Collection $collection    The collection
-     *
-     * @dataProvider provideElementToAdd
-     */
-    public function testAddWithoutIndex($element, $expectedCount, $expectedIndex, Collection $collection)
     {
-        $collection->add($element);
+        $this->emptyCollection->add('test1');
 
-        static::assertTrue($collection->has($element));
+        static::assertTrue($this->emptyCollection->has('test1'));
-        static::assertEquals($expectedCount, $collection->count());
+        static::assertEquals(1, $this->emptyCollection->count());
-        static::assertEquals($element, $collection[$expectedIndex]);
+        static::assertEquals('test1', $this->emptyCollection[0]);
     }
 
-    /**
+    public function testAddWithIndex()
-     * @param mixed      $element       The element to add
-     * @param mixed      $index         Index of element to add
-     * @param int        $expectedCount Expected count of elements in collection
-     * @param int        $expectedIndex Expected index of added element in collection
-     * @param Collection $collection    The collection
-     *
-     * @dataProvider provideElementToAddWithIndex
-     */
-    public function testAddWithIndex($element, $index, $expectedCount, $expectedIndex, Collection $collection)
     {
-        $collection->add($element, $index);
+        $this->emptyCollection->add('test2', 1234);
 
-        static::assertTrue($collection->has($element));
+        static::assertTrue($this->emptyCollection->has('test2'));
-        static::assertEquals($expectedCount, $collection->count());
+        static::assertEquals(1, $this->emptyCollection->count());
-        static::assertEquals($element, $collection[$expectedIndex]);
+        static::assertEquals('test2', $this->emptyCollection[1234]);
     }
 
     public function testAddMultipleUsingEmptyArray()
@@ -327,87 +309,6 @@ class CollectionTest extends BaseTestCase
         static::assertMethodVisibilityAndArguments(Collection::class, 'exists', OopVisibilityType::IS_PRIVATE, 1, 1);
     }
 
-    /**
-     * Provides element to add to collection
-     *
-     * @return Generator
-     */
-    public function provideElementToAdd()
-    {
-        $collection = new Collection();
-
-        yield[
-            'test1',
-            1,
-            0,
-            $collection,
-        ];
-
-        yield[
-            'test2',
-            2,
-            1,
-            $collection,
-        ];
-
-        yield[
-            'test3',
-            3,
-            2,
-            $collection,
-        ];
-    }
-
-    /**
-     * Provides element with index to add to collection
-     *
-     * @return Generator
-     */
-    public function provideElementToAddWithIndex()
-    {
-        $collection = new Collection();
-
-        yield[
-            'test1',
-            'aa',
-            1,
-            'aa',
-            $collection,
-        ];
-
-        yield[
-            'test2',
-            'oo',
-            2,
-            'oo',
-            $collection,
-        ];
-
-        yield[
-            'test3',
-            null,
-            3,
-            0,
-            $collection,
-        ];
-
-        yield[
-            'test4',
-            '',
-            4,
-            1,
-            $collection,
-        ];
-
-        yield[
-            'test5',
-            'vv',
-            5,
-            'vv',
-            $collection,
-        ];
-    }
-
     /**
      * {@inheritdoc}
      */

tests/Exception/Base/UnknownTypeExceptionTest.php

@@ -16,14 +16,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of the exception used while type of something is unknown
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class UnknownTypeExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(UnknownTestTypeException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(UnknownTestTypeException::class, OopVisibilityType::IS_PUBLIC, 1, 1);
     }
 
     public function testWithoutException()
@@ -41,8 +41,8 @@ class UnknownTypeExceptionTest extends BaseTestCase
 /**
  * Type of something (for testing purposes)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class TestType extends BaseType
 {
@@ -54,31 +54,27 @@ class TestType extends BaseType
 /**
  * An exception used while type of something is unknown (for testing purposes)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class UnknownTestTypeException extends UnknownTypeException
 {
     /**
-     * Creates exception
+     * Class constructor
      *
-     * @param string $unknownType The unknown type of something (for testing purposes)
+     * @param int|string $unknownType The unknown type of something (for testing purposes)
-     * @return UnknownTestTypeException
      */
-    public static function createException($unknownType)
+    public function __construct($unknownType)
     {
-        /* @var UnknownTestTypeException $exception */
+        parent::__construct($unknownType, new TestType(), 'type of something used for testing');
-        $exception = parent::create($unknownType, new TestType(), 'type of something used for testing');
-
-        return $exception;
     }
 }
 
 /**
  * Service used together with type of something (for testing purposes)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class TestService
 {
@@ -86,8 +82,9 @@ class TestService
      * Returns translated type (for testing purposes)
      *
      * @param string $type Type of something (for testing purposes)
-     * @throws UnknownTestTypeException
      * @return string
+     *
+     * @throws UnknownTestTypeException
      */
     public function getTranslatedType($type)
     {

tests/Exception/Date/UnknownDatePartTypeExceptionTest.php

@@ -9,7 +9,7 @@
 namespace Meritoo\Common\Test\Exception\Date;
 
 use Generator;
-use Meritoo\Common\Exception\Type\UnknownDatePartTypeException;
+use Meritoo\Common\Exception\Date\UnknownDatePartTypeException;
 use Meritoo\Common\Test\Base\BaseTestCase;
 use Meritoo\Common\Type\DatePartType;
 use Meritoo\Common\Type\OopVisibilityType;
@@ -17,14 +17,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while type of date part, e.g. "year", is unknown
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class UnknownDatePartTypeExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(UnknownDatePartTypeException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(UnknownDatePartTypeException::class, OopVisibilityType::IS_PUBLIC, 2, 2);
     }
 
     /**
@@ -34,9 +34,9 @@ class UnknownDatePartTypeExceptionTest extends BaseTestCase
      *
      * @dataProvider provideDatePartAndValue
      */
-    public function testMessage($unknownDatePart, $value, $expectedMessage)
+    public function testConstructorMessage($unknownDatePart, $value, $expectedMessage)
     {
-        $exception = UnknownDatePartTypeException::createException($unknownDatePart, $value);
+        $exception = new UnknownDatePartTypeException($unknownDatePart, $value);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Exception/File/EmptyFileExceptionTest.php

@@ -16,14 +16,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while file with given path is empty (has no content)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class EmptyFileExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(EmptyFileException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(EmptyFileException::class, OopVisibilityType::IS_PUBLIC, 1, 1);
     }
 
     /**
@@ -32,9 +32,9 @@ class EmptyFileExceptionTest extends BaseTestCase
      *
      * @dataProvider providePathOfFile
      */
-    public function testMessage($emptyFilePath, $expectedMessage)
+    public function testConstructorMessage($emptyFilePath, $expectedMessage)
     {
-        $exception = EmptyFileException::create($emptyFilePath);
+        $exception = new EmptyFileException($emptyFilePath);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Exception/File/EmptyFilePathExceptionTest.php

@@ -15,19 +15,19 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while path of given file is empty
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class EmptyFilePathExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(EmptyFilePathException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(EmptyFilePathException::class, OopVisibilityType::IS_PUBLIC);
     }
 
     public function testConstructorMessage()
     {
-        $exception = EmptyFilePathException::create();
+        $exception = new EmptyFilePathException();
         static::assertEquals('Path of the file is empty. Did you provide path of proper file?', $exception->getMessage());
     }
 }

tests/Exception/File/NotExistingFileExceptionTest.php

@@ -16,14 +16,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while file with given path does not exist
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class NotExistingFileExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(NotExistingFileException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(NotExistingFileException::class, OopVisibilityType::IS_PUBLIC, 1, 1);
     }
 
     /**
@@ -34,7 +34,7 @@ class NotExistingFileExceptionTest extends BaseTestCase
      */
     public function testConstructorMessage($notExistingFilePath, $expectedMessage)
     {
-        $exception = NotExistingFileException::create($notExistingFilePath);
+        $exception = new NotExistingFileException($notExistingFilePath);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Exception/Method/DisabledMethodExceptionTest.php

@@ -16,14 +16,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while method cannot be called, because is disabled
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class DisabledMethodExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(DisabledMethodException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(DisabledMethodException::class, OopVisibilityType::IS_PUBLIC, 2, 1);
     }
 
     /**
@@ -36,7 +36,7 @@ class DisabledMethodExceptionTest extends BaseTestCase
      */
     public function testConstructorMessage($disabledMethod, $alternativeMethod, $expectedMessage)
     {
-        $exception = DisabledMethodException::create($disabledMethod, $alternativeMethod);
+        $exception = new DisabledMethodException($disabledMethod, $alternativeMethod);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Exception/Reflection/CannotResolveClassNameExceptionTest.php

@@ -16,14 +16,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while name of class or trait cannot be resolved
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class CannotResolveClassNameExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(CannotResolveClassNameException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(CannotResolveClassNameException::class, OopVisibilityType::IS_PUBLIC, 2, 1);
     }
 
     /**
@@ -37,7 +37,7 @@ class CannotResolveClassNameExceptionTest extends BaseTestCase
      */
     public function testConstructorMessage($source, $forClass, $expectedMessage)
     {
-        $exception = CannotResolveClassNameException::create($source, $forClass);
+        $exception = new CannotResolveClassNameException($source, $forClass);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Exception/Reflection/MissingChildClassesExceptionTest.php

@@ -16,14 +16,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while given class has no child classes
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class MissingChildClassesExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(MissingChildClassesException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(MissingChildClassesException::class, OopVisibilityType::IS_PUBLIC, 1, 1);
     }
 
     /**
@@ -35,7 +35,7 @@ class MissingChildClassesExceptionTest extends BaseTestCase
      */
     public function testConstructorMessage($parentClass, $expectedMessage)
     {
-        $exception = MissingChildClassesException::create($parentClass);
+        $exception = new MissingChildClassesException($parentClass);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Exception/Reflection/TooManyChildClassesExceptionTest.php

@@ -16,14 +16,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while given class has more than one child class
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class TooManyChildClassesExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(TooManyChildClassesException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(TooManyChildClassesException::class, OopVisibilityType::IS_PUBLIC, 2, 2);
     }
 
     /**
@@ -36,7 +36,7 @@ class TooManyChildClassesExceptionTest extends BaseTestCase
      */
     public function testConstructorMessage($parentClass, array $childClasses, $expectedMessage)
     {
-        $exception = TooManyChildClassesException::create($parentClass, $childClasses);
+        $exception = new TooManyChildClassesException($parentClass, $childClasses);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Exception/Regex/IncorrectColorHexLengthExceptionTest.php

@@ -16,14 +16,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while length of given hexadecimal value of color is incorrect
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class IncorrectColorHexLengthExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(IncorrectColorHexLengthException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(IncorrectColorHexLengthException::class, OopVisibilityType::IS_PUBLIC, 1, 1);
     }
 
     /**
@@ -34,7 +34,7 @@ class IncorrectColorHexLengthExceptionTest extends BaseTestCase
      */
     public function testConstructorMessage($color, $expectedMessage)
     {
-        $exception = IncorrectColorHexLengthException::create($color);
+        $exception = new IncorrectColorHexLengthException($color);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Exception/Regex/InvalidColorHexValueExceptionTest.php

@@ -16,14 +16,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while given hexadecimal value of color is invalid
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class InvalidColorHexValueExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(InvalidColorHexValueException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(InvalidColorHexValueException::class, OopVisibilityType::IS_PUBLIC, 1, 1);
     }
 
     /**
@@ -34,7 +34,7 @@ class InvalidColorHexValueExceptionTest extends BaseTestCase
      */
     public function testConstructorMessage($color, $expectedMessage)
     {
-        $exception = InvalidColorHexValueException::create($color);
+        $exception = new InvalidColorHexValueException($color);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Exception/Regex/InvalidHtmlAttributesExceptionTest.php

@@ -1,65 +0,0 @@
-<?php
-
-/**
- * (c) Meritoo.pl, http://www.meritoo.pl
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
-namespace Meritoo\Common\Test\Exception\Regex;
-
-use Generator;
-use Meritoo\Common\Exception\Regex\InvalidHtmlAttributesException;
-use Meritoo\Common\Test\Base\BaseTestCase;
-use Meritoo\Common\Type\OopVisibilityType;
-
-/**
- * Test case of an exception used while html attributes are invalid
- *
- * @author    Meritoo <github@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
- */
-class InvalidHtmlAttributesExceptionTest extends BaseTestCase
-{
-    public function testConstructorVisibilityAndArguments()
-    {
-        static::assertConstructorVisibilityAndArguments(InvalidHtmlAttributesException::class, OopVisibilityType::IS_PUBLIC, 3);
-    }
-
-    /**
-     * @param string $htmlAttributes  Invalid html attributes
-     * @param string $expectedMessage Expected exception's message
-     *
-     * @dataProvider provideHtmlAttributes
-     */
-    public function testConstructorMessage($htmlAttributes, $expectedMessage)
-    {
-        $exception = InvalidHtmlAttributesException::create($htmlAttributes);
-        static::assertEquals($expectedMessage, $exception->getMessage());
-    }
-
-    /**
-     * Provides html attributes
-     *
-     * @return Generator
-     */
-    public function provideHtmlAttributes()
-    {
-        $template = 'HTML attributes \'%s\' are invalid. Is there everything ok?';
-
-        yield[
-            'abc = def',
-            sprintf($template, 'abc = def'),
-        ];
-
-        yield[
-            'abc = def ghi = jkl',
-            sprintf($template, 'abc = def ghi = jkl'),
-        ];
-
-        yield[
-            'abc=def ghi=jkl',
-            sprintf($template, 'abc=def ghi=jkl'),
-        ];
-    }
-}

tests/Exception/Regex/InvalidUrlExceptionTest.php

@@ -16,14 +16,14 @@ use Meritoo\Common\Type\OopVisibilityType;
 /**
  * Test case of an exception used while url is invalid
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class InvalidUrlExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(InvalidUrlException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(InvalidUrlException::class, OopVisibilityType::IS_PUBLIC, 1, 1);
     }
 
     /**
@@ -34,7 +34,7 @@ class InvalidUrlExceptionTest extends BaseTestCase
      */
     public function testConstructorMessage($url, $expectedMessage)
     {
-        $exception = InvalidUrlException::create($url);
+        $exception = new InvalidUrlException($url);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Exception/Type/UnknownOopVisibilityTypeExceptionTest.php

@@ -17,14 +17,14 @@ use Meritoo\Common\Type\OopVisibilityType;
  * Test case of an exception used while the visibility of a property, a method or (as of PHP 7.1.0) a constant is
  * unknown
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class UnknownOopVisibilityTypeExceptionTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
-        static::assertConstructorVisibilityAndArguments(UnknownOopVisibilityTypeException::class, OopVisibilityType::IS_PUBLIC, 3);
+        static::assertConstructorVisibilityAndArguments(UnknownOopVisibilityTypeException::class, OopVisibilityType::IS_PUBLIC, 1, 1);
     }
 
     /**
@@ -35,7 +35,7 @@ class UnknownOopVisibilityTypeExceptionTest extends BaseTestCase
      */
     public function testConstructorMessage($unknownType, $expectedMessage)
     {
-        $exception = UnknownOopVisibilityTypeException::createException($unknownType);
+        $exception = new UnknownOopVisibilityTypeException($unknownType);
         static::assertEquals($expectedMessage, $exception->getMessage());
     }
 

tests/Test/Base/BaseTestCaseTest.php

@@ -17,8 +17,8 @@ use Meritoo\Common\Utilities\GeneratorUtility;
 /**
  * Test case of the base test case with common methods and data providers
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class BaseTestCaseTest extends BaseTestCase
 {
@@ -132,7 +132,7 @@ class BaseTestCaseTest extends BaseTestCase
             $directoryPath .= '/';
         }
 
-        $expectedContains = sprintf('/data/tests/%s%s', $directoryPath, $fileName);
+        $expectedContains = sprintf('/.data/tests/%s%s', $directoryPath, $fileName);
         static::assertContains($expectedContains, $path);
     }
 
@@ -168,8 +168,8 @@ class BaseTestCaseTest extends BaseTestCase
 /**
  * Simple test case
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class SimpleTestCase extends BaseTestCase
 {

tests/Type/Base/BaseTypeTest.php

@@ -15,8 +15,8 @@ use Meritoo\Common\Type\Base\BaseType;
 /**
  * Test case of the base / abstract type of something
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class BaseTypeTest extends BaseTestCase
 {
@@ -184,8 +184,8 @@ class BaseTypeTest extends BaseTestCase
 /**
  * Empty type of something used for testing
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class TestEmptyType extends BaseType
 {
@@ -194,8 +194,8 @@ class TestEmptyType extends BaseType
 /**
  * Type of something used for testing
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class TestType extends BaseType
 {

tests/Type/DatePartTypeTest.php

@@ -14,8 +14,8 @@ use Meritoo\Common\Type\DatePartType;
 /**
  * Test case of the type of date part, e.g. "year"
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class DatePartTypeTest extends BaseTypeTestCase
 {

tests/Utilities/ArraysTest.php

@@ -14,8 +14,8 @@ use Meritoo\Common\Utilities\Arrays;
 /**
  * Test case of the useful arrays methods
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class ArraysTest extends BaseTestCase
 {

tests/Utilities/BundleTest.php

@@ -8,16 +8,14 @@
 
 namespace Meritoo\Common\Test\Utilities;
 
-use Generator;
-use Meritoo\Common\Exception\Bundle\IncorrectBundleNameException;
 use Meritoo\Common\Test\Base\BaseTestCase;
 use Meritoo\Common\Utilities\Bundle;
 
 /**
  * Test case of the useful methods for bundle
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class BundleTest extends BaseTestCase
 {
@@ -26,227 +24,22 @@ class BundleTest extends BaseTestCase
         static::assertHasNoConstructor(Bundle::class);
     }
 
-    /**
+    public function testGetBundleViewPathEmptyPathAndBundle()
-     * @param string $viewPath   Path of the view / template, e.g. "MyDirectory/my-template"
-     * @param string $bundleName Full name of the bundle, e.g. "MyExtraBundle"
-     *
-     * @throws IncorrectBundleNameException
-     * @dataProvider provideEmptyViewPathAndBundle
-     */
-    public function testGetBundleViewPathUsingEmptyPathAndBundle($viewPath, $bundleName)
     {
-        self::assertNull(Bundle::getBundleViewPath($viewPath, $bundleName));
+        self::assertNull(Bundle::getBundleViewPath('', ''));
+        self::assertNull(Bundle::getBundleViewPath('test', ''));
+        self::assertNull(Bundle::getBundleViewPath('', 'test'));
     }
 
-    /**
+    public function testGetBundleViewPathWithDefaultExtension()
-     * @param string $viewPath   Path of the view / template, e.g. "MyDirectory/my-template"
-     * @param string $bundleName Full name of the bundle, e.g. "MyExtraBundle"
-     *
-     * @dataProvider provideViewPathAndIncorrectBundleName
-     */
-    public function testGetBundleViewPathUsingIncorrectBundleName($viewPath, $bundleName)
     {
-        $template = 'Name of bundle \'%s\' is incorrect. It should start with big letter and end with "Bundle". Is'
+        self::assertEquals('Lorem:Ipsum.html.twig', Bundle::getBundleViewPath('Ipsum', 'Lorem'));
-            . ' there everything ok?';
+        self::assertEquals('LobortisTincidunt:FusceElementum.html.twig', Bundle::getBundleViewPath('FusceElementum', 'LobortisTincidunt'));
-
-        $message = sprintf($template, $bundleName);
-        $this->setExpectedException(IncorrectBundleNameException::class, $message);
-
-        Bundle::getBundleViewPath($viewPath, $bundleName);
     }
 
-    /**
+    public function testGetBundleViewPathWithCustomExtension()
-     * @param string $viewPath   Path of the view / template, e.g. "MyDirectory/my-template"
-     * @param string $bundleName Full name of the bundle, e.g. "MyExtraBundle"
-     * @param string $expected   Expected path to view / template
-     *
-     * @throws IncorrectBundleNameException
-     * @dataProvider provideViewPathAndBundle
-     */
-    public function testGetBundleViewPathUsingDefaultExtension($viewPath, $bundleName, $expected)
     {
-        self::assertEquals($expected, Bundle::getBundleViewPath($viewPath, $bundleName));
+        self::assertNull(Bundle::getBundleViewPath('Ipsum', 'Lorem', ''));
-    }
+        self::assertEquals('Lorem:Ipsum.js.twig', Bundle::getBundleViewPath('Ipsum', 'Lorem', 'js.twig'));
-
-    /**
-     * @param string $viewPath   Path of the view / template, e.g. "MyDirectory/my-template"
-     * @param string $bundleName Full name of the bundle, e.g. "MyExtraBundle"
-     * @param string $extension  (optional) Extension of the view / template
-     * @param string $expected   Expected path to view / template
-     *
-     * @throws IncorrectBundleNameException
-     * @dataProvider provideViewPathAndBundleAndExtension
-     */
-    public function testGetBundleViewPathUsingCustomExtension($viewPath, $bundleName, $extension, $expected)
-    {
-        self::assertEquals($expected, Bundle::getBundleViewPath($viewPath, $bundleName, $extension));
-    }
-
-    /**
-     * @param mixed $emptyValue Empty value, e.g. ""
-     *
-     * @throws IncorrectBundleNameException
-     * @dataProvider provideEmptyValue
-     */
-    public function testGetShortBundleNameUsingEmptyValue($emptyValue)
-    {
-        $this->setExpectedException(IncorrectBundleNameException::class);
-        Bundle::getShortBundleName($emptyValue);
-    }
-
-    /**
-     * @param string $bundleName Full name of the bundle, e.g. "MyExtraBundle"
-     *
-     * @throws IncorrectBundleNameException
-     * @dataProvider provideIncorrectBundleName
-     */
-    public function testGetShortBundleNameUsingIncorrectBundleName($bundleName)
-    {
-        $this->setExpectedException(IncorrectBundleNameException::class);
-        Bundle::getShortBundleName($bundleName);
-    }
-
-    /**
-     * @param string $fullBundleName  Full name of the bundle, e.g. "MyExtraBundle"
-     * @param string $shortBundleName Short name of bundle (without "Bundle")
-     *
-     * @throws IncorrectBundleNameException
-     * @dataProvider provideFullAndShortBundleName
-     */
-    public function testGetShortBundleName($fullBundleName, $shortBundleName)
-    {
-        self::assertEquals($shortBundleName, Bundle::getShortBundleName($fullBundleName));
-    }
-
-    /**
-     * Provides empty path of the view / template and/or name of bundle
-     *
-     * @return Generator
-     */
-    public function provideEmptyViewPathAndBundle()
-    {
-        yield[
-            '',
-            '',
-        ];
-
-        yield[
-            'test',
-            '',
-        ];
-
-        yield[
-            '',
-            'test',
-        ];
-    }
-
-    /**
-     * Provides path of the view / template and incorrect name of bundle
-     *
-     * @return Generator
-     */
-    public function provideViewPathAndIncorrectBundleName()
-    {
-        yield[
-            'User:Active',
-            'myExtra',
-        ];
-
-        yield[
-            'User:Active',
-            'MyExtra',
-        ];
-
-        yield[
-            'User:Active',
-            'MySuperExtraGorgeous',
-        ];
-    }
-
-    /**
-     * Provides path of the view / template and name of bundle
-     *
-     * @return Generator
-     */
-    public function provideViewPathAndBundle()
-    {
-        yield[
-            'User',
-            'MyExtraBundle',
-            '@MyExtra/User.html.twig',
-        ];
-
-        yield[
-            'User:Active',
-            'MyExtraBundle',
-            '@MyExtra/User/Active.html.twig',
-        ];
-
-        yield[
-            'User:Active',
-            'MySuperExtraGorgeousBundle',
-            '@MySuperExtraGorgeous/User/Active.html.twig',
-        ];
-    }
-
-    /**
-     * Provides path of the view / template, name of bundle and extension of the view / template
-     *
-     * @return Generator
-     */
-    public function provideViewPathAndBundleAndExtension()
-    {
-        yield[
-            'User:Active',
-            'MyExtraBundle',
-            '',
-            null,
-        ];
-
-        yield[
-            'User:Active',
-            'MyExtraBundle',
-            'js.twig',
-            '@MyExtra/User/Active.js.twig',
-        ];
-    }
-
-    /**
-     * Provides incorrect name of bundle
-     *
-     * @return Generator
-     */
-    public function provideIncorrectBundleName()
-    {
-        yield[
-            'myExtra',
-        ];
-
-        yield[
-            'MyExtra',
-        ];
-
-        yield[
-            'MySuperExtraGorgeous',
-        ];
-    }
-
-    /**
-     * Provides full and short name of bundle
-     *
-     * @return Generator
-     */
-    public function provideFullAndShortBundleName()
-    {
-        yield[
-            'MyExtraBundle',
-            'MyExtra',
-        ];
-
-        yield[
-            'MySuperExtraGorgeousBundle',
-            'MySuperExtraGorgeous',
-        ];
     }
 }

tests/Utilities/ComposerTest.php

@@ -15,8 +15,8 @@ use Meritoo\Common\Utilities\Composer;
 /**
  * Test case of the useful Composer-related methods
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class ComposerTest extends BaseTestCase
 {

tests/Utilities/DatePeriodTest.php

@@ -6,21 +6,21 @@
  * file that was distributed with this source code.
  */
 
-namespace Meritoo\Common\Test\Type;
+namespace Meritoo\Common\Test\Utilities;
 
 use DateTime;
 use Generator;
-use Meritoo\Common\Test\Base\BaseTypeTestCase;
+use Meritoo\Common\Test\Base\BaseTestCase;
-use Meritoo\Common\Type\DatePeriod;
 use Meritoo\Common\Type\OopVisibilityType;
+use Meritoo\Common\Utilities\DatePeriod;
 
 /**
  * Test case of date's period
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
-class DatePeriodTest extends BaseTypeTestCase
+class DatePeriodTest extends BaseTestCase
 {
     public function testConstructorVisibilityAndArguments()
     {
@@ -58,6 +58,33 @@ class DatePeriodTest extends BaseTypeTestCase
         self::assertEquals($endDate, $period->getEndDate());
     }
 
+    /**
+     * @param mixed $period Empty value, e.g. ""
+     * @dataProvider provideEmptyValue
+     */
+    public function testIsCorrectPeriodEmptyPeriod($period)
+    {
+        self::assertFalse(DatePeriod::isCorrectPeriod($period));
+    }
+
+    /**
+     * @param int $period Incorrect period to verify
+     * @dataProvider provideIncorrectPeriod
+     */
+    public function testIsCorrectPeriodIncorrectPeriod($period)
+    {
+        self::assertFalse(DatePeriod::isCorrectPeriod($period));
+    }
+
+    /**
+     * @param int $period The period to verify
+     * @dataProvider providePeriod
+     */
+    public function testIsCorrectPeriod($period)
+    {
+        self::assertTrue(DatePeriod::isCorrectPeriod($period));
+    }
+
     /**
      * @param DatePeriod $period The date period to verify
      * @param string     $format Format used to format the date
@@ -115,6 +142,36 @@ class DatePeriodTest extends BaseTypeTestCase
         ];
     }
 
+    /**
+     * Provides incorrect period
+     *
+     * @return Generator
+     */
+    public function provideIncorrectPeriod()
+    {
+        yield[-1];
+        yield[0];
+        yield[10];
+    }
+
+    /**
+     * Provides period to verify
+     *
+     * @return Generator
+     */
+    public function providePeriod()
+    {
+        yield[DatePeriod::LAST_WEEK];
+        yield[DatePeriod::THIS_WEEK];
+        yield[DatePeriod::NEXT_WEEK];
+        yield[DatePeriod::LAST_MONTH];
+        yield[DatePeriod::THIS_MONTH];
+        yield[DatePeriod::NEXT_MONTH];
+        yield[DatePeriod::LAST_YEAR];
+        yield[DatePeriod::THIS_YEAR];
+        yield[DatePeriod::NEXT_YEAR];
+    }
+
     /**
      * Provides period and incorrect format of date to verify
      *
@@ -213,68 +270,4 @@ class DatePeriodTest extends BaseTypeTestCase
             '2002-02-02 00:00',
         ];
     }
-
-    /**
-     * Returns all expected types of the tested type
-     *
-     * @return array
-     */
-    protected function getAllExpectedTypes()
-    {
-        return [
-            'LAST_MONTH' => DatePeriod::LAST_MONTH,
-            'LAST_WEEK'  => DatePeriod::LAST_WEEK,
-            'LAST_YEAR'  => DatePeriod::LAST_YEAR,
-            'NEXT_MONTH' => DatePeriod::NEXT_MONTH,
-            'NEXT_WEEK'  => DatePeriod::NEXT_WEEK,
-            'NEXT_YEAR'  => DatePeriod::NEXT_YEAR,
-            'THIS_MONTH' => DatePeriod::THIS_MONTH,
-            'THIS_WEEK'  => DatePeriod::THIS_WEEK,
-            'THIS_YEAR'  => DatePeriod::THIS_YEAR,
-        ];
-    }
-
-    /**
-     * {@inheritdoc}
-     */
-    protected function getTestedTypeInstance()
-    {
-        return new DatePeriod();
-    }
-
-    /**
-     * {@inheritdoc}
-     */
-    public function provideTypeToVerify()
-    {
-        yield[
-            '',
-            false,
-        ];
-
-        yield[
-            -1,
-            false,
-        ];
-
-        yield[
-            true,
-            false,
-        ];
-
-        yield[
-            DatePeriod::LAST_MONTH,
-            true,
-        ];
-
-        yield[
-            DatePeriod::NEXT_WEEK,
-            true,
-        ];
-
-        yield[
-            DatePeriod::THIS_YEAR,
-            true,
-        ];
-    }
 }

tests/Utilities/DateTest.php

@@ -11,16 +11,15 @@ namespace Meritoo\Common\Test\Utilities;
 use DateInterval;
 use DateTime;
 use Generator;
-use Meritoo\Common\Exception\Type\UnknownDatePartTypeException;
+use Meritoo\Common\Exception\Date\UnknownDatePartTypeException;
 use Meritoo\Common\Test\Base\BaseTestCase;
-use Meritoo\Common\Type\DatePeriod;
 use Meritoo\Common\Utilities\Date;
 
 /**
  * Test case of the Date methods (only static functions)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class DateTest extends BaseTestCase
 {
@@ -507,35 +506,6 @@ class DateTest extends BaseTestCase
         self::assertTrue($randomDate >= $intervalMinDate && $randomDate <= $intervalMaxDate);
     }
 
-    /**
-     * @param mixed $period Empty value, e.g. ""
-     * @dataProvider provideEmptyValue
-     */
-    public function testGetDatesForPeriodUsingEmptyPeriod($period)
-    {
-        self::assertNull(Date::getDatesForPeriod($period));
-    }
-
-    /**
-     * @param int $period Incorrect period to verify
-     * @dataProvider provideIncorrectPeriod
-     */
-    public function testGetDatesForPeriodUsingIncorrectPeriod($period)
-    {
-        self::assertNull(Date::getDatesForPeriod($period));
-    }
-
-    /**
-     * @param int        $period   The period, type of period. One of DatePeriod class constants, e.g. DatePeriod::LAST_WEEK.
-     * @param DatePeriod $expected Expected start and end date for given period
-     *
-     * @dataProvider provideCorrectPeriod
-     */
-    public function testGetDatesForPeriod($period, DatePeriod $expected)
-    {
-        self::assertEquals($expected, Date::getDatesForPeriod($period));
-    }
-
     /**
      * Provides incorrect invalidCount of DateTime
      *
@@ -788,112 +758,4 @@ class DateTest extends BaseTestCase
             50,
         ];
     }
-
-    /**
-     * Provides incorrect period
-     *
-     * @return Generator
-     */
-    public function provideIncorrectPeriod()
-    {
-        yield[-1];
-        yield[0];
-        yield[10];
-    }
-
-    /**
-     * Provides correct period
-     *
-     * @return Generator
-     */
-    public function provideCorrectPeriod()
-    {
-        yield[
-            DatePeriod::LAST_WEEK,
-            new DatePeriod(
-                (new DateTime('this week'))->sub(new DateInterval('P7D'))->setTime(0, 0, 0),
-                (new DateTime('this week'))->sub(new DateInterval('P1D'))->setTime(23, 59, 59)
-            ),
-        ];
-
-        yield[
-            DatePeriod::THIS_WEEK,
-            new DatePeriod(
-                (new DateTime('this week'))->setTime(0, 0, 0),
-                (new DateTime('this week'))->add(new DateInterval('P6D'))->setTime(23, 59, 59)
-            ),
-        ];
-
-        yield[
-            DatePeriod::NEXT_WEEK,
-            new DatePeriod(
-                (new DateTime('this week'))->add(new DateInterval('P7D'))->setTime(0, 0, 0),
-                (new DateTime('this week'))->add(new DateInterval('P7D'))->add(new DateInterval('P6D'))->setTime(23, 59, 59)
-            ),
-        ];
-
-        yield[
-            DatePeriod::LAST_MONTH,
-            new DatePeriod(
-                (new DateTime('first day of last month'))->setTime(0, 0, 0),
-                (new DateTime('last day of last month'))->setTime(23, 59, 59)
-            ),
-        ];
-
-        yield[
-            DatePeriod::THIS_MONTH,
-            new DatePeriod(
-                Date::getDatesForPeriod(DatePeriod::LAST_MONTH)
-                    ->getEndDate()
-                    ->add(new DateInterval('P1D'))
-                    ->setTime(0, 0, 0),
-                Date::getDatesForPeriod(DatePeriod::NEXT_MONTH)
-                    ->getStartDate()
-                    ->sub(new DateInterval('P1D'))
-                    ->setTime(23, 59, 59)
-            ),
-        ];
-
-        yield[
-            DatePeriod::NEXT_MONTH,
-            new DatePeriod(
-                (new DateTime('first day of next month'))->setTime(0, 0, 0),
-                (new DateTime('last day of next month'))->setTime(23, 59, 59)
-            ),
-        ];
-
-        $lastYearStart = (new DateTime())->modify('-1 year');
-        $lastYearEnd = (new DateTime())->modify('-1 year');
-        $year = $lastYearStart->format('Y');
-
-        yield[
-            DatePeriod::LAST_YEAR,
-            new DatePeriod(
-                $lastYearStart->setDate($year, 1, 1)->setTime(0, 0, 0),
-                $lastYearEnd->setDate($year, 12, 31)->setTime(23, 59, 59)
-            ),
-        ];
-
-        $year = (new DateTime())->format('Y');
-
-        yield[
-            DatePeriod::THIS_YEAR,
-            new DatePeriod(
-                (new DateTime())->setDate($year, 1, 1)->setTime(0, 0, 0),
-                (new DateTime())->setDate($year, 12, 31)->setTime(23, 59, 59)
-            ),
-        ];
-
-        $nextYearStart = (new DateTime())->modify('1 year');
-        $nextYearEnd = (new DateTime())->modify('1 year');
-        $year = $nextYearStart->format('Y');
-
-        yield[
-            DatePeriod::NEXT_YEAR,
-            new DatePeriod(
-                $nextYearStart->setDate($year, 1, 1)->setTime(0, 0, 0),
-                $nextYearEnd->setDate($year, 12, 31)->setTime(23, 59, 59)
-            ),
-        ];
-    }
 }

tests/Utilities/GeneratorUtilityTest.php

@@ -14,8 +14,8 @@ use Meritoo\Common\Utilities\GeneratorUtility;
 /**
  * Test case of the useful methods for the Generator class
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class GeneratorUtilityTest extends BaseTestCase
 {

tests/Utilities/LocaleTest.php

@@ -11,19 +11,15 @@ namespace Meritoo\Common\Test\Utilities;
 use Generator;
 use Meritoo\Common\Test\Base\BaseTestCase;
 use Meritoo\Common\Utilities\Locale;
-use ReflectionException;
 
 /**
  * Test case of the useful locale methods
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class LocaleTest extends BaseTestCase
 {
-    /**
-     * @throws ReflectionException
-     */
     public function testConstructor()
     {
         static::assertHasNoConstructor(Locale::class);
@@ -44,7 +40,7 @@ class LocaleTest extends BaseTestCase
      * @param string $encoding     Encoding of the final locale
      * @param string $expected     Expected long form of the locale
      *
-     * @dataProvider provideLanguageEncodingAndCountryCode
+     * @dataProvider provideLanguageAndCountryCode
      */
     public function testGetLongForm($languageCode, $countryCode, $encoding, $expected)
     {
@@ -60,46 +56,24 @@ class LocaleTest extends BaseTestCase
         self::assertFalse(Locale::setLocale($emptyValue, $emptyValue));
     }
 
-    public function testSetLocaleIncorrectCategory()
-    {
-        self::assertFalse(Locale::setLocale(-1, 'en'));
-    }
-
     /**
      * @param int    $category     Named constant specifying the category of the functions affected by the locale
      *                             setting. It's the same constant as required by setlocale() function.
      * @param string $languageCode Language code, in ISO 639-1 format. Short form of the locale, e.g. "fr".
-     * @param string $countryCode    Country code, in ISO 3166-1 alpha-2 format, e.g. "FR"
-     * @param string $expectedLocale Expected locale
      *
-     * @dataProvider provideCategoryLanguageCodeAndExpectedLocale
+     * @dataProvider provideCategoryAndLanguageCode
      */
-    public function testSetLocale($category, $languageCode, $countryCode, $expectedLocale)
+    public function testSetLocale($category, $languageCode)
     {
-        self::assertEquals($expectedLocale, Locale::setLocale($category, $languageCode, $countryCode));
+        self::assertTrue(Locale::setLocale($category, $languageCode));
     }
 
     /**
-     * @param int    $category       Named constant specifying the category of the functions affected by the locale setting.
+     * Provides language and country code
-     *                               It's the same constant as required by setlocale() function.
-     * @param string $languageCode   Language code, in ISO 639-1 format. Short form of the locale, e.g. "fr".
-     * @param string $countryCode    Country code, in ISO 3166-1 alpha-2 format, e.g. "FR"
-     * @param string $expectedLocale Expected locale
-     *
-     * @dataProvider provideCategoryLanguageCodeAndExpectedLocale
-     */
-    public function testGetLocale($category, $languageCode, $countryCode, $expectedLocale)
-    {
-        Locale::setLocale($category, $languageCode, $countryCode);
-        self::assertEquals($expectedLocale, Locale::getLocale($category));
-    }
-
-    /**
-     * Provides language, encoding and country code
      *
      * @return Generator
      */
-    public function provideLanguageEncodingAndCountryCode()
+    public function provideLanguageAndCountryCode()
     {
         yield[
             'fr',
@@ -128,97 +102,33 @@ class LocaleTest extends BaseTestCase
             'UTF-8',
             'fr_FR.UTF-8',
         ];
-
-        yield[
-            'en',
-            'US',
-            '',
-            'en_US',
-        ];
-
-        yield[
-            'en',
-            'US',
-            'UTF-8',
-            'en_US.UTF-8',
-        ];
-
-        yield[
-            'en',
-            'US',
-            'ISO-8859-1',
-            'en_US.ISO-8859-1',
-        ];
     }
 
     /**
-     * Provides category
+     * Provides category and language
      *
      * @return Generator
      */
-    public function provideCategoryLanguageCodeAndExpectedLocale()
+    public function provideCategoryAndLanguageCode()
     {
         yield[
             LC_ALL,
             'fr',
-            '',
-            'fr_FR.UTF-8',
         ];
 
         yield[
             LC_COLLATE,
             'fr',
-            'FR',
-            'fr_FR.UTF-8',
         ];
 
         yield[
             LC_CTYPE,
             'en',
-            'US',
-            'en_US.UTF-8',
         ];
 
         yield[
             LC_NUMERIC,
             'en',
-            'GB',
-            'en_GB.UTF-8',
-        ];
-
-        yield[
-            LC_MONETARY,
-            'es',
-            '',
-            'es_ES.UTF-8',
-        ];
-
-        yield[
-            LC_MONETARY,
-            'es',
-            'ES',
-            'es_ES.UTF-8',
-        ];
-
-        yield[
-            LC_TIME,
-            'it',
-            '',
-            'it_IT.UTF-8',
-        ];
-
-        yield[
-            LC_TIME,
-            'it',
-            'IT',
-            'it_IT.UTF-8',
-        ];
-
-        yield[
-            LC_TIME,
-            'it',
-            'it',
-            'it_IT.UTF-8',
         ];
     }
 }

tests/Utilities/MimeTypesTest.php

@@ -15,8 +15,8 @@ use Meritoo\Common\Utilities\MimeTypes;
 /**
  * Test case of the useful methods for mime types of files
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class MimeTypesTest extends BaseTestCase
 {

tests/Utilities/MiscellaneousTest.php

@@ -14,25 +14,20 @@ use Meritoo\Common\Exception\Regex\InvalidColorHexValueException;
 use Meritoo\Common\Test\Base\BaseTestCase;
 use Meritoo\Common\Utilities\Locale;
 use Meritoo\Common\Utilities\Miscellaneous;
-use ReflectionException;
 use stdClass;
 
 /**
  * Test case of the Miscellaneous methods (only static functions)
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class MiscellaneousTest extends BaseTestCase
 {
     private $stringSmall;
     private $stringCommaSeparated;
     private $stringDotSeparated;
-    private $stringWithoutSpaces;
 
-    /**
-     * @throws ReflectionException
-     */
     public function testConstructor()
     {
         static::assertHasNoConstructor(Miscellaneous::class);
@@ -366,36 +361,37 @@ class MiscellaneousTest extends BaseTestCase
     {
         self::assertEquals('Lorem ipsum dolor sit<br>amet, consectetur<br>adipiscing<br>elit', Miscellaneous::breakLongText($this->stringCommaSeparated, 20));
         self::assertEquals('Lorem ipsum dolor sit---amet, consectetur---adipiscing---elit', Miscellaneous::breakLongText($this->stringCommaSeparated, 20, '---'));
-        self::assertEquals('LoremIpsum<br>DolorSitAm<br>etConsecte<br>turAdipisc<br>ingElit', Miscellaneous::breakLongText($this->stringWithoutSpaces, 10));
     }
 
-    public function testRemoveDirectoryUsingNotExistingDirectory()
+    public function testRemoveDirectory()
     {
-        self::assertNull(Miscellaneous::removeDirectory('/abc/def/ghi'));
+        /*
-    }
+         * Removing not existing directory
+         */
+        self::assertTrue(Miscellaneous::removeDirectory('/abc/def/ghi'));
 
-    public function testRemoveDirectoryUsingNoDirectory()
+        /*
-    {
+         * Removing not directory
+         */
         $directoryPath = sys_get_temp_dir() . '/ipsum.txt';
         touch($directoryPath);
         self::assertTrue(Miscellaneous::removeDirectory($directoryPath));
-    }
 
-    public function testRemoveDirectoryUsingSimpleDirectory()
+        /*
-    {
+         * Removing simple directory
+         */
         $directoryPath = sys_get_temp_dir() . '/lorem/ipsum';
         mkdir($directoryPath, 0777, true);
         self::assertTrue(Miscellaneous::removeDirectory($directoryPath));
-    }
 
-    public function testRemoveDirectoryUsingComplexDirectory()
+        /*
-    {
+         * Removing more complex directory
+         */
         $directory1Path = sys_get_temp_dir() . '/lorem/ipsum';
         $directory2Path = sys_get_temp_dir() . '/lorem/dolor/sit';
 
         mkdir($directory1Path, 0777, true);
         mkdir($directory2Path, 0777, true);
-
         self::assertTrue(Miscellaneous::removeDirectory(sys_get_temp_dir() . '/lorem', false));
     }
 
@@ -665,10 +661,6 @@ class MiscellaneousTest extends BaseTestCase
         self::assertEquals(255, Miscellaneous::getValidColorComponent(255, false));
     }
 
-    /**
-     * @throws IncorrectColorHexLengthException
-     * @throws InvalidColorHexValueException
-     */
     public function testGetInvertedColorWithIncorrectLength()
     {
         $this->setExpectedException(IncorrectColorHexLengthException::class);
@@ -683,10 +675,6 @@ class MiscellaneousTest extends BaseTestCase
         Miscellaneous::getInvertedColor('1234567');
     }
 
-    /**
-     * @throws IncorrectColorHexLengthException
-     * @throws InvalidColorHexValueException
-     */
     public function testGetInvertedColorWithInvalidValue()
     {
         $this->setExpectedException(InvalidColorHexValueException::class);
@@ -698,10 +686,6 @@ class MiscellaneousTest extends BaseTestCase
         Miscellaneous::getInvertedColor('00ppqq');
     }
 
-    /**
-     * @throws IncorrectColorHexLengthException
-     * @throws InvalidColorHexValueException
-     */
     public function testGetInvertedColor()
     {
         /*
@@ -1190,7 +1174,6 @@ class MiscellaneousTest extends BaseTestCase
         $this->stringSmall = 'Lorem ipsum dolor sit amet.';
         $this->stringCommaSeparated = 'Lorem ipsum dolor sit amet, consectetur adipiscing elit';
         $this->stringDotSeparated = 'Etiam ullamcorper. Suspendisse a pellentesque dui, non felis.';
-        $this->stringWithoutSpaces = 'LoremIpsumDolorSitAmetConsecteturAdipiscingElit';
     }
 
     /**
@@ -1203,6 +1186,5 @@ class MiscellaneousTest extends BaseTestCase
         unset($this->stringSmall);
         unset($this->stringCommaSeparated);
         unset($this->stringDotSeparated);
-        unset($this->stringWithoutSpaces);
     }
 }

tests/Utilities/QueryBuilderUtilityTest.php

@@ -1,341 +0,0 @@
-<?php
-
-/**
- * (c) Meritoo.pl, http://www.meritoo.pl
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
-namespace Meritoo\Common\Test\Utilities;
-
-use Doctrine\Common\Collections\ArrayCollection;
-use Doctrine\ORM\EntityManager;
-use Doctrine\ORM\EntityManagerInterface;
-use Doctrine\ORM\Query\Expr;
-use Doctrine\ORM\Query\Parameter;
-use Doctrine\ORM\QueryBuilder;
-use Generator;
-use Meritoo\Common\Test\Base\BaseTestCase;
-use Meritoo\Common\Utilities\QueryBuilderUtility;
-
-/**
- * Test case of the useful methods for query builder (the Doctrine's QueryBuilder class)
- *
- * @author    Meritoo <github@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
- */
-class QueryBuilderUtilityTest extends BaseTestCase
-{
-    public function testConstructor()
-    {
-        static::assertHasNoConstructor(QueryBuilderUtility::class);
-    }
-
-    /**
-     * @param QueryBuilder $queryBuilder The query builder to retrieve root alias
-     * @param null|string  $rootAlias    Expected root alias of given query builder
-     *
-     * @dataProvider provideQueryBuilderAndRootAlias
-     */
-    public function testGetRootAlias(QueryBuilder $queryBuilder, $rootAlias)
-    {
-        static::assertSame($rootAlias, QueryBuilderUtility::getRootAlias($queryBuilder));
-    }
-
-    /**
-     * @param QueryBuilder $queryBuilder  The query builder to verify
-     * @param string       $propertyName  Name of property that maybe is joined
-     * @param null|string  $propertyAlias Expected alias of given property joined in given query builder
-     *
-     * @dataProvider provideQueryBuilderAndPropertyAlias
-     */
-    public function testGetJoinedPropertyAlias(QueryBuilder $queryBuilder, $propertyName, $propertyAlias)
-    {
-        static::assertSame($propertyAlias, QueryBuilderUtility::getJoinedPropertyAlias($queryBuilder, $propertyName));
-    }
-
-    public function testSetCriteriaWithoutCriteria()
-    {
-        $entityManager = $this->getMock(EntityManagerInterface::class);
-        $queryBuilder = new QueryBuilder($entityManager);
-        $newQueryBuilder = QueryBuilderUtility::setCriteria($queryBuilder);
-
-        static::assertSame($queryBuilder, $newQueryBuilder);
-        static::assertCount(0, $newQueryBuilder->getParameters());
-        static::assertNull($newQueryBuilder->getDQLPart('where'));
-    }
-
-    public function testSetCriteriaWithoutAlias()
-    {
-        $criteria = [
-            'lorem' => 11,
-            'ipsum' => 22,
-        ];
-
-        $entityManager = $this->getMock(EntityManagerInterface::class);
-        $queryBuilder = new QueryBuilder($entityManager);
-        $newQueryBuilder = QueryBuilderUtility::setCriteria($queryBuilder, $criteria);
-
-        static::assertSame($queryBuilder, $newQueryBuilder);
-        static::assertCount(count($criteria), $newQueryBuilder->getParameters());
-        static::assertNotNull($newQueryBuilder->getDQLPart('where'));
-    }
-
-    /**
-     * @param QueryBuilder $queryBuilder The query builder
-     * @param array        $criteria     The criteria used in WHERE clause
-     *
-     * @dataProvider provideQueryBuilderAndCriteria
-     */
-    public function testSetCriteria(QueryBuilder $queryBuilder, array $criteria)
-    {
-        $newQueryBuilder = QueryBuilderUtility::setCriteria($queryBuilder, $criteria);
-        $criteriaCount = count($criteria);
-        $nullsCount = 0;
-
-        /*
-         * I have to verify count/amount of NULLs and decrease $criteriaCount, because for null parameter is not added
-         */
-        array_walk($criteria, function ($value) use (&$nullsCount) {
-            if (null === $value) {
-                ++$nullsCount;
-            }
-        });
-
-        static::assertSame($queryBuilder, $newQueryBuilder);
-        static::assertCount($criteriaCount - $nullsCount, $newQueryBuilder->getParameters());
-        static::assertNotNull($newQueryBuilder->getDQLPart('where'));
-    }
-
-    public function testDeleteEntitiesWithoutFlush()
-    {
-        $methods = [
-            'remove',
-            'flush',
-        ];
-
-        $entityManager = $this->getMock(EntityManager::class, $methods, [], '', false);
-        $entities1 = [];
-
-        $entities2 = [
-            new \stdClass(),
-        ];
-
-        static::assertFalse(QueryBuilderUtility::deleteEntities($entityManager, $entities1, false));
-        static::assertTrue(QueryBuilderUtility::deleteEntities($entityManager, $entities2, false));
-    }
-
-    public function testDeleteEntities()
-    {
-        $methods = [
-            'remove',
-            'flush',
-        ];
-
-        $entityManager = $this->getMock(EntityManager::class, $methods, [], '', false);
-        $entities1 = [];
-
-        $entities2 = [
-            new \stdClass(),
-        ];
-
-        static::assertFalse(QueryBuilderUtility::deleteEntities($entityManager, $entities1));
-        static::assertTrue(QueryBuilderUtility::deleteEntities($entityManager, $entities2));
-    }
-
-    /**
-     * @param QueryBuilder          $queryBuilder The query builder
-     * @param array|ArrayCollection $parameters   Parameters to add. Collection of Doctrine\ORM\Query\Parameter
-     *                                            instances or an array with key-value pairs.
-     *
-     * @dataProvider provideQueryBuilderAndParameters
-     */
-    public function testAddParameters(QueryBuilder $queryBuilder, $parameters)
-    {
-        $newQueryBuilder = QueryBuilderUtility::addParameters($queryBuilder, $parameters);
-
-        static::assertSame($queryBuilder, $newQueryBuilder);
-        static::assertCount(count($parameters), $newQueryBuilder->getParameters());
-    }
-
-    /**
-     * Provides query builder to retrieve root alias and expected root alias
-     *
-     * @return Generator
-     */
-    public function provideQueryBuilderAndRootAlias()
-    {
-        $entityManager = $this->getMock(EntityManagerInterface::class);
-
-        yield[
-            new QueryBuilder($entityManager),
-            null,
-        ];
-
-        yield[
-            (new QueryBuilder($entityManager))->from('lorem_ipsum', 'lm'),
-            'lm',
-        ];
-
-        yield[
-            (new QueryBuilder($entityManager))
-                ->from('lorem', 'l')
-                ->leftJoin('l.ipsum', 'i'),
-            'l',
-        ];
-    }
-
-    /**
-     * Provides query builder, name of property and expected alias of given property
-     *
-     * @return Generator
-     */
-    public function provideQueryBuilderAndPropertyAlias()
-    {
-        $entityManager = $this->getMock(EntityManagerInterface::class);
-
-        yield[
-            new QueryBuilder($entityManager),
-            '',
-            null,
-        ];
-
-        yield[
-            new QueryBuilder($entityManager),
-            'lorem',
-            null,
-        ];
-
-        yield[
-            (new QueryBuilder($entityManager))->from('lorem_ipsum', 'lm'),
-            'lm',
-            null,
-        ];
-
-        yield[
-            (new QueryBuilder($entityManager))
-                ->from('lorem', 'l')
-                ->leftJoin('l.ipsum', 'i'),
-            'ipsum',
-            'i',
-        ];
-
-        yield[
-            (new QueryBuilder($entityManager))
-                ->from('lorem', 'l')
-                ->leftJoin('l.ipsum', 'i')
-                ->innerJoin('i.dolor', 'd'),
-            'ipsum1',
-            null,
-        ];
-
-        yield[
-            (new QueryBuilder($entityManager))
-                ->from('lorem', 'l')
-                ->leftJoin('l.ipsum', 'i')
-                ->innerJoin('i.dolor', 'd'),
-            'ipsum',
-            'i',
-        ];
-
-        yield[
-            (new QueryBuilder($entityManager))
-                ->from('lorem', 'l')
-                ->leftJoin('l.ipsum', 'i')
-                ->innerJoin('i.dolor', 'd'),
-            'dolor',
-            'd',
-        ];
-    }
-
-    /**
-     * Provides query builder and criteria used in WHERE clause
-     *
-     * @return Generator
-     */
-    public function provideQueryBuilderAndCriteria()
-    {
-        $entityManager = $this->getMock(EntityManager::class, ['getExpressionBuilder'], [], '', false);
-
-        $entityManager
-            ->expects(static::any())
-            ->method('getExpressionBuilder')
-            ->willReturn(new Expr());
-
-        yield[
-            (new QueryBuilder($entityManager))->from('lorem_ipsum', 'lm'),
-            [
-                'lorem' => 11,
-                'ipsum' => 22,
-                'dolor' => null,
-            ],
-        ];
-
-        yield[
-            (new QueryBuilder($entityManager))->from('lorem_ipsum', 'lm'),
-            [
-                'lorem' => [
-                    11,
-                    '>=',
-                ],
-                'ipsum' => [
-                    22,
-                    '<',
-                ],
-                'dolor' => null,
-            ],
-        ];
-    }
-
-    /**
-     * Provides query builder and parameters to add to given query builder
-     *
-     * @return Generator
-     */
-    public function provideQueryBuilderAndParameters()
-    {
-        $entityManager = $this->getMock(EntityManagerInterface::class);
-
-        yield[
-            new QueryBuilder($entityManager),
-            [],
-        ];
-
-        yield[
-            new QueryBuilder($entityManager),
-            new ArrayCollection(),
-        ];
-
-        yield[
-            new QueryBuilder($entityManager),
-            [
-                'lorem' => 11,
-                'ipsum' => 22,
-            ],
-        ];
-
-        yield[
-            new QueryBuilder($entityManager),
-            new ArrayCollection([
-                'lorem' => 11,
-                'ipsum' => 22,
-            ]),
-        ];
-
-        yield[
-            new QueryBuilder($entityManager),
-            [
-                new Parameter('lorem', 11),
-                new Parameter('ipsum', 22),
-            ],
-        ];
-
-        yield[
-            new QueryBuilder($entityManager),
-            new ArrayCollection([
-                new Parameter('lorem', 11),
-                new Parameter('ipsum', 22),
-            ]),
-        ];
-    }
-}

tests/Utilities/Reflection/A.php

@@ -12,8 +12,8 @@ namespace Meritoo\Common\Test\Utilities\Reflection;
  * The A class.
  * Used for testing the Reflection class.
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
 class A
 {

tests/Utilities/Reflection/B.php

@@ -12,10 +12,10 @@ namespace Meritoo\Common\Test\Utilities\Reflection;
  * The B class.
  * Used for testing the Reflection class.
  *
- * @author    Meritoo <github@meritoo.pl>
+ * @author    Krzysztof Niziol <krzysztof.niziol@meritoo.pl>
- * @copyright Meritoo <http://www.meritoo.pl>
+ * @copyright Meritoo.pl
  */
-class B extends A implements I
+class B extends A
 {
     protected $name = 'Lorem Ipsum';