Merge pull request #3 from SkillDisplay/feature/campaign-integration

Integrate campaign selection for verification tracking
2024-12-22 06:16:09 +01:00 · 2020-12-01 13:42:10 +01:00 · 2020-12-01 13:42:10 +01:00 · ce53a9ab6e
commit ce53a9ab6e
parent 174d2ff691 5d5c502794
20 changed files with 91 additions and 12 deletions
--- a/Classes/TcaEnhancer.php
+++ b/Classes/TcaEnhancer.php
@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace SkillDisplay\Typo3Extension;
+
+use SkillDisplay\PHPToolKit\Api\Campaigns;
+use SkillDisplay\PHPToolKit\Entity\Campaign;
+use TYPO3\CMS\Core\Utility\GeneralUtility;
+
+class TcaEnhancer
+{
+    public function getCampaignsForTCA(array $params): void
+    {
+        $params['items'] = [
+            ['', 0],
+        ];
+
+        $campaigns = GeneralUtility::makeInstance(Campaigns::class)->getForUser();
+        /** @var Campaign $campaign */
+        foreach ($campaigns as $campaign) {
+            $params['items'][] = [
+                $campaign->getTitle(),
+                $campaign->getId(),
+            ];
+        }
+    }
+}
--- a/Classes/ViewHelpers/Verification/ButtonViewHelper.php
+++ b/Classes/ViewHelpers/Verification/ButtonViewHelper.php
@ -34,7 +34,8 @@ class ButtonViewHelper extends VerificationViewHelper
        return $link->getVerificationButton(
            $arguments['type'],
            static::getId($arguments),
-            static::getType($arguments)
+            static::getType($arguments),
+            $arguments['campaign']
        );
    }
 }
--- a/Classes/ViewHelpers/Verification/UrlViewHelper.php
+++ b/Classes/ViewHelpers/Verification/UrlViewHelper.php
@ -34,7 +34,8 @@ class UrlViewHelper extends VerificationViewHelper
        return $link->getVerificationLink(
            $arguments['type'],
            static::getId($arguments),
-            static::getType($arguments)
+            static::getType($arguments),
+            $arguments['campaign']
        );
    }
 }
--- a/Classes/ViewHelpers/VerificationViewHelper.php
+++ b/Classes/ViewHelpers/VerificationViewHelper.php
@ -38,6 +38,7 @@ abstract class VerificationViewHelper extends AbstractViewHelper
        $this->registerArgument('skill', 'integer', 'ID of the Skill.');
        $this->registerArgument('skillSet', 'integer', 'ID of the SkillSet.');
        $this->registerArgument('type', 'string', 'Type of verification', false, 'self');
+        $this->registerArgument('campaign', 'int', 'ID of campaign', false, 0);
    }

    public static function renderStatic(
--- a/Configuration/Services.yaml
+++ b/Configuration/Services.yaml
@ -28,6 +28,10 @@ services:
        arguments:
            $settings: '@skilldisplay.settings'

+    SkillDisplay\PHPToolKit\Api\Campaigns:
+        arguments:
+            $settings: '@skilldisplay.settings'
+
    SkillDisplay\PHPToolKit\Verification\Link:
        public: true
        arguments:
--- a/Configuration/TCA/Overrides/tt_content.php
+++ b/Configuration/TCA/Overrides/tt_content.php
@ -32,6 +32,15 @@
                    'size' => 10,
                ],
            ],
+            'skilldisplay_campaign' => [
+                'label' => $languagePath . 'skilldisplay_campaign',
+                'config' => [
+                    'type' => 'select',
+                    'renderType' => 'selectSingle',
+                    'itemsProcFunc' => \SkillDisplay\Typo3Extension\TcaEnhancer::class . '->' . 'getCampaignsForTCA',
+                    'items' => []
+                ]
+            ],
        ],
    ]);
 })('skilldisplay', 'tt_content');
--- a/Configuration/TCA/Overrides/tt_content_skills.php
+++ b/Configuration/TCA/Overrides/tt_content_skills.php
@ -15,6 +15,7 @@
                    '--div--;LLL:EXT:core/Resources/Private/Language/Form/locallang_tabs.xlf:general',
                    '--palette--;;general',
                    'skilldisplay_skills',
+                    'skilldisplay_campaign',
                    '--div--;LLL:EXT:frontend/Resources/Private/Language/locallang_ttc.xlf:tabs.appearance',
                    '--palette--;;frames',
                    '--palette--;;appearanceLinks',
