Optimize package

Author: parsilver

.github/workflows/tests.yml

@@ -0,0 +1,22 @@
+name: Test
+
+on:
+  push:
+    branches:
+      - main
+      - master
+
+jobs:
+  tests:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: true
+      matrix:
+        php: [ '7.2', '7.3', '7.4', '8.0', '8.1' ]
+    name: PHP ${{ matrix.php }}
+    steps:
+      - uses: actions/checkout@v2
+      - name: Install Composer
+        run: composer install
+      - name: Run Tests
+        run: composer test

.travis.yml

@@ -4,6 +4,15 @@
 
 [![Build Status](https://travis-ci.org/parsilver/thailand-provinces-php.svg?branch=master)](https://github.com/parsilver/thailand-provinces-php)
 
+
+## ความต้องการของระบบ
+```
+{
+    "php" : "^7.3||^8.0",
+}
+```
+
+
 ## การติดตั้ง
 
 ```sh

README.md

@@ -1,26 +1,34 @@
 {
     "name": "parsilver/thailand-provinces-php",
-    "description": "Provinces database in thailand.",
-    "keywords": ["province", "thailand", "amphure"],
+    "description": "Thailand address database",
+    "keywords": ["province", "thailand", "amphure", "address"],
     "type": "library",
     "license": "MIT",
     "authors": [
         {
-            "name": "Parkorn Soonthornchai",
-            "email": "magician_canobi@hotmail.com"
+            "name": "PA",
+            "role": "DevOps"
         }
     ],
     "minimum-stability": "stable",
     "prefer-stable": true,
     "require": {
-        "php": "^7.2"
+        "php": "^7.2||^8.0"
     },
     "require-dev": {
-        "phpunit/phpunit": "8.*"
+        "phpunit/phpunit": "^9.4"
     },
     "autoload": {
         "psr-4": {
             "PA\\ProvinceTh\\": "src/"
         }
+    },
+    "autoload-dev": {
+        "psr-4": {
+            "Farzai\\ThailandAddress\\Tests\\":  "tests"
+        }
+    },
+    "scripts": {
+        "test" : "vendor/bin/phpunit"
     }
 }

composer.json

@@ -12,7 +12,7 @@
 
     <testsuites>
         <testsuite name="Application Test Suite">
-            <directory suffix=".php">./tests/</directory>
+            <directory suffix="Test.php">./tests/</directory>
         </testsuite>
     </testsuites>
 

phpunit.xml

@@ -1,12 +1,14 @@
-<?php namespace PA\ProvinceTh;
+<?php
+
+namespace PA\ProvinceTh;
 
 use PA\ProvinceTh\Provider\Amphure;
 use PA\ProvinceTh\Provider\District;
 use PA\ProvinceTh\Provider\Geography;
 use PA\ProvinceTh\Provider\Province;
 
-class Factory {
-
+class Factory
+{
 
     /**
      * @return Geography
@@ -33,14 +35,11 @@ public static function amphure()
         return new Amphure();
     }
 
-
     /**
      * @return District
      */
     public static function district()
     {
         return new District();
     }
-
 }
-

src/Factory.php

@@ -1,9 +1,10 @@
-<?php namespace PA\ProvinceTh\Provider;
+<?php 
 
-
-class District extends ProviderCollection {
+namespace PA\ProvinceTh\Provider;
 
 
+class District extends ProviderCollection 
+{
     /**
      * @return array
      */
@@ -8933,6 +8934,4 @@ public function data()
             ['id' => '380803', 'zip_code' => '38000', 'name_th' => 'หนองเดิน', 'name_en' => 'Nong Doen', 'amphure_id' => '1006'],
         ];
     }
-
-
-}
+}

src/Provider/Amphure.php

@@ -1,7 +1,10 @@
-<?php namespace PA\ProvinceTh\Provider;
+<?php
 
+namespace PA\ProvinceTh\Provider;
 
-class Geography extends ProviderCollection  {
+
+class Geography extends ProviderCollection
+{
 
 
     /**
@@ -26,6 +29,4 @@ public function data()
             ['id' => '6', 'name' => 'ภาคใต้'],
         ];
     }
-
-
-}
+}

src/Provider/District.php

@@ -1,8 +1,11 @@
-<?php namespace PA\ProvinceTh\Provider;
+<?php
+
+namespace PA\ProvinceTh\Provider;
 
 use PA\ProvinceTh\Support\Collection;
 
-abstract class ProviderCollection extends Collection {
+abstract class ProviderCollection extends Collection
+{
 
     /**
      * @return array
@@ -37,10 +40,10 @@ public function find($id)
      */
     public function each(callable $callback)
     {
-        foreach($this->getItems() as $key => $value) {
+        foreach ($this->getItems() as $key => $value) {
             $callbackResult = $callback(new static($value), $key);
 
-            if($callbackResult === false) {
+            if ($callbackResult === false) {
                 break;
             }
         }
@@ -63,13 +66,13 @@ public function getPrimaryKey()
      */
     protected function hasMany($provider, $foreignKey = null)
     {
-        if($this->itemIsObject()) {
+        if ($this->itemIsObject()) {
             $this->setItems([$this->getItems()]);
         }
 
         $provider = new $provider();
 
-        if(is_null($foreignKey)) {
+        if (is_null($foreignKey)) {
             $foreignKey = strtolower($this->getClassName($this)) . '_id';
         }
 
@@ -84,13 +87,13 @@ protected function hasMany($provider, $foreignKey = null)
      */
     protected function belongsTo($provider, $localForeignKey = null)
     {
-        if($this->itemIsObject()) {
+        if ($this->itemIsObject()) {
             $this->setItems([$this->getItems()]);
         }
 
         $provider = new $provider();
 
-        if(is_null($localForeignKey)) {
+        if (is_null($localForeignKey)) {
             $localForeignKey = strtolower($this->getClassName($provider)) . '_id';
         }
 
@@ -113,4 +116,4 @@ private function itemIsObject()
     {
         return isset($this->getItems()[$this->getPrimaryKey()]);
     }
-}
+}