github-actions(deps): bump actions/checkout from 3 to 4

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Merge pull request #10 from netzmacht/dependabot/github_actions/actions/cache-3
2025-11-29 20:23:39 +01:00 · 2023-09-04 13:59:23 +00:00 · 2022-11-29 14:28:14 +01:00 · 2022-11-29 14:28:05 +01:00 · 2022-11-29 14:27:54 +01:00 · 2022-11-29 12:41:27 +00:00
24 changed files with 481 additions and 258 deletions
--- a/.composer-require-checker.json
+++ b/.composer-require-checker.json
@@ -0,0 +1,11 @@
+{
+  "symbol-whitelist": [
+    "null", "true", "false",
+    "static", "self", "parent",
+    "array", "string", "int", "float", "bool", "iterable", "callable", "void", "object",
+    "Contao\\ManagerPlugin\\Bundle\\BundlePluginInterface",
+    "Contao\\ManagerPlugin\\Bundle\\Config\\BundleConfig",
+    "Contao\\ManagerPlugin\\Bundle\\Parser\\ParserInterface",
+    "Contao\\ManagerPlugin\\Bundle\\Config\\BundleConfig"
+  ]
+}
--- a/.editorconfig
+++ b/.editorconfig
@@ -0,0 +1,17 @@
+root = true
+
+[*]
+charset = utf-8
+indent_style = space
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true
+
+[*.php]
+indent_size = 4
+
+[*.js]
+indent_size = 4
+
+[{*.vue,*.json,*.html5,*.xlf,*.twig}]
+indent_size = 2
--- a/.gitattributes
+++ b/.gitattributes
@@ -8,3 +8,8 @@ build.xml export-ignore
 phpunit.xml.dist export-ignore
 /tests export-ignore
 /specs export-ignore
+.phpcq.lock export-ignore
+.phpcq.yaml.dis export-ignore
+psalm.xml export-ignore
+phpcs.xml.dis export-ignore
+composer.lock export-ignore
--- a/.github/FUNDING.yml
+++ b/.github/FUNDING.yml
@@ -0,0 +1 @@
+github: [dmolineus]
--- a/.github/dependabot.yaml
+++ b/.github/dependabot.yaml
@@ -0,0 +1,27 @@
+# https://docs.github.com/en/github/administering-a-repository/configuration-options-for-dependency-updates
+
+version: 2
+
+updates:
+  - commit-message:
+      include: "scope"
+      prefix: "composer"
+    directory: "/"
+    labels:
+      - "dependency"
+    open-pull-requests-limit: 0
+    package-ecosystem: "composer"
+    schedule:
+      interval: "weekly"
+    versioning-strategy: "increase"
+
+  - commit-message:
+      include: "scope"
+      prefix: "github-actions"
+    directory: "/"
+    labels:
+      - "dependency"
+    open-pull-requests-limit: 10
+    package-ecosystem: "github-actions"
+    schedule:
+      interval: "weekly"
--- a/.github/workflows/diagnostics.yml
+++ b/.github/workflows/diagnostics.yml
@@ -0,0 +1,75 @@
+name: Code Quality Diagnostics
+
+on:
+  pull_request:
+  push:
+    branches:
+      - master
+      - develop
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    name: PHP ${{ matrix.php }}
+
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          - php: '7.4'
+            phpcq_install: 'install'
+            phpcq_flags: ''
+          - php: '8.0'
+            phpcq_install: 'update'
+            phpcq_flags: ''
+          - php: '8.1'
+            phpcq_install: 'update'
+            phpcq_flags: ''
+
+    steps:
+      - name: Pull source
+        uses: actions/checkout@v4
+
+      - name: Setup PHP with PECL extension
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          coverage: xdebug2
+
+      - name: Cache composer cache directory
+        uses: actions/cache@v3
+        env:
+          cache-name: composer-cache-dir-${{ matrix.php }}
+        with:
+          path: ~/.cache/composer
+          key: ${{ runner.os }}-build-${{ env.cache-name }}
+
+      - name: Install composer dependencies
+        run: composer install
+
+      - name: Cache vendor directory of phpcq tool runner
+        uses: actions/cache@v3
+        env:
+          cache-name: vendor-${{ matrix.php }}
+        with:
+          path: $GITHUB_WORKSPACE/vendor
+          key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/composer.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-build-${{ env.cache-name }}-
+
+      - name: Update phpcq
+        run: $GITHUB_WORKSPACE/vendor/bin/phpcq self-update --unsigned
+
+      - name: Install phpcq toolchain
+        run: $GITHUB_WORKSPACE/vendor/bin/phpcq ${{ matrix.phpcq_install }} -v
+
+      - name: Run tests
+        run: $GITHUB_WORKSPACE/vendor/bin/phpcq run -o github-action -o default ${{ matrix.phpcq_flags }} -v
+
+      - name: Upload build directory to artifact
+        uses: actions/upload-artifact@v3
+        if: ${{ success() }} || ${{ failure() }}
+        with:
+          name: phpcq-builds-php-${{ matrix.php }}
+          path: .phpcq/build/
--- a/.gitignore
+++ b/.gitignore
@@ -1,24 +1,10 @@
-# OS
-.DS_Store
-Thumbs.db
-.directory
-
-# IDEs 
-.buildpath
-.project
-.settings/
-.build/
-.external*/
-.idea/
-nbproject/
-
 # composer related
 vendor/
 composer.lock

