Merged in master

Author: quentin.schmick

.gitattributes

@@ -11,3 +11,4 @@
 /tests              export-ignore
 /.editorconfig      export-ignore
 /.github            export-ignore
+/docker            export-ignore

.github/workflows/tests.yml

@@ -2,6 +2,7 @@ name: tests
 
 on:
     push:
+    pull_request:
     schedule:
         - cron: '0 0 * * *'
 
@@ -23,8 +24,9 @@ jobs:
         strategy:
             fail-fast: true
             matrix:
-                php: [7.3, 7.4]
-                stability: [prefer-lowest, prefer-stable]
+                php: [7.3, 7.4, 8.0, 8.1]
+                stability: 
+                  - prefer-stable
 
         name: PHP ${{ matrix.php }} - ${{ matrix.stability }}
 

README.md

@@ -1,8 +1,9 @@
 # Laravel Process Stamps
 
-[![Latest Version on Packagist](https://img.shields.io/packagist/v/orisintel/laravel-process-stamps.svg?style=flat-square)](https://packagist.org/packages/orisintel/laravel-process-stamps)
-[![Build Status](https://img.shields.io/github/workflow/status/orisintel/laravel-process-stamps/tests?style=flat-square)](https://github.com/orisintel/laravel-process-stamps/actions?query=workflow%3Atests)
-[![Total Downloads](https://img.shields.io/packagist/dt/orisintel/laravel-process-stamps.svg?style=flat-square)](https://packagist.org/packages/orisintel/laravel-process-stamps)
+[![Latest Version on Packagist](https://img.shields.io/packagist/v/always-open/laravel-process-stamps.svg?style=flat-square)](https://packagist.org/packages/always-open/laravel-process-stamps)
+[![Build Status](https://img.shields.io/github/workflow/status/always-open/laravel-process-stamps/tests?style=flat-square)](https://github.com/always-open/laravel-process-stamps/actions?query=workflow%3Atests)
+[![Total Downloads](https://img.shields.io/packagist/dt/always-open/laravel-process-stamps.svg?style=flat-square)](https://packagist.org/packages/always-open/laravel-process-stamps)
+[![Maintainability](https://api.codeclimate.com/v1/badges/f7dfbbd4462347976259/maintainability)](https://codeclimate.com/github/always-open/laravel-process-stamps/maintainability)
 
 It is sometimes very useful to know which process created or modified a particular record in your database. This package provides a trait to add to your Laravel models which automatically logs that for you.
 
@@ -11,13 +12,13 @@ It is sometimes very useful to know which process created or modified a particul
 You can install the package via composer:
 
 ```bash
-composer require orisintel/laravel-process-stamps
+composer require always-open/laravel-process-stamps
 ```
 
 ## Configuration
 
 ``` php
-php artisan vendor:publish --provider="\OrisIntel\ProcessStamps\ProcessStampsServiceProvider"
+php artisan vendor:publish --provider="\AlwaysOpen\ProcessStamps\ProcessStampsServiceProvider"
 ```
 
 Running the above command will publish both the migration and the config file.
@@ -59,13 +60,32 @@ That will generate the nessesary field names to track processes.
 composer test
 ```
 
+## Using Docker
+All assets are set up under the docker-compose.yml file. The first time you run the docker image you must build it with 
+the following command:
+```bash
+docker-compose build
+```
+
+Then you can bring it up in the background using:
+```bash
+docker-compose up -d
+```
+
+And the image is aliased so you can access its command line via:
+```bash
+docker exec -it processes-stamp-app /bin/bash
+```
+
+From there you can run the tests within an isolated environment
+
 ## Contributing
 
 Please see [CONTRIBUTING](CONTRIBUTING.md) for details.
 
 ### Security
 
-If you discover any security related issues, please email [[email protected]](mailto:[email protected]) instead of using the issue tracker.
+If you discover any security related issues, please email @tomschlick or @qschmick directly instead of using the issue tracker.
 
 ## Credits
 

SECURITY.md

@@ -2,4 +2,4 @@
 
 ## Reporting a Vulnerability
 
-If you encounter what you believe to be a security vulnerability, please report it immediately to [email protected]. Please include your name, your email address, the url of the package, and a description of the suspected issue. 
+If you encounter what you believe to be a security vulnerability, please report it immediately to @tomschlick or @qschmick. Please include your name, your email address, the url of the package, and a description of the suspected issue. 

composer.json

@@ -9,32 +9,23 @@
     ],
     "homepage": "https://github.com/always-open/laravel-process-stamps",
     "license": "MIT",
-    "authors": [
-        {
-            "name": "Tom Schlick",
-            "email": "[email protected]",
-            "role": "Developer"
-        }
-    ],
     "require": {
         "php": "^8.0.0|^8.1.0",
         "laravel/framework": "^9.0"
     },
     "require-dev": {
-        "doctrine/dbal": "^3.3",
-        "larapack/dd": "^1.0",
-        "mockery/mockery": "~1.0",
+        "doctrine/dbal": "^2.9|^3.0",
         "orchestra/testbench": "^7.0",
         "phpunit/phpunit": "^9.0"
     },
     "autoload": {
         "psr-4": {
-            "OrisIntel\\ProcessStamps\\": "src"
+            "AlwaysOpen\\ProcessStamps\\": "src"
         }
     },
     "autoload-dev": {
         "psr-4": {
-            "OrisIntel\\ProcessStamps\\Tests\\": "tests"
+            "AlwaysOpen\\ProcessStamps\\Tests\\": "tests"
         }
     },
     "scripts": {
@@ -48,7 +39,7 @@
     "extra": {
         "laravel": {
             "providers": [
-                "OrisIntel\\ProcessStamps\\ProcessStampsServiceProvider"
+                "AlwaysOpen\\ProcessStamps\\ProcessStampsServiceProvider"
             ]
         }
     }

docker-compose.yml

@@ -4,8 +4,8 @@ services:
     build:
       context: ./
       dockerfile: ./docker/Dockerfile
-    image: processstamps
-    container_name: processstamps-app
+    image: processes-stamp
+    container_name: processes-stamp-app
     restart: unless-stopped
     working_dir: /var/www/
     volumes:

src/ProcessStamp.php

@@ -1,12 +1,12 @@
 <?php
 
-namespace OrisIntel\ProcessStamps;
+namespace AlwaysOpen\ProcessStamps;
 
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\ModelNotFoundException;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
 use Illuminate\Database\Eloquent\Relations\HasMany;
 use Illuminate\Support\Facades\Cache;
-use Illuminate\Support\Facades\DB;
 
 class ProcessStamp extends Model
 {
@@ -42,6 +42,8 @@ public function getTable() : string
      * @param null|string $hash
      *
      * @return ProcessStamp
+     *
+     * @throws ModelNotFoundException
      */
     public static function firstOrCreateByProcess(array $process, ?string $hash = null) : self
     {
@@ -59,23 +61,31 @@ public static function firstOrCreateByProcess(array $process, ?string $hash = nu
             $parent = static::firstOrCreateByProcess(static::getProcessName($process['type'], $process['parent_name']));
         }
 
-        $stamp = static::where('hash', $hash)->first();
+        return retry(4, function() use ($hash, $process, $parent) {
+            $stamp = static::firstWhere('hash', $hash);
+
+            /*
+             * If stamp does not exist in the database yet, go ahead and obtain a lock to create it.
+             * This specifically doesn't lock as the first step to avoid all calls obtaining a lock from the cache if
+             * the item already exists in the DB.
+             */
+            if (! $stamp) {
+                Cache::lock('process-stamps-hash-create-' . $hash, 10)
+                    ->get(function () use (&$stamp, $hash, $process, $parent) {
+                        $stamp = static::firstOrCreate(['hash' => $hash], [
+                            'name'      => trim($process['name']),
+                            'type'      => $process['type'],
+                            'parent_id' => optional($parent)->getKey(),
+                        ]);
+                    });
+            }
 
-        /*
-         * If stamp does not exist in the database yet, go ahead and obtain a lock to create it.
-         * This specifically doesn't lock as the first step to avoid all calls obtaining a lock from the cache if the item already exists in the DB.
-         */
-        if (! $stamp) {
-            Cache::lock('process-stamps-hash-create-' . $hash, 10)->get(function () use (&$stamp, $hash, $process, $parent) {
-                $stamp = static::firstOrCreate(['hash' => $hash], [
-                    'name'      => trim($process['name']),
-                    'type'      => $process['type'],
-                    'parent_id' => optional($parent)->getKey(),
-                ]);
-            });
-        }
+            if (null === $stamp) {
+                throw new ModelNotFoundException();
+            }
 
-        return $stamp;
+            return $stamp;
+        }, 25);
     }
 
     /**

src/ProcessStampable.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace OrisIntel\ProcessStamps;
+namespace AlwaysOpen\ProcessStamps;
 
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
 

src/ProcessStampsServiceProvider.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace OrisIntel\ProcessStamps;
+namespace AlwaysOpen\ProcessStamps;
 
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Arr;

tests/Fakes/Post.php

@@ -1,9 +1,9 @@
 <?php
 
-namespace OrisIntel\ProcessStamps\Tests\Fakes;
+namespace AlwaysOpen\ProcessStamps\Tests\Fakes;
 
 use Illuminate\Database\Eloquent\Model;
-use OrisIntel\ProcessStamps\ProcessStampable;
+use AlwaysOpen\ProcessStamps\ProcessStampable;
 
 class Post extends Model
 {