[TYPO3-core] RFC: #7333: Solution for "Page is being generated"

Ingo Renner ingo at typo3.org
Thu Jan 31 23:08:22 CET 2008


Bernhard Kraft wrote:

Hi Bernhard,

> BT:
> http://bugs.typo3.org/view.php?id=7333
> 
> Problem:
> Get rid of this "Page is being generated" screen ...

tested: works, couldn't get PIBG messages anymore! (note: I had to 
create typo3temp/pagegen_lock/ with correct access rights before)

some remarks on the patch itself:

* lockPageGenerate -> lockPageGeneration
* please split lockPageGenerate into lock... and unlock...
* lockPageGenerate is missing phpDoc
* please transform lockPageGenerate so that you have just one single 
return statement
* in index_tx.php please find a better variable name than $ok - what 
exactly is ok?
* please clean this block up by properly using curly braces, otherwise 
it's really hard to understand:

+		if ($TSFE->TYPO3_CONF_VARS['FE']['disablePageIsBeingGenerated']>0)	{
+			$ok = $TSFE->lockPageGenerate(1);
+			if (!$ok)	{
+			if ($GLOBALS['debugPageIsBeingGenerated']) debug_PIBG("aquired 
blocking lock - rereading cache ...");
+				$TSFE->getFromCache();
+			} else	{
+				if ($GLOBALS['debugPageIsBeingGenerated']) debug_PIBG("aquired 
nonblocking lock ...");
+			}
+		}


so +1 after testing and reading if you make these changes.


all the best
Ingo

-- 
Ingo Renner
TYPO3 Core Developer, Release Manager TYPO3 4.2


More information about the TYPO3-team-core mailing list