Merge branch '3.x' into support-nested-blocks-validation

Author: ifox

Diff for: .github/workflows/main.yml

@@ -108,7 +108,7 @@ jobs:
         id: composer-cache
         run: |
           echo "::set-output name=dir::$(composer config cache-files-dir)"
-      - uses: actions/cache@v2
+      - uses: actions/cache@v4
         with:
           path: ${{ steps.composer-cache.outputs.dir }}
           key: ${{ matrix.os }}-${{ matrix.laravel }}-${{ matrix.php }}-${{ matrix.dbal }}-composer-${{ hashFiles('**/composer.lock') }}
@@ -129,10 +129,6 @@ jobs:
         run: |
           npm ci
 
-      - name: Create blocks dir
-        run: |
-          mkdir ./frontend/js/components/blocks/customs
-
       - name: Build twill.
         run: ./vendor/bin/testbench twill:build --forTesting
         env:

Diff for: .github/workflows/release.yml

@@ -0,0 +1,61 @@
+name: Release
+
+on:
+  workflow_dispatch:
+    inputs:
+      version_bump_type:
+        description: 'Type of version bump (patch, minor, major)'
+        required: true
+        default: 'patch'
+
+jobs:
+  build-and-pr:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v3
+        with:
+          ref: 3.x
+      
+      - name: Setup Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: '20'
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Build project
+        run: npm run build
+
+      - name: Copy build artifacts
+        run: |
+          mkdir -p twill-assets/assets
+          rm -rf twill-assets/assets/*
+          cp -r dist/assets/* twill-assets/assets/
+
+      - name: Bump version
+        id: bump_version
+        run: |
+          npm version ${{ github.event.inputs.version_bump_type }} --no-git-tag-version
+          echo "::set-output name=new_version::$(node -p "require('./package.json').version")"
+      
+      - name: Update version in PHP file
+        run: |
+          NEW_VERSION="${{ steps.bump_version.outputs.new_version }}"
+          sed -i "s/\(public const VERSION = '\)[^']*\(';\)/\1${NEW_VERSION}\2/" src/TwillServiceProvider.php
+
+      - name: Create Pull Request
+        uses: peter-evans/create-pull-request@v5
+        with:
+          token: ${{ secrets.GITHUB_TOKEN }}
+          commit-message: |
+            Update assets and bump version to ${{ steps.bump_version.outputs.new_version }}
+          title: |
+            Twill ${{ steps.bump_version.outputs.new_version }} release
+          body: |
+            This PR updates the build and bumps the version number to ${{ steps.bump_version.outputs.new_version }}.
+          branch: release-${{ steps.bump_version.outputs.new_version }}
+          base: 3.x

Diff for: CHANGELOG.md

@@ -2,6 +2,65 @@
 
 All notable changes to `twill` will be documented in this file.
 
+## 3.5.2
+
+### Fixed
+
+- Fix cropper regression (likely a from a recent browser update) by [@13twelve](https://github.com/13twelve) in [#2744](https://github.com/area17/twill/pull/2744)
+
+
+## 3.5.1
+
+### Fixed
+
+- Fix file library uploader regression by [@joyceverheije](https://github.com/joyceverheije) in https://github.com/area17/twill/pull/2740
+- Fix `timeOnly` variant of the `Datepicker` field by [@MamlukiSn](https://github.com/MamlukiSn) in https://github.com/area17/twill/pull/2739 
+
+### Improved
+
+- Add a warning during the `twill:update` and `twill:build` commands for developers that have `twill.load_default_migrations` set to false by [@ifox](https://github.com/ifox) in [de274175](https://github.com/area17/twill/commit/de274175)
+
+## 3.5.0
+
+### Added
+
+- Add a `ModuleController::setPreviewView()` method by @zachgarwood in https://github.com/area17/twill/pull/2724
+- Add a `disableCrop()` method to `medias` fields by @ifox in https://github.com/area17/twill/pull/2686
+
+### Fixed
+
+- Fix position management in `medias` and `files` fields by @zeezo887 in https://github.com/area17/twill/pull/2694
+- Fix issues with touch actions by @13twelve in https://github.com/area17/twill/pull/2713
+- Fix issue with `Relation` column by @zachgarwood in https://github.com/area17/twill/pull/2720
+- Fix endpoints initialization for `Browser` component by @zeezo887 in https://github.com/area17/twill/pull/2722
+- Fix image cropper for erroneous EXIF orientation: Use JS to read image dimensions on upload by @13twelve in https://github.com/area17/twill/pull/2705
+- Fix square crops having mismatching width/height values by @13twelve in https://github.com/area17/twill/pull/2706
+- Fix `hideActiveCrop()` not working for medias form field when max is greater than 1 by @zeezo887 in https://github.com/area17/twill/pull/2686
+- Fix #2641: `InlineRepeater` updates by @13twelve in https://github.com/area17/twill/pull/2714
+- Fix Vue dropdown console warnings: update useCapture param for removeEventListener to match addEventListener by @zeezo887 in https://github.com/area17/twill/pull/2687
+- Fix #2657: adds missing Vue draggable deprecations by @13twelve in https://github.com/area17/twill/pull/2707
+
+### Docs
+
+- Added instructions to disable publish switch in create modal by @ordigital in https://github.com/area17/twill/pull/2698
+- Added instructions for `MultiSelect` field with dynamic values in form builder by @ordigital in https://github.com/area17/twill/pull/2699
+- Added instructions for free cropping with `null` or `0` ratio value by @LucaRed in https://github.com/area17/twill/pull/2715
+
+### Translations
+
+- Fix #2619: adds missing iso languages by @13twelve in https://github.com/area17/twill/pull/2708
+
+## 3.4.1
+
+### Improved
+
+- Allow media and file library disk configuration using an environement variable by [@antonioribeiro](https://github.com/antonioribeiro) in https://github.com/area17/twill/pull/2676
+
+### Fixed
+
+- Fix #2671: 3.4.0 regression on related browsers previews by [@Tofandel](https://github.com/Tofandel) in https://github.com/area17/twill/pull/2672
+- Fix #2674: 3.4.0 regression on relation column using a one-to-one relationship by [@Tofandel](https://github.com/Tofandel) in https://github.com/area17/twill/pull/2675
+
 ## 3.4.0
 
 ### Added
@@ -55,6 +114,12 @@ All notable changes to `twill` will be documented in this file.
 - Bump body-parser and express by [@dependabot](https://github.com/dependabot) in https://github.com/area17/twill/pull/2659
 - Bump cookie and express by [@dependabot](https://github.com/dependabot) in https://github.com/area17/twill/pull/2664
 
+## 3.3.2
+ 
+### Fixed
+ 
+- Backport fix from 3.5.1: `timeOnly` variant of the `Datepicker` field by [@MamlukiSn](https://github.com/MamlukiSn) in https://github.com/area17/twill/pull/2739
+
 ## 3.3.1
 
 ### Fixed

Diff for: composer.json

@@ -48,9 +48,9 @@
     "area17/phptorch": "dev-main",
     "chillerlan/php-qrcode": "~4.0",
     "friendsofphp/php-cs-fixer": "^3.0",
+    "larastan/larastan": "^2.9",
     "nette/php-generator": "^4.0.3",
     "nunomaduro/collision": "^6.0 || ^7.0 || ^8.0",
-    "nunomaduro/larastan": "^2.0",
     "orchestra/testbench": "^7.8 || ^8.0 || ^9.0",
     "orchestra/testbench-dusk": "^7.8 || ^8.0 || ^9.0",
     "phpunit/php-invoker": "^3.1 || ^4.0",

Diff for: config/file_library.php

@@ -18,7 +18,7 @@
     | - 'A17\Twill\Services\FileLibrary\Disk'
     |
      */
