Skip to content

Commit

Permalink
[TASK] Make automatic tests happy again
Browse files Browse the repository at this point in the history
  • Loading branch information
einpraegsam committed Oct 26, 2024
1 parent f0552c3 commit 1d1095c
Show file tree
Hide file tree
Showing 7 changed files with 15 additions and 23 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/testing.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ on: [push, pull_request]
jobs:
php-lint:
name: "PHP linter"
runs-on: ubuntu-20.04
runs-on: ubuntu-24.04
steps:
- name: "Checkout"
uses: actions/checkout@v2
Expand All @@ -25,7 +25,7 @@ jobs:
- 8.3
typoscript-lint:
name: "TypoScript linter"
runs-on: ubuntu-20.04
runs-on: ubuntu-24.04
steps:
- name: "Checkout"
uses: actions/checkout@v2
Expand All @@ -36,7 +36,7 @@ jobs:
config_file: ".project/tests/typoscript-lint.yml"
php-cs-fixer:
name: "PHP CS Fixer"
runs-on: ubuntu-20.04
runs-on: ubuntu-24.04
needs: php-lint
steps:
- name: "Checkout"
Expand All @@ -51,7 +51,7 @@ jobs:
run: "composer test:php:cs"
unit-tests:
name: "PHP Unit Tests"
runs-on: ubuntu-20.04
runs-on: ubuntu-24.04
needs: php-lint
steps:
- name: Checkout
Expand All @@ -70,5 +70,5 @@ jobs:
fail-fast: false
matrix:
php-version:
- 8.1
- 8.2
- 8.3
3 changes: 2 additions & 1 deletion Classes/Controller/AbstractController.php
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,8 @@ public function __construct(
protected readonly CacheLayer $cacheLayer,
protected readonly ModuleTemplateFactory $moduleTemplateFactory,
protected readonly IconFactory $iconFactory
) {}
) {
}

/**
* Pass some important variables to all views
Expand Down
14 changes: 7 additions & 7 deletions Classes/Controller/FrontendController.php
Original file line number Diff line number Diff line change
Expand Up @@ -127,13 +127,13 @@ public function dispatchRequestAction(
*/
public function pageRequestAction(string $identificator, array $arguments): ResponseInterface
{
$visitor = $this->getVisitor($identificator);
$this->callAdditionalTrackers($visitor, $arguments);
$this->companyTracker->track($visitor);
$pagevisit = $this->pageTracker->track($visitor, $arguments);
$this->newsTracker->track($visitor, $arguments, $pagevisit);
$this->searchTracker->track($visitor, $arguments, $pagevisit);
return $this->jsonResponse(json_encode($this->afterAction($visitor)));
$visitor = $this->getVisitor($identificator);
$this->callAdditionalTrackers($visitor, $arguments);
$this->companyTracker->track($visitor);
$pagevisit = $this->pageTracker->track($visitor, $arguments);
$this->newsTracker->track($visitor, $arguments, $pagevisit);
$this->searchTracker->track($visitor, $arguments, $pagevisit);
return $this->jsonResponse(json_encode($this->afterAction($visitor)));
}

/**
Expand Down
1 change: 0 additions & 1 deletion Classes/Domain/Tracker/UtmTracker.php
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
use In2code\Lux\Events\PageTrackerEvent;
use Psr\EventDispatcher\EventDispatcherInterface;
use Throwable;
use TYPO3\CMS\Core\Utility\GeneralUtility;

class UtmTracker
{
Expand Down
1 change: 0 additions & 1 deletion Classes/Hooks/PageOverview.php
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
use In2code\Lux\Utility\BackendUtility;
use In2code\Lux\Utility\ConfigurationUtility;
use TYPO3\CMS\Backend\Controller\Event\ModifyPageLayoutContentEvent;
use TYPO3\CMS\Backend\Controller\PageLayoutController;
use TYPO3\CMS\Backend\Utility\BackendUtility as BackendUtilityCore;
use TYPO3\CMS\Core\Configuration\Exception\ExtensionConfigurationExtensionNotConfiguredException;
use TYPO3\CMS\Core\Configuration\Exception\ExtensionConfigurationPathDoesNotExistException;
Expand Down
1 change: 0 additions & 1 deletion Classes/ViewHelpers/Pagination/UriViewHelper.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
declare(strict_types=1);
namespace In2code\Lux\ViewHelpers\Pagination;

use In2code\Lux\Utility\ConfigurationUtility;
use TYPO3\CMS\Core\Utility\GeneralUtility;
use TYPO3\CMS\Extbase\Mvc\Web\Routing\UriBuilder;
use TYPO3\CMS\Extbase\Service\ExtensionService;
Expand Down
8 changes: 1 addition & 7 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,6 @@
"replace": {
"typo3-ter/lux": "self.version"
},
"repositories": [
{
"type": "vcs",
"url": "https://github.com/bmack/TYPO3-Console"
}
],
"suggest": {
"typo3/cms-dashboard": "*"
},
Expand Down Expand Up @@ -73,7 +67,7 @@
"phpmd/phpmd": "^2.8",
"friendsofphp/php-cs-fixer": "^3.10",
"helmich/typo3-typoscript-lint": "^3.1",
"helhum/typo3-console": "dev-issue/1169"
"helhum/typo3-console": "^8.2"
},
"autoload": {
"psr-4": {
Expand Down

0 comments on commit 1d1095c

Please sign in to comment.