--- a/Configuration/TCA/Overrides/tt_content_skillset.php
+++ b/Configuration/TCA/Overrides/tt_content_skillset.php
@ -15,6 +15,7 @@
                    '--div--;LLL:EXT:core/Resources/Private/Language/Form/locallang_tabs.xlf:general',
                    '--palette--;;general',
                    'skilldisplay_skillset',
+                    'skilldisplay_campaign',
                    '--div--;LLL:EXT:frontend/Resources/Private/Language/locallang_ttc.xlf:tabs.appearance',
                    '--palette--;;frames',
                    '--palette--;;appearanceLinks',
--- a/Resources/Private/Templates/Backend/ContentElements/Skills.html
+++ b/Resources/Private/Templates/Backend/ContentElements/Skills.html
@ -16,7 +16,7 @@
                        </f:then>
                        <f:else>
                            <strong>{skill.title}</strong><br>
-                            {sd:verification.url(skill: skill.id)}
+                            {sd:verification.url(skill: skill.id, campaign: skilldisplay_campaign)}
                        </f:else>
                    </f:if>
                </li>
--- a/Resources/Private/Templates/Backend/ContentElements/Skillset.html
+++ b/Resources/Private/Templates/Backend/ContentElements/Skillset.html
@ -16,7 +16,7 @@
                        </f:then>
                        <f:else>
                            <strong>{skillSet.title}</strong><br>
-                            {sd:verification.url(skillSet: skillSet.id)}
+                            {sd:verification.url(skillSet: skillSet.id, campaign: skilldisplay_campaign)}
                        </f:else>
                    </f:if>
                </li>
--- a/Resources/Private/Templates/ContentElements/SkillDisplaySkillSet.html
+++ b/Resources/Private/Templates/ContentElements/SkillDisplaySkillSet.html
@ -7,7 +7,7 @@
            count: '{skillSet.skills -> f:count()}',
            detailUrl: 'https://my.skilldisplay.eu/skillset/{skillSet.id}',
            brandLogoUrl: skillSet.brand.logoPublicUrl,
-            verificationUrl: '{sd:verification.url(skillSet: skillSet.id)}',
+            verificationUrl: '{sd:verification.url(skillSet: skillSet.id, campaign: data.skilldisplay_campaign)}',
            description: skillSet.description
        })}
    </f:for>
--- a/Resources/Private/Templates/ContentElements/SkillDisplaySkills.html
+++ b/Resources/Private/Templates/ContentElements/SkillDisplaySkills.html
@ -6,7 +6,7 @@
            title: skill.title,
            detailUrl: 'https://my.skilldisplay.eu/skill/{skill.id}/0',
            brandLogoUrl: skill.brands.0.logoPublicUrl,
-            verificationUrl: '{sd:verification.url(skill: skill.id)}',
+            verificationUrl: '{sd:verification.url(skill: skill.id, campaign: data.skilldisplay_campaign)}',
            description: skill.description
        })}
    </f:for>
--- a/Tests/Unit/Backend/PreviewTest.php
+++ b/Tests/Unit/Backend/PreviewTest.php
@ -21,6 +21,7 @@ namespace SkillDisplay\Typo3Extension\Tests\Unit\Backend;
 * 02110-1301, USA.
 */

+use Prophecy\PhpUnit\ProphecyTrait;
 use SkillDisplay\PHPToolKit\Api\Skill;
 use SkillDisplay\PHPToolKit\Api\SkillSet;
 use SkillDisplay\PHPToolKit\Entity\Skill as SkillEntity;