-# build
-build/
-build.properties
-
 # Translations
 .tx
+
+# phpcq
+/.phpcq/
+.phpcs-cache
--- a/.phpcq.lock
+++ b/.phpcq.lock
--- a/.phpcq.yaml.dist
+++ b/.phpcq.yaml.dist
@@ -0,0 +1,96 @@
+phpcq:
+  repositories:
+    - https://phpcq.github.io/repository/repository.json
+  directories:
+    - src
+  artifact: .phpcq/build
+
+  plugins:
+    composer-require-checker:
+      version: ^1.0
+      signed: false
+    phpmd:
+      version: ^1.0
+      signed: false
+      requirements:
+        phpmd:
+          signed: false
+    phpcpd:
+      version: ^1.1
+      signed: false
+    phploc:
+      version: ^1.0
+      signed: false
+    phpcs:
+      version: ^1.0
+      signed: false
+    composer-normalize:
+      version: ^1.0
+      signed: false
+  trusted-keys:
+    # composer-require-checker
+    - 033E5F8D801A2F8D
+    # sb@sebastian-bergmann.de
+    - 4AA394086372C20A
+    # psalm
+    - 8A03EA3B385DBAA1
+    - 12CE0F1D262429A5
+    # magl@magll.net
+    - D2CCAC42F6295E7D
+    # PHP_CodeSniffer
+    - 31C7E470E2138192
+    # Composer normalize
+    - C00543248C87FB13
+    # phpmd
+    - 0F9684B8B16B7AB0
+    - A4E55EA12C7C085C
+
+tasks:
+  fix:
+    - composer-normalize-fix
+    - phpcbf
+
+  verify:
+    - composer-require-checker
+    - composer-normalize
+
+  analyze:
+    - phploc
+    - phpcpd
+    - phpmd
+    - phpcs
+#    - psalm
+
+  default:
+    - verify
+    - analyze
+
+  composer-require-checker:
+    config:
+      config_file: '.composer-require-checker.json'
+
+  phpcs:
+    plugin: phpcs
+    config: &phpcs-config
+      standard: ~
+      standard_paths:
+        - ./vendor/slevomat/coding-standard
+        - ./vendor/doctrine/coding-standard/lib
+      excluded:
+        - src/Resources/contao/languages
+
+  phpcbf:
+    plugin: phpcs
+    config:
+      <<: *phpcs-config
+      fix: true
+
+  composer-normalize:
+    config:
+      ignore_output:
+        - '#Root package is not of type "project", we will not installing Contao extensions\.#'
+
+  composer-normalize-fix:
+    plugin: composer-normalize
+    config:
+      dry_run: false
--- a/.travis.yml
+++ b/.travis.yml
@@ -1,34 +0,0 @@
-language: php
-
-php:
-  - "5.6"
-  - "7.0"
-  - "7.1"
-  - "7.2"
-
-env:
-  - CONTAO_VERSION=contao/core-bundle ~4.4.2
-  - CONTAO_VERSION=contao/core-bundle ~4.5.0
-
-matrix:
-  exclude:
-    - php: 5.6
-      env: CONTAO_VERSION=contao/core-bundle ~4.5.0
-    - php: 7.0
-      env: CONTAO_VERSION=contao/core-bundle ~4.5.0
-
-sudo: false
-
-install:
-  - travis_retry composer self-update && composer --version
-  - travis_retry composer require $CONTAO_VERSION --no-update
-  - travis_retry composer update --prefer-dist --no-interaction
-
-script: ant -keep-going
-
-git:
-  depth: 2147483647
-
-cache:
-  directories:
-    - vendor
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,7 +2,15 @@
 Changelog
 =========

