[PHP] Raar probleem met aanpassen variabele

[tomk94]

Hallo allemaal,

Ik heb momenteel een heel raar probleem met een stukje php.

Ik heb namelijk de volgende code:

PHP:

public function generateList($startTime = false, $middleTime = false, $endTime = false, $itemsToUse = false) {
            $dutchDayNames = array('' ,'ma','di','wo','do','vr','vr','za','zo');
            if($startTime !== false) $this->startTime = $startTime;
            if($endTime !== false) $this->endTime = $endTime;
            if(($k2Items = $this->getK2Items()) == false) return false;
            $dates = array();
            $beginMonth = false;
            $showEndMonth = false;
            $endMonth = false;
            $returnHTML = '';
            $colspan = 0;
            for($time=$this->startTime;$time<=$this->endTime;$time=$time+(24 * 60 * 60)) {
                if($beginMonth == false) { $beginMonth = date('m', $time); $beginYear = date('Y', $time); }
                if(date('m', $time) != $beginMonth) { $showEndMonth = true; $endMonth = date('m', $time); $endYear = date('Y', $time); }
                $dates[] = array('day'=>date('d', $time), 'month'=>date('m', $time), 'year'=>date('Y', $time), 'time'=>$this->toCorrectTime($time));
                $colspan = $colspan + 1;
            }
            
            $returnHTML .= '<div class="availability_clearer"></div><table class="availability_table">';
            $returnHTML .= '<tr><td colspan="1">&nbsp;</td><td colspan="'.floor($colspan / 2).'" class="availability_beginMonth">'.$this->monthNames[($beginMonth/1)].' '.$beginYear.' </td>';
            if($showEndMonth) $returnHTML .= '<td colspan="'.ceil($colspan / 2).'" class="availability_endMonth">'.$this->monthNames[($endMonth/1)].' '.$endYear.'</td>';
            $returnHTML .='</tr>';
            
            $returnHTML .='<tr><td class="availability_table_title">&nbsp;</td>';
            
            foreach($dates as $date) {
                $returnHTML .= '<td>'.$dutchDayNames[(date('w', $date['time'])/1)].'<br />'.($date['day'] == date('d', $middleTime) ? '<strong>'.$date['day'].'</strong>' : $date['day'] ) . '</td>';
            }
            
            $returnHTML .= '</tr>';
            $k2Items = $this->array_sort($k2Items);
    
            foreach($k2Items as $item)  {
                if(is_array($itemsToUse) && !in_array($item->id, $itemsToUse)) {} else {
                $h2 = ($this->getCategoryName($item->catid) != 'Accommodatie') ? true : false;
                $returnHTML .= '<tr><td class="availability_table_title"><a href="'.JRoute::_(($h2 ? "index.php?option=com_k2&view=item&id=".$item->id : "index.php?option=com_availability&Itemid=66&view_id=".$item->id)).'">'.($h2 ? '<h2>' : '').$item->title.' <small>['.$this->getCategoryName($item->catid).']</small>'.($h2 ? '</h2>' : '').'</a></td>';
                foreach($dates as $date) {
                    $av = $this->getAvailability($item->id, $date['time']);
                    if($av == 'unknown' ||  $av == 'booked' && $av == 'part') {
                        $returnHTML .= '<td class="test"><img src="components/com_availability/img/'.$av.'.png" style="width: 100%;height: 100%;"></td>';
                    } else {
                        $returnHTML .= '<td class="test"><img src="components/com_availability/img/available.png" style="width: 100%;height: 100%;"></td>';
                    }
                }
                $returnHTML .= '</tr>';
                }
            }
            $returnHTML .= '</table>';
            
            return $returnHTML;
        }

Dit stukje zorgt voor een lege pagina. Ik heb error_reporting op E_ALL staan, en display_errors op 1, maar er verschijnen geen error. Als ik expres een parse error tevoorschijn tover is er wel een error. Nou ben ik er achter gekomen wat voor de witte pagina zorgt.

Dat doen namelijk regel 40 en 42 in het stukje hierboven. Deze regels zijn bijna hetzelfde. Hieronder heb ik regel 40 nog een keer herhaald:

PHP:

$returnHTML .= '<td class="test"><img src="components/com_availability/img/'.$av.'.png" style="width: 100%;height: 100%;"></td>';

Wanneer ik class="test" weghaal, werkt alles opeens weer. En als ik bijvoorbeeld id="test" toevoeg, werkt het ook. Maar weer niet als ik cccc="test" doe... Ik dacht dus dat php class als nieuwe klasse zag, maar aangezien ccc="test" ook niet werkt, is dat het niet.