@ -34,6 +35,8 @@ use TYPO3\TestingFramework\Core\Unit\UnitTestCase as TestCase;
 */
 class PreviewTest extends TestCase
 {
+    use ProphecyTrait;
+
    /**
     * @test
     */
--- a/Tests/Unit/Frontend/DataProcessing/SkillSetsTest.php
+++ b/Tests/Unit/Frontend/DataProcessing/SkillSetsTest.php
@ -21,6 +21,7 @@ namespace SkillDisplay\Typo3Extension\Tests\Unit\Frontend\DataProcessing;
 * 02110-1301, USA.
 */

+use Prophecy\PhpUnit\ProphecyTrait;
 use SkillDisplay\PHPToolKit\Api\SkillSet;
 use SkillDisplay\PHPToolKit\Entity\SkillSet as SkillSetEntity;
 use SkillDisplay\Typo3Extension\Frontend\DataProcessing\SkillSets;
@ -32,6 +33,8 @@ use TYPO3\TestingFramework\Core\Unit\UnitTestCase as TestCase;
 */
 class SkillSetsTest extends TestCase
 {
+    use ProphecyTrait;
+
    /**
     * @test
     */
--- a/Tests/Unit/Frontend/DataProcessing/SkillsTest.php
+++ b/Tests/Unit/Frontend/DataProcessing/SkillsTest.php
@ -21,6 +21,7 @@ namespace SkillDisplay\Typo3Extension\Tests\Unit\Frontend\DataProcessing;
 * 02110-1301, USA.
 */

+use Prophecy\PhpUnit\ProphecyTrait;
 use SkillDisplay\PHPToolKit\Api\Skill;
 use SkillDisplay\PHPToolKit\Entity\Skill as SkillEntity;
 use SkillDisplay\Typo3Extension\Frontend\DataProcessing\Skills;
@ -32,6 +33,8 @@ use TYPO3\TestingFramework\Core\Unit\UnitTestCase as TestCase;
 */
 class SkillsTest extends TestCase
 {
+    use ProphecyTrait;
+
    /**
     * @test
     */
--- a/Tests/Unit/SettingsFactoryTest.php
+++ b/Tests/Unit/SettingsFactoryTest.php
@ -21,6 +21,7 @@ namespace SkillDisplay\Typo3Extension\Tests\Unit;
 * 02110-1301, USA.
 */

+use Prophecy\PhpUnit\ProphecyTrait;
 use SkillDisplay\PHPToolKit\Configuration\Settings;
 use SkillDisplay\Typo3Extension\SettingsFactory;
 use TYPO3\CMS\Core\Http\ServerRequest;
@ -32,6 +33,8 @@ use TYPO3\TestingFramework\Core\Unit\UnitTestCase as TestCase;
 */
 class SettingsFactoryTest extends TestCase
 {
+    use ProphecyTrait;
+
    /**
     * @test
     */
--- a/Tests/Unit/ViewHelpers/Verification/ButtonViewHelperTest.php
+++ b/Tests/Unit/ViewHelpers/Verification/ButtonViewHelperTest.php
@ -21,6 +21,8 @@ namespace SkillDisplay\Typo3Extension\Tests\Unit\ViewHelpers\Verification;
 * 02110-1301, USA.
 */

+use Prophecy\PhpUnit\ProphecyTrait;
+use Prophecy\Prophecy\ObjectProphecy;
 use SkillDisplay\PHPToolKit\Verification\Link;
 use SkillDisplay\Typo3Extension\ViewHelpers\Verification\ButtonViewHelper;
 use TYPO3Fluid\Fluid\Core\Rendering\RenderingContextInterface;
@ -33,6 +35,8 @@ use TYPO3\TestingFramework\Core\Unit\UnitTestCase as TestCase;
 */
 class ButtonViewHelperTest extends TestCase
 {
+    use ProphecyTrait;
+
    /**
     * @test
     */
@ -47,6 +51,7 @@ class ButtonViewHelperTest extends TestCase
            [
                'skill' => 10,
                'skillSet' => 10,
+                'campaign' => 0,
            ],
            function () {
            },
@ -79,14 +84,16 @@ class ButtonViewHelperTest extends TestCase
    public function returnsRenderedButtonForSkill(): void
    {
        $renderingContext = $this->prophesize(RenderingContextInterface::class);
+        /** @var Link|ObjectProphecy $link */
        $link = $this->prophesize(Link::class);
-        $link->getVerificationButton('self', 10, Link::SKILL)->willReturn('<p>expected HTML</p>');
+        $link->getVerificationButton('self', 10, Link::SKILL, 0)->willReturn('<p>expected HTML</p>');
        GeneralUtility::addInstance(Link::class, $link->reveal());

        $result = ButtonViewHelper::renderStatic(
            [
                'skill' => 10,
                'type' => 'self',
+                'campaign' => 0,
            ],
            function () {
            },
@ -101,14 +108,16 @@ class ButtonViewHelperTest extends TestCase
    public function returnsRenderedButtonForSkillSet(): void
    {
        $renderingContext = $this->prophesize(RenderingContextInterface::class);
+        /** @var Link|ObjectProphecy $link */
        $link = $this->prophesize(Link::class);
-        $link->getVerificationButton('self', 10, Link::SKILL_SET)->willReturn('<p>expected HTML</p>');
+        $link->getVerificationButton('self', 10, Link::SKILL_SET, 0)->willReturn('<p>expected HTML</p>');
        GeneralUtility::addInstance(Link::class, $link->reveal());

        $result = ButtonViewHelper::renderStatic(
            [
                'skillSet' => 10,
                'type' => 'self',
+                'campaign' => 0,
            ],
            function () {
            },
--- a/Tests/Unit/ViewHelpers/Verification/UrlViewHelperTest.php
+++ b/Tests/Unit/ViewHelpers/Verification/UrlViewHelperTest.php
@ -21,6 +21,8 @@ namespace SkillDisplay\Typo3Extension\Tests\Unit\ViewHelpers\Verification;
 * 02110-1301, USA.
 */

+use Prophecy\PhpUnit\ProphecyTrait;
+use Prophecy\Prophecy\ObjectProphecy;
 use SkillDisplay\PHPToolKit\Verification\Link;
 use SkillDisplay\Typo3Extension\ViewHelpers\Verification\UrlViewHelper;
 use TYPO3Fluid\Fluid\Core\Rendering\RenderingContextInterface;
@ -33,6 +35,8 @@ use TYPO3\TestingFramework\Core\Unit\UnitTestCase as TestCase;
 */
 class UrlViewHelperTest extends TestCase
 {
+    use ProphecyTrait;
+
    /**
     * @test
     */
@ -47,6 +51,7 @@ class UrlViewHelperTest extends TestCase
            [
                'skill' => 10,
                'skillSet' => 10,
+                'campaign' => 0,
            ],
            function () {
            },
@ -79,14 +84,17 @@ class UrlViewHelperTest extends TestCase
    public function returnsRenderedUrlForSkill(): void
    {
        $renderingContext = $this->prophesize(RenderingContextInterface::class);
+        /** @var Link|ObjectProphecy $link */
        $link = $this->prophesize(Link::class);
-        $link->getVerificationLink('self', 10, Link::SKILL)->willReturn('https://example.com/path/to/verification');
+        $link->getVerificationLink('self', 10, Link::SKILL, 0)
+            ->willReturn('https://example.com/path/to/verification');
        GeneralUtility::addInstance(Link::class, $link->reveal());

        $result = UrlViewHelper::renderStatic(
            [
                'skill' => 10,
                'type' => 'self',
+                'campaign' => 0,
            ],
            function () {
            },
@ -101,14 +109,17 @@ class UrlViewHelperTest extends TestCase
    public function returnsRenderedUrlForSkillSet(): void
    {
        $renderingContext = $this->prophesize(RenderingContextInterface::class);
+        /** @var Link|ObjectProphecy $link */
        $link = $this->prophesize(Link::class);
-        $link->getVerificationLink('self', 10, Link::SKILL_SET)->willReturn('https://example.com/path/to/verification');
+        $link->getVerificationLink('self', 10, Link::SKILL_SET, 0)
+            ->willReturn('https://example.com/path/to/verification');
        GeneralUtility::addInstance(Link::class, $link->reveal());

        $result = UrlViewHelper::renderStatic(
            [
                'skillSet' => 10,
                'type' => 'self',
+                'campaign' => 0,
            ],
            function () {
            },
--- a/composer.json
+++ b/composer.json
@ -28,7 +28,7 @@
    "require": {
        "php": "7.2.* || 7.3.* || 7.4.*",
        "typo3/cms-core": "^10.4",
-        "skilldisplay/phptoolkit": "^1.1",
+        "skilldisplay/phptoolkit": "^1.2",
        "typo3/cms-backend": "^10.4",
        "typo3/cms-frontend": "^10.4",
        "typo3fluid/fluid": "^2.6"
@ -41,6 +41,7 @@
        "saschaegerer/phpstan-typo3": "^0.13.1",
        "sensiolabs/security-checker": "^6.0",
        "typo3/testing-framework": "^6.4",
+        "phpspec/prophecy-phpunit": "^2",
        "jangregor/phpstan-prophecy": "^0.8.0"
    },
    "minimum-stability": "dev",
--- a/ext_emconf.php
+++ b/ext_emconf.php
@ -14,7 +14,7 @@ $EM_CONF[$_EXTKEY] = [
    'version' => '0.1.0',
    'constraints' => [
        'depends' => [
-            'core' => '',
+            'typo3' => '10.4.0-10.4.99',
        ],
        'conflicts' => [],
        'suggests' => [],