-Version 1.3.0 (2018-02-xx)
+Version 1.4.0 (2022-11-29)
+--------------------------
+
+[Full Changelog](https://github.com/netzmacht/contao-leaflet-geocode-widget/compare/1.3.0...1.4.0)
+
+ - Support PHP 8 and Contao 5
+ - Update coding standard
+
+Version 1.3.0 (2018-02-13)
 --------------------------

 [Full Changelog](https://github.com/netzmacht/contao-leaflet-geocode-widget/compare/1.2.1...1.3.0)
--- a/README.md
+++ b/README.md
@@ -1,13 +1,12 @@
 Leaflet geocode widget
 ======================

-[![Build Status](http://img.shields.io/travis/netzmacht/contao-leaflet-geocode-widget/master.svg?style=flat-square)](https://travis-ci.org/netzmacht/contao-leaflet-geocode-widget)
+[![Build Status](https://img.shields.io/github/workflow/status/netzmacht/contao-leaflet-geocode-widget/Code%20Quality%20Diagnostics?logo=githubactions&logoColor=%23fff&style=for-the-badge)](https://github.com/netzmacht/contao-leaflet-geocode-widget/actions)
 [![Version](http://img.shields.io/packagist/v/netzmacht/contao-leaflet-geocode-widget.svg?style=flat-square)](http://packagist.org/packages/netzmacht/contao-leaflet-geocode-widget)
 [![License](http://img.shields.io/packagist/l/netzmacht/contao-leaflet-geocode-widget.svg?style=flat-square)](http://packagist.org/packages/netzmacht/contao-leaflet-geocode-widget)
 [![Downloads](http://img.shields.io/packagist/dt/netzmacht/contao-leaflet-geocode-widget.svg?style=flat-square)](http://packagist.org/packages/netzmacht/contao-leaflet-geocode-widget)
-[![Contao Community Alliance coding standard](http://img.shields.io/badge/cca-coding_standard-red.svg?style=flat-square)](https://github.com/contao-community-alliance/coding-standard)

-This extension provides an widget to pick coordinates from a map. It uses the leaflet framework.
+This extension provides a widget to pick coordinates from a map. It uses the leaflet framework.

 Changlog
 --------
@@ -17,7 +16,8 @@ See [CHANGELOG](CHANGELOG.md).
 Requirements
 ------------

- - Contao ~4.4
+ - Contao ^4.13||^5.0
+ - PHP ^7.4||^8.0


 Install
@@ -38,7 +38,7 @@ If you use the managed edition of Contao you can skip this step.

  // Dependency is automatically installed and has to be registered
  new Contao\CoreBundle\HttpKernel\Bundle\ContaoModuleBundle('leaflet-libs', $this->getRootDir()),
-  
+
  // Register the bundle
  new Netzmacht\Contao\Leaflet\GeocodeWidget\LeafletGeocodeWidgetBundle(),
 ```
@@ -67,7 +67,7 @@ $GLOBALS['TL_DCA']['tl_example']['fields']['coordinates'] = [
 #### Coordinates and radius

 To pick the radius in meters as well, you have to configure the `eval.radius` option for the related radius field.
-The radius field should be a simle text input. The `default`, `minval` and `maxval` flags are passed to the geocode 
+The radius field should be a simle text input. The `default`, `minval` and `maxval` flags are passed to the geocode
 widget so that only radius in that boundary can be chosen.

 ```php
@@ -84,7 +84,7 @@ $GLOBALS['TL_DCA']['tl_page']['fields']['coordinates'] = [
 $GLOBALS['TL_DCA']['tl_page']['fields']['radius'] = [
    'label'     => ['Radius', 'Angabe des Radius in Metern'],
    'inputType' => 'leaflet_radius', // Optional, you can use a text widget as well
-    'eval'      => [      
+    'eval'      => [
        'default'  => 500,
        'minval'   => 100,
        'maxval'   => 5000,
--- a/build.default.properties
+++ b/build.default.properties
@@ -1,4 +0,0 @@
-phpcs.standard=${basedir}/vendor/phpcq/coding-standard/phpcs/PhpCodeQuality/ruleset.xml
-phpmd.ruleset=${basedir}/vendor/phpcq/coding-standard/phpmd/ruleset.xml
-phpcs.excluded = src/Resources/contao/languages
-phpcpd.excluded = contao
--- a/build.xml
+++ b/build.xml
@@ -1,5 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project name="netzmacht/contao-leaflet-geocode-widget" default="build">
-    <!-- import the main build system -->
-    <import file="vendor/phpcq/phpcq/phpcq.main.xml" />
-</project>
--- a/composer.json
+++ b/composer.json
@@ -1,51 +1,63 @@
 {
-    "name": "netzmacht/contao-leaflet-geocode-widget",
-    "description": "Geocode widget based on Leaflet",
-    "keywords": [
-        "geocode",
-        "contao",
-        "widget",
-        "leaflet"
-    ],
-    "type": "contao-bundle",
-    "license": "LGPL-3.0-or-later",
-    "authors": [
-        {
-            "name": "David Molineus",
-            "email": "mail@netzmacht.de",
-            "homepage": "https://netzmacht.de",
-            "role": "Developer"
-        }
-    ],
-    "support": {
-        "email": "mail@netzmacht.de",
-        "issues": "https://github.com/contao-bootstrap/core/issues",
-        "wiki": "http://contao-bootstrap.netzmacht.de",
-        "source": "https://github.com/contao-bootstrap/core"
-    },
-    "require": {
-        "php": ">=5.6.0",
-        "contao/core-bundle": "^4.4.0",
-        "netzmacht/contao-leaflet-libraries": "~1.3"
-    },
-    "require-dev": {
-        "contao/manager-plugin": "^2.0",
-        "phpcq/all-tasks": "^1.2",
-        "php-http/guzzle6-adapter": "^1.1"
-    },
-    "suggest": {
-        "netzmacht/contao-leaflet-maps": "Leaflet Maps for Contao"
-    },
-    "autoload": {
-        "psr-4": {
-            "Netzmacht\\Contao\\Leaflet\\GeocodeWidget\\": "src/"
-        }
-    },
-    "extra": {
-        "branch-alias": {
-            "dev-master": "1.3.x-dev",
-            "dev-develop": "1.4.x-dev"
-        },
-        "contao-manager-plugin": "Netzmacht\\Contao\\Leaflet\\GeocodeWidget\\ContaoManager\\Plugin"
+  "name": "netzmacht/contao-leaflet-geocode-widget",
+  "description": "Geocode widget based on Leaflet",
+  "license": "LGPL-3.0-or-later",
+  "type": "contao-bundle",
+  "keywords": [
+    "geocode",
+    "contao",
+    "widget",
+    "leaflet"
+  ],
+  "authors": [
+    {
+      "name": "David Molineus",
+      "email": "mail@netzmacht.de",
+      "homepage": "https://netzmacht.de",
+      "role": "Developer"
    }
+  ],
+  "support": {
+    "email": "mail@netzmacht.de",
+    "issues": "https://github.com/netzmacht/contao-leaflet-geocode-widget/issues",
+    "source": "https://github.com/netzmacht/contao-leaflet-geocode-widget"
+  },
+  "require": {
+    "php": "^7.4 || ^8.0",
+    "contao/core-bundle": "^4.13 || ^5.0",
+    "netzmacht/contao-leaflet-libraries": "~1.3",
+    "symfony/config": "^5.4 || ^6.0",
+    "symfony/dependency-injection": "^5.4 || ^6.0",
+    "symfony/http-kernel": "^5.4 || ^6.0"
+  },
+  "require-dev": {
+    "contao/manager-plugin": "^2.0",
+    "doctrine/coding-standard": "^9.0",
+    "phpcq/runner-bootstrap": "1.x-dev",
+    "phpspec/phpspec": "^6.3.0 || ^7.0"
+  },
+  "suggest": {
+    "netzmacht/contao-leaflet-maps": "Leaflet Maps for Contao"
+  },
+  "autoload": {
+    "psr-4": {
+      "Netzmacht\\Contao\\Leaflet\\GeocodeWidget\\": "src/"
+    }
+  },
+  "config": {
+    "allow-plugins": {
+      "contao-components/installer": true,
+      "contao/manager-plugin": true,
+      "dealerdirect/phpcodesniffer-composer-installer": true,
+      "contao-community-alliance/composer-plugin": true
+    },
+    "sort-packages": true
+  },
+  "extra": {
+    "branch-alias": {
+      "dev-develop": "1.5.x-dev",
+      "dev-master": "1.4.x-dev"
+    },
+    "contao-manager-plugin": "Netzmacht\\Contao\\Leaflet\\GeocodeWidget\\ContaoManager\\Plugin"
+  }
 }
--- a/phpcs.xml.dist
+++ b/phpcs.xml.dist
@@ -0,0 +1,102 @@
+<?xml version="1.0"?>
+<ruleset>
+    <arg name="basepath" value="."/>
+    <arg name="extensions" value="php"/>
+    <arg name="parallel" value="80"/>
+    <arg name="cache" value=".phpcs-cache"/>
+    <arg name="colors"/>
+
+    <!-- Required for the SlevomatCodingStandard which utilizes composer dependencies -->
+    <autoload>vendor/autoload.php</autoload>
+
+    <!-- Ignore warnings, show progress of the run and show sniff names -->
+    <arg value="nps"/>
+
+    <!-- Directories to be checked -->
+    <file>src</file>
+
+    <!-- Include full Doctrine Coding Standard -->
+    <rule ref="Doctrine">
+        <exclude name="SlevomatCodingStandard.Classes.SuperfluousAbstractClassNaming.SuperfluousPrefix"/>
+        <exclude name="SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint"/>
+        <exclude name="SlevomatCodingStandard.TypeHints.PropertyTypeHint.MissingNativeTypeHint"/>
+    </rule>
+
+    <rule ref="Generic.Files.LineLength">
+        <properties>
+            <property name="lineLimit" value="120"/>
+            <property name="absoluteLineLimit" value="120"/>
+        </properties>
+        <exclude-pattern>src/Resources/contao/languages/*</exclude-pattern>
+    </rule>
+
+    <rule ref="SlevomatCodingStandard.TypeHints.ParameterTypeHint">
+        <properties>
+            <property name="enableObjectTypeHint" value="false" />
+        </properties>
+    </rule>
+
+    <rule ref="SlevomatCodingStandard.TypeHints.PropertyTypeHint">
+        <properties>
+            <property name="enableMixedTypeHint" value="false" />
+            <property name="enableUnionTypeHint" value="false" />
+        </properties>
+    </rule>
+
+    <rule ref="SlevomatCodingStandard.TypeHints.ReturnTypeHint">
+        <properties>
+            <property name="enableStaticTypeHint" value="false" />
+            <property name="enableMixedTypeHint" value="false" />
+            <property name="enableUnionTypeHint" value="false" />
+        </properties>
+    </rule>
+
+    <rule ref="PSR1.Methods.CamelCapsMethodName.NotCamelCaps">
+        <exclude-pattern>spec/.*\.php</exclude-pattern>
+    </rule>
+
+    <!-- Require specific order of phpDoc annotations with empty newline between specific groups -->
+    <rule ref="SlevomatCodingStandard.Commenting.DocCommentSpacing">
+        <properties>
+            <property name="linesCountBeforeFirstContent" value="0"/>
+            <property name="linesCountAfterLastContent" value="0"/>
+            <property name="linesCountBetweenDescriptionAndAnnotations" value="1"/>
+            <property name="linesCountBetweenAnnotationsGroups" value="1"/>
+            <property name="annotationsGroups" type="array">
+                <element value="
+                    @internal,
+                    @deprecated,
+                "/>
+                <element value="
+                    @template,
+                    @psalm-template,
+                    @template-implements,
+                    @template-extends,
+                    @implements,
+                    @extends
+                "/>
+                <element value="
+                    @link,
+                    @see,
+                    @uses,
+                "/>
+                <element value="
+                    @ORM\,
+                    @ODM\,
+                    @PHPCR\,
+                "/>
+                <element value="
+                    @param,
+                    @psalm-param,
+                    @phpstan-param,
+                "/>
+                <element value="
+                    @return,
+                    @psalm-return,
+                    @phpstan-return,
+                "/>
+                <element value="@throws"/>
+            </property>
+        </properties>
+    </rule>
+</ruleset>
--- a/src/ContaoManager/Plugin.php
+++ b/src/ContaoManager/Plugin.php
@@ -1,14 +1,6 @@
 <?php

-/**
- * Geocode backend widget based on Leaflet.
- *
- * @package    netzmacht
- * @author     David Molineus <david.molineus@netzmacht.de>
- * @copyright  2016-2017 netzmacht David Molineus. All rights reserved.
- * @license    LGPL-3.0 https://github.com/netzmacht/contao-leaflet-geocode-widget/blob/master/LICENSE
- * @filesource
- */
+declare(strict_types=1);

 namespace Netzmacht\Contao\Leaflet\GeocodeWidget\ContaoManager;

@@ -18,17 +10,12 @@ use Contao\ManagerPlugin\Bundle\Config\BundleConfig;
 use Contao\ManagerPlugin\Bundle\Parser\ParserInterface;
 use Netzmacht\Contao\Leaflet\GeocodeWidget\LeafletGeocodeWidgetBundle;

-/**
- * Contao Manager plugin.
- *
- * @package Netzmacht\Contao\Leaflet\GeocodeWidget\ContaoManager
- */
 class Plugin implements BundlePluginInterface
 {
    /**
     * {@inheritdoc}
     */
-    public function getBundles(ParserInterface $parser)
+    public function getBundles(ParserInterface $parser): array
    {
        return [
            BundleConfig::create(LeafletGeocodeWidgetBundle::class)
--- a/src/DependencyInjection/LeafletGeocodeWidgetExtension.php
+++ b/src/DependencyInjection/LeafletGeocodeWidgetExtension.php
@@ -1,14 +1,6 @@
 <?php

-/**
- * Geocode backend widget based on Leaflet.
- *
- * @package    netzmacht
- * @author     David Molineus <david.molineus@netzmacht.de>
- * @copyright  2016-2018 netzmacht David Molineus. All rights reserved.
- * @license    LGPL-3.0 https://github.com/netzmacht/contao-leaflet-geocode-widget/blob/master/LICENSE
- * @filesource
- */
+declare(strict_types=1);

 namespace Netzmacht\Contao\Leaflet\GeocodeWidget\DependencyInjection;

@@ -17,15 +9,14 @@ use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Extension\Extension;
 use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;

-/**
- * Class LeafletGeocodeWidgetExtension
- */
+use function dirname;
+
 class LeafletGeocodeWidgetExtension extends Extension
 {
    /**
     * {@inheritdoc}
     */
-    public function load(array $configs, ContainerBuilder $container)
+    public function load(array $configs, ContainerBuilder $container): void
    {
        $loader = new YamlFileLoader(
            $container,
--- a/src/EventListener/RadiusWizardCallbackListener.php
+++ b/src/EventListener/RadiusWizardCallbackListener.php
@@ -1,24 +1,13 @@
 <?php

-/**
- * Geocode backend widget based on Leaflet.
- *
- * @package    netzmacht
- * @author     David Molineus <david.molineus@netzmacht.de>
- * @copyright  2016-2018 netzmacht David Molineus. All rights reserved.
- * @license    LGPL-3.0 https://github.com/netzmacht/contao-leaflet-geocode-widget/blob/master/LICENSE
- * @filesource
- */
+declare(strict_types=1);

 namespace Netzmacht\Contao\Leaflet\GeocodeWidget\EventListener;

 use Contao\DataContainer;

-/**
- * Class RadiusWizardCallbackListener
- *
- * @package Netzmacht\Contao\Leaflet\GeocodeWidget\EventListener
- */
+use function sprintf;
+
 class RadiusWizardCallbackListener
 {
    /**
@@ -26,13 +15,13 @@ class RadiusWizardCallbackListener
     *
     * @param DataContainer $dataContainer Data container driver.
     *
-     * @return string
-     *
     * @SuppressWarnings(PHPMD.Superglobals)
     */
-    public function generateWizard($dataContainer)
+    public function generateWizard(DataContainer $dataContainer): string
    {
-        if (!isset($GLOBALS['TL_DCA'][$dataContainer->table]['fields'][$dataContainer->field]['eval']['coordinates'])) {
+        $fields = $GLOBALS['TL_DCA'][$dataContainer->table]['fields'];
+
+        if (! isset($fields[$dataContainer->field]['eval']['coordinates'])) {
            return '';
        }

--- a/src/GeocodeWidget.php
+++ b/src/GeocodeWidget.php
@@ -1,30 +1,17 @@
 <?php

-/**
- * Geocode backend widget based on Leaflet.
- *
- * @package    netzmacht
- * @author     David Molineus <david.molineus@netzmacht.de>
- * @copyright  2016-2018 netzmacht David Molineus. All rights reserved.
- * @license    LGPL-3.0 https://github.com/netzmacht/contao-leaflet-geocode-widget/blob/master/LICENSE
- * @filesource
- */
+declare(strict_types=1);

 namespace Netzmacht\Contao\Leaflet\GeocodeWidget;

 use Netzmacht\Contao\Leaflet\GeocodeWidget\Widget\GeocodeWidget as BaseWidget;

-/**
- * Class GeocodeWidget
- *
- * @deprecated
- */
 class GeocodeWidget extends BaseWidget
 {
    /**
     * {@inheritdoc}
     */
-    public function __construct(array $arrAttributes = null)
+    public function __construct(?array $arrAttributes = null)
    {
        parent::__construct($arrAttributes);

--- a/src/LeafletGeocodeWidgetBundle.php
+++ b/src/LeafletGeocodeWidgetBundle.php
@@ -1,24 +1,11 @@
 <?php

-/**
- * Geocode backend widget based on Leaflet.
- *
- * @package    netzmacht
- * @author     David Molineus <david.molineus@netzmacht.de>
- * @copyright  2016-2017 netzmacht David Molineus. All rights reserved.
- * @license    LGPL-3.0 https://github.com/netzmacht/contao-leaflet-geocode-widget/blob/master/LICENSE
- * @filesource
- */
+declare(strict_types=1);

 namespace Netzmacht\Contao\Leaflet\GeocodeWidget;

 use Symfony\Component\HttpKernel\Bundle\Bundle;

-/**
- * Class LeafletGeocodeWidgetBundle
- *
- * @package Netzmacht\Contao\Leaflet\GeocodeWidget
- */
 class LeafletGeocodeWidgetBundle extends Bundle
 {
 }
--- a/src/Resources/contao/config/config.php
+++ b/src/Resources/contao/config/config.php
@@ -1,14 +1,9 @@
 <?php

-/**
- * Geocode backend widget based on Leaflet.
- *
- * @package    netzmacht
- * @author     David Molineus <david.molineus@netzmacht.de>
- * @copyright  2016-2018 netzmacht David Molineus. All rights reserved.
- * @license    LGPL-3.0 https://github.com/netzmacht/contao-leaflet-geocode-widget/blob/master/LICENSE
- * @filesource
- */
+declare(strict_types=1);

-$GLOBALS['BE_FFL']['leaflet_geocode'] = 'Netzmacht\Contao\Leaflet\GeocodeWidget\Widget\GeocodeWidget';
-$GLOBALS['BE_FFL']['leaflet_radius']  = 'Netzmacht\Contao\Leaflet\GeocodeWidget\Widget\RadiusWidget';
+use Netzmacht\Contao\Leaflet\GeocodeWidget\Widget\GeocodeWidget;
+use Netzmacht\Contao\Leaflet\GeocodeWidget\Widget\RadiusWidget;
+
+$GLOBALS['BE_FFL']['leaflet_geocode'] = GeocodeWidget::class;
+$GLOBALS['BE_FFL']['leaflet_radius']  = RadiusWidget::class;
--- a/src/Widget/GeocodeWidget.php
+++ b/src/Widget/GeocodeWidget.php
@@ -1,36 +1,34 @@
 <?php

-/**
- * Geocode backend widget based on Leaflet.
- *
- * @package    netzmacht
- * @author     David Molineus <david.molineus@netzmacht.de>
- * @copyright  2016-2018 netzmacht David Molineus. All rights reserved.
- * @license    LGPL-3.0 https://github.com/netzmacht/contao-leaflet-geocode-widget/blob/master/LICENSE
- * @filesource
- */
+declare(strict_types=1);

 namespace Netzmacht\Contao\Leaflet\GeocodeWidget\Widget;

+use Contao\BackendTemplate;
+use Contao\StringUtil;
+use Contao\Widget;
+
+use function is_array;
+use function preg_match;
+use function sprintf;
+
 /**
- * Class GeocodeWidget
- *
 * @property int  size
 * @property bool multiple
 */
-class GeocodeWidget extends \Widget
+class GeocodeWidget extends Widget
 {
    /**
     * Submit user input.
     *
-     * @var boolean
+     * @var bool
     */
    protected $blnSubmitInput = true;

    /**
     * Add a for attribute.
     *
-     * @var boolean
+     * @var bool
     */
    protected $blnForAttribute = true;

@@ -61,7 +59,7 @@ class GeocodeWidget extends \Widget
    {
        $value = parent::validator($value);

-        if (!$value) {
+        if (! $value) {
            return $value;
        }

@@ -74,10 +72,10 @@ class GeocodeWidget extends \Widget
        }

        // See: http://stackoverflow.com/a/18690202
-        if (!preg_match(
-            '#^[-+]?([1-8]?\d(\.\d+)?|90(\.0+)?),[-+]?(180(\.0+)?|((1[0-7]\d)|([1-9]?\d))(\.\d+)?)(,[-+]?\d+)?$#',
-            $value
-        )) {
+        $pattern = '#^[-+]?([1-8]?\d(\.\d+)?|90(\.0+)?),[-+]?(180(\.0+)?|'
+            . '((1[0-7]\d)|([1-9]?\d))(\.\d+)?)(,[-+]?\d+)?$#';
+
+        if (! preg_match($pattern, $value)) {
            $this->addError(
                sprintf(
                    $GLOBALS['TL_LANG']['ERR']['leafletInvalidCoordinate'],
@@ -91,39 +89,37 @@ class GeocodeWidget extends \Widget

    /**
     * Generate the widget.
-     *
-     * @return string
     */
-    public function generate()
+    public function generate(): string
    {
        $wrapperClass = 'wizard';

-        if (!$this->multiple || !$this->size) {
+        if (! $this->multiple || ! $this->size) {
            $this->size = 1;
        } else {
            $wrapperClass .= ' wizard_' . $this->size;
        }

-        if (!is_array($this->value)) {
+        if (! is_array($this->value)) {
            $this->value = [$this->value];
        }

        $buffer = '';

        for ($index = 0; $index < $this->size; $index++) {
-            $template = new \BackendTemplate($this->widgetTemplate);
+            $template = new BackendTemplate($this->widgetTemplate);
            $template->setData(
                [
                    'wrapperClass' => $wrapperClass,
                    'widget'       => $this,
-                    'value'        => \StringUtil::specialchars($this->value[$index]),
+                    'value'        => StringUtil::specialchars($this->value[$index]),
                    'class'        => $this->strClass ? ' ' . $this->strClass : '',
-                    'id'           => $this->strId . (($this->size > 1) ? '_' . $index : ''),
-                    'name'         => $this->strName . (($this->size > 1) ? '[]' : ''),
+                    'id'           => $this->strId . ($this->size > 1 ? '_' . $index : ''),
+                    'name'         => $this->strName . ($this->size > 1 ? '[]' : ''),
                    'attributes'   => $this->getAttributes(),
                    'wizard'       => $this->wizard,
                    'label'        => $this->strLabel,
-                    'radius'       => $this->buildRadiusOptions()
+                    'radius'       => $this->buildRadiusOptions(),
                ]
            );

@@ -136,13 +132,13 @@ class GeocodeWidget extends \Widget
    /**
     * Build the radius options.
     *
-     * @return array|null
+     * @return array<string,mixed>|null
     *
     * @SuppressWarnings(PHPMD.Superglobals)
     */
-    private function buildRadiusOptions()
+    private function buildRadiusOptions(): ?array
    {
-        if (!$this->radius || !isset($GLOBALS['TL_DCA'][$this->strTable]['fields'][$this->radius])) {
+        if (! $this->radius || ! isset($GLOBALS['TL_DCA'][$this->strTable]['fields'][$this->radius])) {
            return null;
        }

@@ -150,7 +146,7 @@ class GeocodeWidget extends \Widget
            'element'      => 'ctrl_' . $this->radius,
            'min'          => 0,
            'max'          => 0,
-            'defaultValue' => 0
+            'defaultValue' => 0,
        ];

        if (isset($GLOBALS['TL_DCA'][$this->strTable]['fields'][$this->radius]['eval'])) {
--- a/src/Widget/RadiusWidget.php
+++ b/src/Widget/RadiusWidget.php
@@ -1,23 +1,18 @@
 <?php

-/**
- * Geocode backend widget based on Leaflet.
- *
- * @package    netzmacht
- * @author     David Molineus <david.molineus@netzmacht.de>
- * @copyright  2016-2018 netzmacht David Molineus. All rights reserved.
- * @license    LGPL-3.0 https://github.com/netzmacht/contao-leaflet-geocode-widget/blob/master/LICENSE
- * @filesource
- */
+declare(strict_types=1);

 namespace Netzmacht\Contao\Leaflet\GeocodeWidget\Widget;

 use Contao\BackendTemplate;
+use Contao\StringUtil;
 use Contao\TextField;

-/**
- * Class RadiusWidget
- */
+use function is_array;
+use function is_numeric;
+use function round;
+use function trim;
+
 class RadiusWidget extends TextField
 {
    /**
@@ -41,20 +36,18 @@ class RadiusWidget extends TextField

    /**
     * Generate the widget.
-     *
-     * @return string
     */
-    public function generate()
+    public function generate(): string
    {
        $wrapperClass = $this->coordinates ? 'wizard' : '';

-        if (!$this->multiple || !$this->size) {
+        if (! $this->multiple || ! $this->size) {
            $this->size = 1;
        } else {
            $wrapperClass .= ' wizard_' . $this->size;
        }

-        if (!is_array($this->value)) {
+        if (! is_array($this->value)) {
            $this->value = [$this->value];
        }

@@ -66,14 +59,14 @@ class RadiusWidget extends TextField
                [
                    'wrapperClass' => trim($wrapperClass),
                    'widget'       => $this,
-                    'value'        => \StringUtil::specialchars($this->value[$index]),
+                    'value'        => StringUtil::specialchars($this->value[$index]),
                    'class'        => $this->strClass ? ' ' . $this->strClass : '',
-                    'id'           => $this->strId . (($this->size > 1) ? '_' . $index : ''),
-                    'name'         => $this->strName . (($this->size > 1) ? '[]' : ''),
+                    'id'           => $this->strId . ($this->size > 1 ? '_' . $index : ''),
+                    'name'         => $this->strName . ($this->size > 1 ? '[]' : ''),
                    'attributes'   => $this->getAttributes(),
                    'wizard'       => $this->wizard,
                    'label'        => $this->strLabel,
-                    'coordinates'  => $this->coordinates
+                    'coordinates'  => $this->coordinates,
                ]
            );

@@ -91,7 +84,7 @@ class RadiusWidget extends TextField
        if (is_numeric($varInput) && $this->steps > 0) {
            $steps    = (int) $this->steps;
            $varInput = (int) $varInput;
-            $varInput = ($steps * round($varInput / $steps));
+            $varInput = $steps * round($varInput / $steps);
        }

        return parent::validator($varInput);
Author	SHA1	Message	Date
dependabot[bot]	a5f98a35bc	github-actions(deps): bump actions/checkout from 3 to 4 Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>	2023-09-04 13:59:23 +00:00
David Molineus	eca2b6e00c	Merge pull request #10 from netzmacht/dependabot/github_actions/actions/cache-3 github-actions(deps): bump actions/cache from 1 to 3	2022-11-29 14:28:14 +01:00
David Molineus	2becf0344a	Merge pull request #11 from netzmacht/dependabot/github_actions/actions/checkout-3 github-actions(deps): bump actions/checkout from 2 to 3	2022-11-29 14:28:05 +01:00
David Molineus	b4ab2bbc78	Merge pull request #12 from netzmacht/dependabot/github_actions/actions/upload-artifact-3 github-actions(deps): bump actions/upload-artifact from 2 to 3	2022-11-29 14:27:54 +01:00
dependabot[bot]	741efef65d	github-actions(deps): bump actions/upload-artifact from 2 to 3 Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 2 to 3. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](https://github.com/actions/upload-artifact/compare/v2...v3) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>	2022-11-29 12:41:27 +00:00
dependabot[bot]	5ac8d1141c	github-actions(deps): bump actions/checkout from 2 to 3 Bumps [actions/checkout](https://github.com/actions/checkout) from 2 to 3. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/v2...v3) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>	2022-11-29 12:41:25 +00:00
dependabot[bot]	8f05dfe6d6	github-actions(deps): bump actions/cache from 1 to 3 Bumps [actions/cache](https://github.com/actions/cache) from 1 to 3. - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](https://github.com/actions/cache/compare/v1...v3) --- updated-dependencies: - dependency-name: actions/cache dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>	2022-11-29 12:41:22 +00:00
David Molineus	2da4b02ddc	Merge tag '1.4.0'	2022-11-29 13:40:44 +01:00
David Molineus	6b31c6ebad	Prepare release 1.4.0	2022-11-29 13:40:22 +01:00
David Molineus	addf773edc	Merge tag '1.4.0'	2022-11-29 13:39:36 +01:00
David Molineus	bad7175bab	Prepare release 1.4.0	2022-11-29 13:39:12 +01:00
David Molineus	268372c12a	Fix links (#8 )	2022-11-29 11:10:48 +01:00
David Molineus	accedc63e4	Update coding standard	2022-11-29 11:05:30 +01:00
David Molineus	37f94737c5	Merge pull request #9 from qzminski/master Fix compatibility with Contao 5	2022-11-29 10:30:39 +01:00
Kamil Kuzminski	03668e5852	Fix compatibility with Contao 5	2022-09-14 12:15:03 +02:00
Kamil Kuzminski	c7e65d2db5	Update to Contao 5 (composer only for now)	2022-09-01 16:20:11 +02:00
David Molineus	08acfbc473	Merge tag '1.3.0'	2018-02-23 14:13:05 +01:00