Weet iemand waar dit anders mis kan gaan? Wie helpt me uit de brand

En owja, zo roep ik het aan:

PHP:

<?php 
echo $availability->generateList($beginTime, $middleTime, $endTime, $itemsToUse);
?>

.

Als ik dit weghaal werkt het ook gewoon...

[Janoz]

Wat je je goed moet beseffen is dat je een serverside deel hebt en een clientside deel. Wanneer je pagina "leeg blijft", kijk dan ook even naar de source (en dan bedoel ik de source in je browser, niet je php source). Zeer waarschijnlijk wordt alles dan wel gewoon opgestuurd. Je moet in dat geval je probleem niet in je php, maar in je html zoeken.

[Matis]

Misschien niet geheel irrelevant; Wat staat er in je CSS betreffende de td.test.

[X_lawl_X]

Bekijk de source van de pagina eens.

edit: Janoz

[ Voor 20% gewijzigd door X_lawl_X op 11-04-2011 15:49 ]

[TJHeuvel]

Wat je evt ook nog kan doen is de verloop van $returnHTML bekijken, misschien wordt hij op een gegeven moment ook wel leeg in de functie.

[tomk94]

Bron is echt leeg volgens firefox(Cmd+U). Dus probleem ligt serverside.

CyCloneNL schreef op maandag 11 april 2011 @ 15:54:
Wat je evt ook nog kan doen is de verloop van $returnHTML bekijken, misschien wordt hij op een gegeven moment ook wel leeg in de functie.

Bedankt, ik zal alles nakijken.

[chaozz]

Kwestie van debuggen. Die fout moet je zo gevonden hebben. Zet wat "echo's" in je bron op logische punten en bekijk het verloop.

[H3llrais3r]

Hmm wat mij opvalt in regel 2 is dat je 2x vr in je array stopt...

[AlexAuto]

Wat is de uitkomst van de code op regel 5?

code:

$this->getK2Items()

Regel 39 is trouwens buggy:

code:

if($av == 'unknown' ||  $av == 'booked' && $av == 'part') {

Een kwestie van debuggen en goede breakpoints neerzetten.

[tomk94]

Mmm, ben erachter gekomen dat het probleem waarschijnlijk niet aan mij kant ligt maar aan die van joomla.

De laatste 3 regels zijn:

<?php
$availability->generateList($beginTime, $middleTime, $endTime, $itemsToUse);
?>

Wanneer ik hierin exit() zegt(na de generatelist), werkt het.

De verwerking gaat dus mis. Dat word nog leuk uitpluizen in joomla!....

[tomk94]

Ben nu meer dan 1 week verder, en heb het nog steeds niet opgelost.Heb delen opnieuw geschreven maar niks gevonden.

Zijn er misschien dingen in de bovenstaande code die geen errors veroorzaken, en meteen php eruit knallen?

[NMe]

Daar kom je natuurlijk redelijk makkelijk zelf achter door spul even uit te commenten en te kijken of je dan wel output krijgt. Debuggen zul je toch echt zelf moeten doen.

[tomk94]

NMe schreef op donderdag 21 april 2011 @ 19:51:
Daar kom je natuurlijk redelijk makkelijk zelf achter door spul even uit te commenten en te kijken of je dan wel output krijgt. Debuggen zul je toch echt zelf moeten doen.

Ja dat heb ik gedaan, maar dat heeft me weinig geholpen.

Of als ik de 2 returnHTML hieronder comment werkt het:

PHP:

if($av == 'unknown' ||  $av == 'booked' && $av == 'part') {
                        $returnHTML .= '<td class="test"><img src="components/com_availability/img/'.$av.'.png" style="width: 100%;height: 100%;"></td>';
                    } else {
                        $returnHTML .= '<td class="test"><img src="components/com_availability/img/available.png" style="width: 100%;height: 100%;"></td>';
                    }

Of als ik de volgende comment:

PHP:

$returnHTML .= '<tr><td class="availability_table_title"><a href="'.JRoute::_(($h2 ? "index.php?option=com_k2&view=item&id=".$item->id : "index.php?option=com_availability&Itemid=66&view_id=".$item->id)).'">'.($h2 ? '<h2>' : '').$item->title.' <small>['.$this->getCategoryName($item->catid).']</small>'.($h2 ? '</h2>' : '').'</a></td>';

Dus als ik een van de bovenste 2 comment (// ervoor) werkt het...

En laat ik in die regels nou net geen probleem zien:( Ik snap het echt niet meer. Nog effe en ik begin van 0.

Edit
Lijkt haast wel of ik over een of andere limiet heen ga want:

PHP:

echo '<td><img src="components/com_availability/img/'.$av.'.png" style="width: 100%;height: 100%;" class="ad" id="abcdefg"></td>';

werkt niet, en:

PHP:

echo '<td><img src="components/com_availability/img/'.$av.'.png" style="width: 100%;height: 100%;" class="ad"></td>';

Werkt wel:S Weet iemand of dit kan?

[ Voor 12% gewijzigd door tomk94 op 21-04-2011 20:53 ]

[Voutloos]

tomwebfreak schreef op donderdag 21 april 2011 @ 19:48:
Ben nu meer dan 1 week verder, en heb het nog steeds niet opgelost.

Maar je negeert ook al 10 dagen de hint van AlexAuto, die toch echt terecht op een logica fout in r39 wijst...

[skabouter]

Voutloos schreef op donderdag 21 april 2011 @ 21:34:
[...]
Maar je negeert ook al 10 dagen de hint van AlexAuto, die toch echt terecht op een logica fout in r39 wijst...

Die fout zorgt er alleen voor dat regel 42 ipv 40 wordt uitgevoerd of andersom, aangezien het probleem op beide regels voorkomt maakt dat dus wezenlijk geen verschil voor dit probleem.

Wat is de output aan de browser kant? Helemaal niets of onjuiste HTML? In het eerste geval kun je kijken of je error reporting/logging aan staat en of er iets in je apache logs staat.

[edit]
Code even doorgelopen en lijkt verder ok, al kan ik uiteraard niet de functie aanroepen die je doet nagaan...

[ Voor 9% gewijzigd door skabouter op 21-04-2011 22:02 ]

[tomk94]

skabouter schreef op donderdag 21 april 2011 @ 21:48:
[...]


Die fout zorgt er alleen voor dat regel 42 ipv 40 wordt uitgevoerd of andersom, aangezien het probleem op beide regels voorkomt maakt dat dus wezenlijk geen verschil voor dit probleem.

Wat is de output aan de browser kant? Helemaal niets of onjuiste HTML? In het eerste geval kun je kijken of je error reporting/logging aan staat en of er iets in je apache logs staat.

Error reporting staat op z'n hoogst en ik krijg dan allemaal strict errors van Joomla!, maar geen van mijn script. Het lijkt haast wel of er iets is waardoor heel PHP vastloopt... Ik ga de apache logs erbij pakken

[YopY]

Doe eens een debugger eraan hangen en er regel voor regel doorheen bladeren, dan kun je precies volgen wat hij doet en (evt) wanneer hij eruit knalt.

[lauwsa]

Eerste wat ik in je code zie:

code:

if($av == 'unknown' ||  $av == 'booked' && $av == 'part') {

Hoe kan $av ooit booked en part tegelijk zijn?
Maar goed dat hadden andere ook gemeld maar kan het probleem niet zijn omdat je dan wel output zou moeten krijgen.

Kijk eens voor de grap hoe ver je in de code komt, en als hij afhaakt kijk je waarom hij dat doet. je kan wel gaan raden maar dat heeft ook geen zin. Voert hij wel een gedeelte van de code uit? loopt hij niet hier vast?

code:

if(($k2Items = $this->getK2Items()) == false) return false;

etc etc

[Oid]

check je error log van je webserver eens, wat zegt die? Daar kan ik vaak ook nog wel data uithalen....

[Ventieldopje]

PHP:

<?php
public function generateList($startTime = false, $middleTime = false, $endTime = false, $itemsToUse = false) {
            $returnHTML = 'hierjouhelelangestringmetallemaalbullshit'; // en dat nog iets langer dan
            
            return $returnHTML;
        }
?>

Als je tegen een of ander limiet aan zou lopen zou je dat zo makkelijk kunnen testen Bovendien kan ik je aanraden je code eens op te ruimen!

[T020]

In deze regel staat een fout:

PHP:

$returnHTML .= '<tr><td class="availability_table_title"><a href="'.JRoute::_(($h2 ? "index.php?option=com_k2&view=item&id=".$item->id : "index.php?option=com_availability&Itemid=66&view_id=".$item->id)).'">'.($h2 ? '<h2>' : '').$item->title.' <small>['.$this->getCategoryName($item->catid).']</small>'.($h2 ? '</h2>' : '').'</a></td>';

view_id=".$item je gebruikt daar als ik me niet vergis een double quote terwijl de string begint met een single quote, dus vervang dit eens door view_id='.$item en kijk wat het doet.
En zoals Ventieldopje al zegt, ruim je code even op Kleine foutjes zijn veel makkelijker te ontdekken wanneer je alleen al wat meer consistente indenting gebruikt.
Kijk ook eens naar de PHP functie sprintf, dat gebruik ik altijd om dit soort problemen met quotes te voorkomen

[Borizz]

T020 schreef op maandag 25 april 2011 @ 13:33:
In deze regel staat een fout:

PHP:

$returnHTML .= '<tr><td class="availability_table_title"><a href="'.JRoute::_(($h2 ? "index.php?option=com_k2&view=item&id=".$item->id : "index.php?option=com_availability&Itemid=66&view_id=".$item->id)).'">'.($h2 ? '<h2>' : '').$item->title.' <small>['.$this->getCategoryName($item->catid).']</small>'.($h2 ? '</h2>' : '').'</a></td>';

view_id=".$item je gebruikt daar als ik me niet vergis een double quote terwijl de string begint met een single quote, dus vervang dit eens door view_id='.$item en kijk wat het doet.
En zoals Ventieldopje al zegt, ruim je code even op Kleine foutjes zijn veel makkelijker te ontdekken wanneer je alleen al wat meer consistente indenting gebruikt.
Kijk ook eens naar de PHP functie sprintf, dat gebruik ik altijd om dit soort problemen met quotes te voorkomen

Kijk nog eens naar de syntax hilighting en de code die je zelf post. De string die wordt afgesloten begon ook met een dubbele quote. Wat je zegt is dus onzin.
Als het wel zo zou zijn, had het ook een parse error opgeleverd in PHP en zou de melding ook door PHP gerapporteerd zijn omdat de TS aangeeft error_reporting op E_ALL te hebben staan.

[Wizz15]

Thijmen schreef op zaterdag 23 april 2011 @ 23:24:
check je error log van je webserver eens, wat zegt die? Daar kan ik vaak ook nog wel data uithalen....

Ventieldopje schreef op zondag 24 april 2011 @ 22:45:
Als je tegen een of ander limiet aan zou lopen zou je dat zo makkelijk kunnen testen Bovendien kan ik je aanraden je code eens op te ruimen!

Als het een limiet is, dan staat het vrijwel zeker in de error logs van je webserver. Zou de TS willen aanraden om daar even te kijken. Het is de eerste plek waar ik kijk als ik weer eens een lege pagina voor mijn kiezen krijg

[ Voor 36% gewijzigd door Wizz15 op 25-04-2011 14:27 ]

[bindsa]

Duur het laden van de pagina door de browser ook erg lang? In dat geval moet je het namelijk in een of andere loop zoeken, dat heb ik ook wel eens, i.p.v. een time-out error een white screen.

[fikst]

Ik heb ooit een vergelijkbaar probleem gehad met Joomla. Na dagen zoeken en debuggen ben ik er bij uit gekomen dat dit voorvalt wanneer je output zeer lang is, en alles achter elkaar staat, zonder newlines tussen.

Probeer eens achter elke <tr></tr> "\r\n" te echo'en.

[skabouter]

T020 schreef op maandag 25 april 2011 @ 13:33:
In deze regel staat een fout:

PHP:

$returnHTML .= '<tr><td class="availability_table_title"><a href="'.JRoute::_(($h2 ? "index.php?option=com_k2&view=item&id=".$item->id : "index.php?option=com_availability&Itemid=66&view_id=".$item->id)).'">'.($h2 ? '<h2>' : '').$item->title.' <small>['.$this->getCategoryName($item->catid).']</small>'.($h2 ? '</h2>' : '').'</a></td>';

view_id=".$item je gebruikt daar als ik me niet vergis een double quote terwijl de string begint met een single quote, dus vervang dit eens door view_id='.$item en kijk wat het doet.
En zoals Ventieldopje al zegt, ruim je code even op Kleine foutjes zijn veel makkelijker te ontdekken wanneer je alleen al wat meer consistente indenting gebruikt.
Kijk ook eens naar de PHP functie sprintf, dat gebruik ik altijd om dit soort problemen met quotes te voorkomen

Deze code is correct, hij gebruikt echter niet concequent " en ' voor PHP strings, maar hij eindigd het voorgaande blok met " en start een nieuw block met '. Hoewel dit niet netjes is werkt het uiteraard wel, zolang het eerste blok met een " wodt gestart en het laatste blok maar met een ' eindigd.