-    'disk' => 'twill_file_library',
+    'disk' => env('FILE_LIBRARY_DISK', 'twill_file_library'),
     'endpoint_type' => env('FILE_LIBRARY_ENDPOINT_TYPE', 'local'),
     'cascade_delete' => env('FILE_LIBRARY_CASCADE_DELETE', false),
     'local_path' => env('FILE_LIBRARY_LOCAL_PATH', 'uploads'),

Diff for: config/media_library.php

@@ -20,7 +20,7 @@
     | - 'A17\Twill\Services\MediaLibrary\Local'
     |
      */
-    'disk' => 'twill_media_library',
+    'disk' => env('MEDIA_LIBRARY_DISK', 'twill_media_library'),
     'endpoint_type' => env('MEDIA_LIBRARY_ENDPOINT_TYPE', 'local'),
     'cascade_delete' => env('MEDIA_LIBRARY_CASCADE_DELETE', false),
     'local_path' => env('MEDIA_LIBRARY_LOCAL_PATH', 'uploads'),

Diff for: docs/content/1_docs/4_form-fields/06_multi-select.md

@@ -200,21 +200,38 @@ public function sectors() {
 - In your repository, make sure to sync the association when saving:
 
 ```php
-public function afterSave($object, $fields)
+public function afterSave($object, $fields): void
 {
     $object->sectors()->sync($fields['sectors'] ?? []);
 
     parent::afterSave($object, $fields);
 }
 ```
 
+:::tabs=currenttab.FormBuilder&items.FormBuilder|FormView:::
+:::tab=name.FormBuilder:::
+
+```php
+        $form->add(
+            MultiSelect::make()
+            ->name('sectors')
+            ->options(
+                Options::fromArray(app()->make(SectorsRepository::class)->listAll()->toArray())
+              )
+        );
+```
+
+:::#tab:::
+
+:::tab=name.FormView:::
+
 - In your controller, add to the formData the collection of options:
 
 ```php
 protected function formData($request)
 {
     return [
-        'sectors' => app()->make(SectorRepository::class)->listAll()
+        'sectors' => app()->make(SectorRepository::class)->listAll()->toArray()
     ];
 }
 ```
@@ -229,6 +246,9 @@ protected function formData($request)
 />
 ```
 
+:::#tab:::
+:::#tabs:::
+
 When used in a [block](../5_block-editor), no migration is needed.
 
 ## Multi Select Inline

Diff for: docs/content/1_docs/4_form-fields/11_repeater.md

@@ -163,6 +163,17 @@ class ProjectController extends BaseModuleController
 }
 ```
 
+| Option         | Description                                  | Type    | Default value    |
+|:---------------|:---------------------------------------------|:--------|:-----------------|
+| type           | Type of repeater items                       | string  |                  |
+| name           | Name of the field                            | string  | same as `type`   |
+| max            | Maximum amount that can be created           | number  | null (unlimited) |
+| buttonAsLink   | Displays the `Add` button as a centered link | boolean | false            |
+| disableCreate  | Disables ability to add new items            | boolean | false            |
+| disableActions | Removes row item actions                     | boolean | false            |
+| disableReorder | Disables reordering of repeater items        | boolean | false            |
+
+
 ## Blade repeater fields
 
 The following example demonstrates how to define a relationship between `Team` and `TeamMember` modules to implement

Diff for: docs/content/1_docs/5_block-editor/02_creating-a-block-editor.md

@@ -393,6 +393,8 @@ See also [Default Configuration](./11_default-configuration.md).
 
 If the `resource_path('assets/admin/icons')` directory contains a `my-custom-icon.svg` file, you can use this icon in your block by using its basename: `@twillBlockIcon('my-custom-icon')`.
 
+In order to make the icons appear in the CMS, you'll need to run `php artisan twill:build`
+
 ## Use Block traits in your Model and Repository
 
 Now, to handle the block data you must integrate it with your module. *Use* the *Blocks* traits in the Model and Repository associated with your module.

Diff for: docs/content/1_docs/7_media-library/03_role-crop-params.md

@@ -6,7 +6,7 @@ A _role_ is a way to define different contexts in which an image might be placed
 
 _Crops_ are more self-explanatory. Twill comes with some pre-defined crop settings to allow you to set different variants of a given image, so crops can be used in combination with _roles,_ or they can be used on their own with a single role to define multiple cropping ratios on the same image.
 
-Using the Person example, your `cover` image could have a `square` crop for mobile screens, but could use a `16/9` crop on larger screens. Those values are editable at your convenience for each model, even if there are already some crops created in the CMS.
+Using the Person example, your `cover` image could have a `square` crop for mobile screens, but could use a `16/9` crop on larger screens. Those values are editable at your convenience for each model in the `ratio` option, even if there are already some crops created in the CMS. By specifying `null` or `0` as a `ratio`, no automatic crop will happen and you'll be able to crop freely.
 
 The only thing you have to do to make it work is to compose your model and repository with the appropriate traits, respectively `HasMedias` and `HandleMedias`, set up your `$mediasParams` configuration and use the `medias` form partial in your form view (more info in the CRUD section).
 

Diff for: docs/content/2_guides/3_adding-fields-to-the-create-modal.md

@@ -16,7 +16,7 @@ Often you might want to add some mandatory fields to the create modal of your mo
 In our module controller we can override the `getCreateForm` method and add the fields to render:
 
 :::filename:::
-`app/Http/Controllers/Twill/YourModuleController.php`
+`app/Http/Controllers/Twill/BlogController.php`
 :::#filename:::
 
 ```phptorch
@@ -168,3 +168,88 @@ Result:
 
 :::#tab:::
 :::#tabs:::
+
+# Removing publish switch from create modal
+
+In our module controller we can override the `indexData` to remove publish switch from modal. We can also override `setUpController` method to disable language select box used for making a translation active.
+
+:::filename:::
+`app/Http/Controllers/Twill/BlogController.php`
+:::#filename:::
+
+```phptorch
+##CODE##
+<?php
+
+namespace App\Http\Controllers\Twill;
+
+use A17\Twill\Http\Controllers\Admin\NestedModuleController as BaseModuleController;
+use A17\Twill\Services\Forms\Fields\Input;
+use A17\Twill\Services\Forms\Fields\Wysiwyg;
+use A17\Twill\Services\Forms\Form;
+
+class BlogController extends BaseModuleController
+{
+    protected $moduleName = 'blogs';
+
+    public function getCreateForm(): Form
+    {
+        ...
+    }
+
+    protected function setUpController(): void
+    {
+        $this->disablePublish();
+        $this->disableBulkPublish();
+        // $this->disableEditor(); # uncomment this to disable full editor
+        // $this->enableEditInModal(); # uncomment this to enable editing in modal
+
+    }
+
+    protected function formData($request) 
+    { 
+        return [ 'controlLanguagesPublication' => false ]; # disable select box to make language active
+    }
+}
+```
+
+If we also want to make all languages active by default, we can override `prepareFieldsBeforeCreate` function in our module repository:
+
+:::filename:::
+`app/Repositories/BlogRepository.php`
+:::#filename:::
+
+```phptorch
+##CODE##
+<?php
+
+namespace App\Repositories;
+
+use A17\Twill\Repositories\Behaviors\HandleTranslations;
+use A17\Twill\Repositories\Behaviors\HandleSlugs;
+use A17\Twill\Repositories\ModuleRepository;
+use App\Models\Category;
+
+class BlogRepository extends ModuleRepository
+{
+    use HandleTranslations, HandleSlugs;
+
+    public function __construct(Category $model)
+    {
+        $this->model = $model;
+    }
+
+    ### make all languages active for this model
+    public function prepareFieldsBeforeCreate($fields): array
+    {
+        foreach ($fields['languages'] as $key => $language) {
+            $fields['languages'][$key]['published'] = true;
+        }
+
+        return parent::prepareFieldsBeforeCreate($fields); // @phpstan-ignore-line
+    }
+
+}
+```
+
+![Customized create modal without publish switch](./assets/customized-without-publish-switch.png)

Diff for: docs/content/2_guides/4_creating_custom_components_form_fields_and_blocks.md

@@ -409,7 +409,7 @@ npm install vue-numeric --save
 ### Blade view
 
 :::filename:::
-`resources/views/admin/partials/form/_custom_number.blade.php`
+`resources/views/twill/partials/form/_custom_number.blade.php`
 :::#filename:::
 
 ```blade

Diff for: docs/content/2_guides/assets/customized-without-publish-switch.png

@@ -189,13 +189,12 @@
         //
         // from my testing it seems to be a little inconsistent and unpredictable
         // I guess you just need for the rounding error to happen
-        // But, it seems setting the properties individually avoids this...
+        // But, it seems re-setting the properties individually avoids this...
         //
         // -- Mike ([email protected])
+        this.cropper.setData(crop)
         this.cropper.setData({ x: crop.x })
         this.cropper.setData({ y: crop.y })
-        this.cropper.setData({ width: crop.width })
-        this.cropper.setData({ height: crop.height })
       },
       test: function () {
         const crop = this.toNaturalCrop({ x: 0, y: 0, width: 380, height: 475 })