Work-Related/abtest
1
0
mirror of https://github.com/werkraum-media/abtest.git synced 2024-11-15 02:16:11 +01:00
Code Issues Activity

Code improvements

Browse source
This commit is contained in:
Sven Wappler 2017-12-10 22:33:11 +01:00
parent 19ef78f214
commit afa42d5ac5
3 changed files with 93 additions and 95 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
Classes/Helper.php
View file

@ -10,6 +10,7 @@ namespace WapplerSystems\ABTest2;
*/ */
use TYPO3\CMS\Core\Utility\GeneralUtility; use TYPO3\CMS\Core\Utility\GeneralUtility;
use TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController;
use TYPO3\CMS\Frontend\Page\PageRepository; use TYPO3\CMS\Frontend\Page\PageRepository;
/** /**
@ -18,7 +19,8 @@ use TYPO3\CMS\Frontend\Page\PageRepository;
* @package WapplerSystems\ABTest2 * @package WapplerSystems\ABTest2
* @author Sven Wapler <typo3YYYYY@wappler.systems> * @author Sven Wapler <typo3YYYYY@wappler.systems>
*/ */
class Helper { class Helper
{
/** @var int|null */ /** @var int|null */
protected $currentPageId = null; protected $currentPageId = null;
@ -44,28 +46,29 @@ class Helper {
/** /**
* *
* @param array $params * @param array $params
* @param $pObj * @param $pObj TypoScriptFrontendController
* @return void * @return void
* @throws \InvalidArgumentException * @throws \InvalidArgumentException
*/ */
public function determineContentId(array $params, &$pObj) { public function determineContentId(array $params, &$pObj)
{
// only try to change the page if it's not the googlebot. // only try to change the page if it's not the googlebot.
if(false === stripos($_SERVER['HTTP_USER_AGENT'], 'googlebot')) { if (false === stripos($_SERVER['HTTP_USER_AGENT'], 'googlebot')) {
$this->currentPageId = $params['pObj']->id; $this->currentPageId = $pObj->id;
// Get the rootpage_id from realurl config. // Get the rootpage_id from realurl config.
$this->realurlConfig = $params['pObj']->TYPO3_CONF_VARS['EXTCONF']['realurl']; $this->realurlConfig = $pObj->TYPO3_CONF_VARS['EXTCONF']['realurl'];
if(array_key_exists($_SERVER['SERVER_NAME'], $this->realurlConfig)) { if (array_key_exists($_SERVER['SERVER_NAME'], $this->realurlConfig)) {
$this->rootpage_id = $this->realurlConfig[$_SERVER['SERVER_NAME']]['pagePath']['rootpage_id']; $this->rootpage_id = $this->realurlConfig[$_SERVER['SERVER_NAME']]['pagePath']['rootpage_id'];
} else { } else {
$this->rootpage_id = $this->realurlConfig['_DEFAULT']['pagePath']['rootpage_id']; $this->rootpage_id = $this->realurlConfig['_DEFAULT']['pagePath']['rootpage_id'];
} }
// If the ID is NULL, then we set this value to the rootpage_id. NULL is the "Home"page, ID is a specific sub-page, e.g. www.domain.de (NULL) - www.domain.de/page.html (ID) // If the ID is NULL, then we set this value to the rootpage_id. NULL is the "Home"page, ID is a specific sub-page, e.g. www.domain.de (NULL) - www.domain.de/page.html (ID)
if(!$this->currentPageId) { if (!$this->currentPageId) {
if($this->rootpage_id) { if ($this->rootpage_id) {
$this->currentPageId = $this->rootpage_id; $this->currentPageId = $this->rootpage_id;
} else { } else {
// Leave the function because we can not determine the ID. // Leave the function because we can not determine the ID.
@ -79,21 +82,21 @@ class Helper {
$this->selectBSite = $currentPagePropertiesArray['tx_abtest2_b_id']; $this->selectBSite = $currentPagePropertiesArray['tx_abtest2_b_id'];
$this->cookieLifeTime = $currentPagePropertiesArray['tx_abtest2_cookie_time']; $this->cookieLifeTime = $currentPagePropertiesArray['tx_abtest2_cookie_time'];
if($this->selectBSite) { if ($this->selectBSite) {
if((int)$_COOKIE['abtest2-'.$this->currentPageId] > 0) { if ((int)$_COOKIE['abtest2-' . $this->currentPageId] > 0) {
$this->randomAbPageId = (int)$_COOKIE['abtest2-'.$this->currentPageId]; $this->randomAbPageId = (int)$_COOKIE['abtest2-' . $this->currentPageId];
} else { } else {
$randomPage = rand(0,1); // 0 = original ID; 1 = "B" site. $randomPage = rand(0, 1); // 0 = original ID; 1 = "B" site.
if($randomPage) { if ($randomPage) {
$this->randomAbPageId = $this->selectBSite; $this->randomAbPageId = $this->selectBSite;
} else { } else {
$this->randomAbPageId = $this->currentPageId; $this->randomAbPageId = $this->currentPageId;
} }
setcookie('abtest2-'.$this->currentPageId,$this->randomAbPageId,time()+$this->cookieLifeTime); setcookie('abtest2-' . $this->currentPageId, $this->randomAbPageId, time() + $this->cookieLifeTime);
} }
// If current page ID is different from the random page ID we set the correct page ID. // If current page ID is different from the random page ID we set the correct page ID.
if($this->currentPageId != $this->randomAbPageId) { if ($this->currentPageId !== $this->randomAbPageId) {
$pObj->contentPid = $this->randomAbPageId; $pObj->contentPid = $this->randomAbPageId;
$GLOBALS['TSFE']->page['content_from_pid'] = $this->randomAbPageId; $GLOBALS['TSFE']->page['content_from_pid'] = $this->randomAbPageId;
$GLOBALS['TSFE']->page['no_cache'] = true; $GLOBALS['TSFE']->page['no_cache'] = true;
@ -103,7 +106,7 @@ class Helper {
// If additional headerdata is present then we specify additionalHeaderData. // If additional headerdata is present then we specify additionalHeaderData.
$randomPagePropertiesArray = $pageRepository->getPage($this->randomAbPageId); $randomPagePropertiesArray = $pageRepository->getPage($this->randomAbPageId);
$this->additionalHeaderData = $randomPagePropertiesArray['tx_abtest2_header']; $this->additionalHeaderData = $randomPagePropertiesArray['tx_abtest2_header'];
if($this->additionalHeaderData) { if ($this->additionalHeaderData) {
$GLOBALS['TSFE']->additionalHeaderData['abtest2'] = $this->additionalHeaderData; $GLOBALS['TSFE']->additionalHeaderData['abtest2'] = $this->additionalHeaderData;
} }

29
ext_emconf.php
View file

@ -10,34 +10,29 @@
* writing. "version" and "dependencies" must not be touched! * writing. "version" and "dependencies" must not be touched!
***************************************************************/ ***************************************************************/
$EM_CONF[$_EXTKEY] = array ( $EM_CONF[$_EXTKEY] = array(
'title' => 'AB Test Pages', 'title' => 'AB Test Pages',
'description' => 'This extension supports TYPO3 administrators in performing A/B tests. This is useful when a site owner want to measure whether a new version improves or reduces user interaction compared to the current version.', 'description' => 'With this extension, administrators can deliver different content for the same URL (AB test), depending on cookies or parameters.',
'category' => 'misc', 'category' => 'misc',
'author' => 'IllusionFACTORY', 'author' => 'Sven Wappler',
'author_email' => 'info@illusion-factory.de', 'author_email' => 'typo3YYYY@wappler.systems',
'state' => 'alpha', 'author_company' => 'WapplerSystems',
'state' => 'stable',
'uploadfolder' => false, 'uploadfolder' => false,
'createDirs' => '', 'createDirs' => '',
'clearCacheOnLoad' => 1, 'clearCacheOnLoad' => true,
'version' => '1.0.2.sw', 'version' => '0.1.0',
'constraints' => 'constraints' =>
array ( array(
'depends' => 'depends' =>
array ( array(
'typo3' => '7.6.0-8.7.99', 'typo3' => '7.6.0-8.7.99',
'realurl' => '2.0.0-0.0.0', 'realurl' => '2.0.0-0.0.0',
), ),
'conflicts' => 'conflicts' =>
array ( array(),
),
'suggests' => 'suggests' =>
array ( array(),
), ),
),
'comment' => '',
'user' => 'timof',
'clearcacheonload' => true,
'author_company' => NULL,
); );

2
ext_localconf.php
View file

@ -3,4 +3,4 @@
defined('TYPO3_MODE') or die(); defined('TYPO3_MODE') or die();
$GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['tslib/class.tslib_fe.php']['determineId-PostProc']['abtest2'] = 'WapplerSystems\\ABTest2\\ShowPage->SelectId'; $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['tslib/class.tslib_fe.php']['determineId-PostProc']['abtest2'] = 'WapplerSystems\\ABTest2\\Helper->SelectId';