Merge pull request #1 from Delta-Solutions/suggestion

Cleaned up first version

LICENSE.md

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) Delta-Solutions <andreas@deltasolutions.be>
+Copyright (c) Delta-Solutions <info@deltasolutions.be>
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

README.md

@@ -1,19 +1,7 @@
-# This is my package translations
+# Translation loader
 
-[![Latest Version on Packagist](https://img.shields.io/packagist/v/delta-solutions/translations.svg?style=flat-square)](https://packagist.org/packages/delta-solutions/translations)
-[![GitHub Tests Action Status](https://img.shields.io/github/actions/workflow/status/delta-solutions/translations/run-tests.yml?branch=main&label=tests&style=flat-square)](https://github.com/delta-solutions/translations/actions?query=workflow%3Arun-tests+branch%3Amain)
-[![GitHub Code Style Action Status](https://img.shields.io/github/actions/workflow/status/delta-solutions/translations/fix-php-code-style-issues.yml?branch=main&label=code%20style&style=flat-square)](https://github.com/delta-solutions/translations/actions?query=workflow%3A"Fix+PHP+code+style+issues"+branch%3Amain)
-[![Total Downloads](https://img.shields.io/packagist/dt/delta-solutions/translations.svg?style=flat-square)](https://packagist.org/packages/delta-solutions/translations)
+This package provides a migration and a model to extend the spatie/laravel-translation-loader so you can use columns for your labels.  One column per locale.
 
-This is where your description should go. Limit it to a paragraph or two. Consider adding a small example.
-
-## Support us
-
-[<img src="https://github-ads.s3.eu-central-1.amazonaws.com/translations.jpg?t=1" width="419px" />](https://spatie.be/github-ad-click/translations)
-
-We invest a lot of resources into creating [best in class open source packages](https://spatie.be/open-source). You can support us by [buying one of our paid products](https://spatie.be/open-source/support-us).
-
-We highly appreciate you sending us a postcard from your hometown, mentioning which of our package(s) you are using. You'll find our address on [our contact page](https://spatie.be/about-us). We publish all received postcards on [our virtual postcard wall](https://spatie.be/open-source/postcards).
 
 ## Installation
 
@@ -23,43 +11,25 @@ You can install the package via composer:
 composer require delta-solutions/translations
 ```
 
-You can publish and run the migrations with:
-
-```bash
-php artisan vendor:publish --tag="translations-migrations"
-php artisan migrate
-```
+Follow the installation steps as explained in the Spatie docs https://github.com/spatie/laravel-translation-loader
 
-You can publish the config file with:
+Define the locales in the `translation-loader.php` config file, one locale per database column.  This is used in the migration to create the database fields.
 
-```bash
-php artisan vendor:publish --tag="translations-config"
 ```
-
-This is the contents of the published config file:
-
-```php
-return [
-];
+ 'locales' => ['nl', 'fr', 'en', 'de']   
 ```
 
-Optionally, you can publish the views using
+You can publish and run the migrations with:
 
 ```bash
-php artisan vendor:publish --tag="translations-views"
+php artisan vendor:publish --tag="translations-migrations"
+php artisan migrate
 ```
 
-## Usage
+Use the model from this package as the model for the spatie/laravel-translation-loader. This is configured via the `model` in the config. Change it to
 
-```php
-$translations = new DeltaSolutions\Translations();
-echo $translations->echoPhrase('Hello, DeltaSolutions!');
 ```
-
-## Testing
-
-```bash
-composer test
+ 'model' => DeltaSolutions\Translations\Models\LanguageLine::class,
 ```
 
 ## Changelog

composer.json

@@ -18,19 +18,11 @@
     "require": {
         "php": "^8.1",
         "spatie/laravel-package-tools": "^1.14.0",
+        "spatie/laravel-translation-loader" : "^2.0",
         "illuminate/contracts": "^10.0"
     },
     "require-dev": {
-        "laravel/pint": "^1.0",
-        "nunomaduro/collision": "^7.9",
-        "nunomaduro/larastan": "^2.0.1",
-        "orchestra/testbench": "^8.0",
-        "pestphp/pest": "^2.0",
-        "pestphp/pest-plugin-arch": "^2.0",
-        "pestphp/pest-plugin-laravel": "^2.0",
-        "phpstan/extension-installer": "^1.1",
-        "phpstan/phpstan-deprecation-rules": "^1.0",
-        "phpstan/phpstan-phpunit": "^1.0"
+        "laravel/pint": "^1.0"
     },
     "autoload": {
         "psr-4": {
@@ -69,4 +61,4 @@
     },
     "minimum-stability": "dev",
     "prefer-stable": true
-}
+}

database/migrations/create_translations_table.php.stub → database/migrations/update_translations_table.php.stub

@@ -8,12 +8,10 @@ return new class extends Migration
 {
     public function up()
     {
-        Schema::create('translations_table', function (Blueprint $table) {
-            $table->id();
-
-            // add fields
-
-            $table->timestamps();
+        Schema::table('language_lines', function (Blueprint $table) {
+           collect(config('translation-loader.locales'))->each(function ($locale) use ($table) {
+               $table->text($locale);
+           });
         });
     }
 };