php Syntax-error

Tease · 11.01.2011

Hallo zusammen,
ich bin gerade dabei einen Magento-Webshop aufzusetzen und habe nun im Zuge dessen eine Extension installiert, bei der sich leider scheinbar ein Problem eingeschlichen hat. Leider kenn ich mich mit php noch nicht wirklich aus, und deswegen kann ich mit der angezeigten Fehlermeldung nicht so viel anfangen, bzw. denke ich, dass die angezeigte Stelle im Code korrektt ist.
Vielleicht ist jemand hier so nett und gibt mir einen kleinen Denkanstoß:

Hier erstmal die Fehlermeldung, die im Browser angezeigt wirde, wenn ich die Site aufrufen möchte:

Parse error: syntax error, unexpected '<' in /var/www/magento/app/code/local/TM/Featured/Block/Featured.php on line 41

Okay, so weit ich das richtig verstanden habe, stimmt was in Zeile 41 nicht.
Hier die entsprechende Stelle der php-Datei (tor markiert ist Zeile 41):

public function getProductsCount()
{
if (null === $this->_productsCount) {
$this->_productsCount = self:

EFAULT_PRODUCTS_COUNT;
}
return $this->_productsCount;
}
}<?php

Kann mir jemand sagen, was hier flasch ist?
Vielen Dank schon mal im voraus!

LG
Tease

MyBad · 11.01.2011

AW: php Syntax-error

Hi!

Mach doch mal ein Leerzeichen zwischen "}" und "<?php"! Oder noch besser schreib es gleich in eine neue Zeile.

cebito · 11.01.2011

AW: php Syntax-error

Da steht doch schon PHP-Code, warum dann nochmal "<?php"? Wenn dann müsste davor doch "?>" stehen

Tease · 11.01.2011

AW: php Syntax-error

3 Minuten nach Thread-Erstellung schon die erste Antwort! Rekordverdächtig!
Leider hat dein Vorschlag aber das Problem noch nicht gelöst, da jetzt ( ich habe "<?php"! in eine neue Zeile geschrieben) der Fehler in Zeile 42, also in der nächsten Zeile, liegt.

Hmm, aber trotzdem Danke!

gurke · 11.01.2011

AW: php Syntax-error

if (null === $this->_productsCount) {
... zumindest ist da ein = zuviel.

Tease · 11.01.2011

AW: php Syntax-error

cebito schrieb:
Da steht doch schon PHP-Code, warum dann nochmal "<?php"? Wenn dann müsste davor doch "?>" stehen

Zuerst einmal vielen Dank.
Ich habe deine Vorschlag umgesetzt und tatsächlich wird mir nun die Startseite angezeigt, allerdings nicht wie erhofft, sondern mit komischen Zeichen im Header: (oder siehe Screenshot)

class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract { protected $_productsCount = null; const DEFAULT_PRODUCTS_COUNT = 5; protected function _beforeToHtml() { $collection = Mage::getResourceModel('catalog/product_collection'); Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection); Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection); $collection = $this->_addProductAttributesAndPrices($collection) ->addStoreFilter(Mage::app()->getStore()->getId()) ->addAttributeToFilter('featured', array('Yes' => true)) ->setPageSize($this->getProductsCount()) ->setCurPage(1); $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml'); $this->setProductCollection($collection); return parent::_beforeToHtml(); } public function setProductsCount($count) { $this->_productsCount = $count; return $this; } public function getProductsCount() { if (null === $this->_productsCount) { $this->_productsCount = self:

EFAULT_PRODUCTS_COUNT; } return $this->_productsCount; } } ?> class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract { protected $_productsCount = null; const DEFAULT_PRODUCTS_COUNT = 5; protected function _beforeToHtml() { $collection = Mage::getResourceModel('catalog/product_collection'); Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection); Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection); $collection = $this->_addProductAttributesAndPrices($collection) ->addStoreFilter(Mage::app()->getStore()->getId()) ->addAttributeToFilter('featured', array('Yes' => true)) ->setPageSize($this->getProductsCount()) ->setCurPage(1); $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml'); $this->setProductCollection($collection); return parent::_beforeToHtml(); } public function setProductsCount($count) { $this->_productsCount = $count; return $this; } public function getProductsCount() { if (null === $this->_productsCount) { $this->_productsCount = self:

EFAULT_PRODUCTS_COUNT; } return $this->_productsCount; } }

Uploaded with

Huch, das sieht irgendwie gar nicht gut aus.

cebito · 11.01.2011

AW: php Syntax-error

gurke schrieb:
if (null === $this->_productsCount) {
... zumindest ist da ein = zuviel.

Als Steigerung der Vergleichsoperatoren == und != gibt es noch die Operatoren === und !==. Dabei wird neben den Wert auch der Typ überprüft. Somit sind die Werte "10" (als String) und 10 (als Integer) gleich wenn man == verwendet, jedoch unterschiedlich wenn man === verwendet.

@tease - hast den Block denn zwischen den php-tags?

gurke · 11.01.2011

AW: php Syntax-error

Als Steigerung der Vergleichsoperatoren == und != gibt es noch die Operatoren === und !==. Dabei wird neben den Wert auch der Typ überprüft. Somit sind die Werte "10" (als String) und 10 (als Integer) gleich wenn man == verwendet, jedoch unterschiedlich wenn man === verwendet. Als Steigerung der Vergleichsoperatoren == und != gibt es noch die Operatoren === und !==. Dabei wird neben den Wert auch der Typ überprüft. Somit sind die Werte "10" (als String) und 10 (als Integer) gleich wenn man == verwendet, jedoch unterschiedlich wenn man === verwendet.

- oh, danke, das war mir neu. Gucke ich mir mal an.

Tholi · 11.01.2011

AW: php Syntax-error

Man könnte meinen das der erste Teil nicht im php Teil steht?

Tease · 11.01.2011

AW: php Syntax-error

Da habt ihr beiden Recht. Wenn ich mir den Quelltext so ansehe, wir mir schlecht. Die fehlerhaft angezeigten Code-Schnipsel erscheinen im Quelltext der Homepage noch VOR dem DOCTYPE. Das kann ja nicht stimmen! Jetzt stellt sich nur die Frage, warum das so ist, und wie ich das ändern kann. Wäre es einfach 'ne index.php könnte man da ja Hand anlegen, aber da es sich um ein CMS-System handelt, weiß ich nicht in welcher Datei ich das ausbessern muss. Kann man das irgendwie rausfinden? Also, dass es heißt die Datei xy im Ordner sonundso. Ansonsten wüsste ich nicht, wie ich das ändern kann.

Hier der fehlerhafte Quelltext:

Code:

class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract
{
    protected $_productsCount = null;
    const DEFAULT_PRODUCTS_COUNT = 5;
    
    protected function _beforeToHtml()
    {
        $collection = Mage::getResourceModel('catalog/product_collection');
        
        Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection);
        Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection);
        
        $collection = $this->_addProductAttributesAndPrices($collection)
            ->addStoreFilter(Mage::app()->getStore()->getId())
            ->addAttributeToFilter('featured', array('Yes' => true))
            ->setPageSize($this->getProductsCount())
            ->setCurPage(1);
        $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml');
        $this->setProductCollection($collection);
        
        return parent::_beforeToHtml();
    }
    
    public function setProductsCount($count)
    {
        $this->_productsCount = $count;
        return $this;
    }
    public function getProductsCount()
    {
        if (null == $this->_productsCount) {
            $this->_productsCount = self::DEFAULT_PRODUCTS_COUNT;
        }
        return $this->_productsCount;
    }
}
?>
class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract
{
    protected $_productsCount = null;
    const DEFAULT_PRODUCTS_COUNT = 5;
    
    protected function _beforeToHtml()
    {
        $collection = Mage::getResourceModel('catalog/product_collection');
        
        Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection);
        Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection);
        
        $collection = $this->_addProductAttributesAndPrices($collection)
            ->addStoreFilter(Mage::app()->getStore()->getId())
            ->addAttributeToFilter('featured', array('Yes' => true))
            ->setPageSize($this->getProductsCount())
            ->setCurPage(1);
        $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml');
        $this->setProductCollection($collection);
        
        return parent::_beforeToHtml();
    }
    
    public function setProductsCount($count)
    {
        $this->_productsCount = $count;
        return $this;
    }
    public function getProductsCount()
    {
        if (null == $this->_productsCount) {
            $this->_productsCount = self::DEFAULT_PRODUCTS_COUNT;
        }
        return $this->_productsCount;
    }
}
?>
class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract
{
    protected $_productsCount = null;
    const DEFAULT_PRODUCTS_COUNT = 5;
    
    protected function _beforeToHtml()
    {
        $collection = Mage::getResourceModel('catalog/product_collection');
        
        Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection);
        Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection);
        
        $collection = $this->_addProductAttributesAndPrices($collection)
            ->addStoreFilter(Mage::app()->getStore()->getId())
            ->addAttributeToFilter('featured', array('Yes' => true))
            ->setPageSize($this->getProductsCount())
            ->setCurPage(1);
        $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml');
        $this->setProductCollection($collection);
        
        return parent::_beforeToHtml();
    }
    
    public function setProductsCount($count)
    {
        $this->_productsCount = $count;
        return $this;
    }
    public function getProductsCount()
    {
        if (null == $this->_productsCount) {
            $this->_productsCount = self::DEFAULT_PRODUCTS_COUNT;
        }
        return $this->_productsCount;
    }
}
?>
class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract
{
    protected $_productsCount = null;
    const DEFAULT_PRODUCTS_COUNT = 5;
    
    protected function _beforeToHtml()
    {
        $collection = Mage::getResourceModel('catalog/product_collection');
        
        Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection);
        Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection);
        
        $collection = $this->_addProductAttributesAndPrices($collection)
            ->addStoreFilter(Mage::app()->getStore()->getId())
            ->addAttributeToFilter('featured', array('Yes' => true))
            ->setPageSize($this->getProductsCount())
            ->setCurPage(1);
        $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml');
        $this->setProductCollection($collection);
        
        return parent::_beforeToHtml();
    }
    
    public function setProductsCount($count)
    {
        $this->_productsCount = $count;
        return $this;
    }
    public function getProductsCount()
    {
        if (null == $this->_productsCount) {
            $this->_productsCount = self::DEFAULT_PRODUCTS_COUNT;
        }
        return $this->_productsCount;
    }
}<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "[URL]http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd[/URL]">
<html xmlns="[url=http://www.w3.org/1999/xhtml]XHTML namespace[/url]" xml:lang="de" lang="de">
<head>
 
 und so weiter und so fort....

Oh man, immer dieses Problem mit den Magento-Extensions.....
Aber Dank eurer Hilfe, bin ich noch nicht ganz verzweifelt! ;-)

Duddle · 11.01.2011

AW: php Syntax-error

Wenn du jedes öffnende <?php entfernst (was du offensichtlich gemacht hast), kann das ja nichts werden. Entferne alle überflüssigen schliessenden ?> und umfasse den gesamten Codeblock vor dem Doctype mit <?php und ?>.

Tease schrieb:
Die fehlerhaft angezeigten Code-Schnipsel erscheinen im Quelltext der Homepage noch VOR dem DOCTYPE. Das kann ja nicht stimmen!

Doch, das stimmt schon. PHP ist es völlig egal, wo es ausgeführt wird.

Duddle

Doitsu · 11.01.2011

AW: php Syntax-error

Dir wurde oben eine tolle Seite genannt (), geh am besten mal die ganze Seite durch - dir fehlen eindeutig die Grundlagen.

Tease · 11.01.2011

AW: php Syntax-error

Duddle schrieb:
Wenn du jedes öffnende <?php entfernst (was du offensichtlich gemacht hast), kann das ja nichts werden. Entferne alle überflüssigen schliessenden ?> und umfasse den gesamten Codeblock vor dem Doctype mit <?php und ?>.

Doch, das stimmt schon. PHP ist es völlig egal, wo es ausgeführt wird.

Duddle

Danke für deine Antwort!

Ich habe nun alle die mitendrin im Code stehen gelöscht, aber nun habe ich wieder die ursprüngliche (nur in einer anderen Zeile) Fehlermeldung.

Hier der Code:

PHP:

<?php

class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract
{
    protected $_productsCount = null;

    const DEFAULT_PRODUCTS_COUNT = 5;
    
    protected function _beforeToHtml()
    {
        $collection = Mage::getResourceModel('catalog/product_collection');
        
        Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection);
        Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection);
        
        $collection = $this->_addProductAttributesAndPrices($collection)
            ->addStoreFilter(Mage::app()->getStore()->getId())
            ->addAttributeToFilter('featured', array('Yes' => true))
            ->setPageSize($this->getProductsCount())
            ->setCurPage(1);

        $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml');
        $this->setProductCollection($collection);
        
        return parent::_beforeToHtml();
    }
    
    public function setProductsCount($count)
    {
        $this->_productsCount = $count;
        return $this;
    }

    public function getProductsCount()
    {
        if (null === $this->_productsCount) {
            $this->_productsCount = self::DEFAULT_PRODUCTS_COUNT;
        }
        return $this->_productsCount;
    }
}

class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract
{
    protected $_productsCount = null;

    const DEFAULT_PRODUCTS_COUNT = 5;
    
    protected function _beforeToHtml()
    {
        $collection = Mage::getResourceModel('catalog/product_collection');
        
        Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection);
        Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection);
        
        $collection = $this->_addProductAttributesAndPrices($collection)
            ->addStoreFilter(Mage::app()->getStore()->getId())
            ->addAttributeToFilter('featured', array('Yes' => true))
            ->setPageSize($this->getProductsCount())
            ->setCurPage(1);

        $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml');
        $this->setProductCollection($collection);
        
        return parent::_beforeToHtml();
    }
    
    public function setProductsCount($count)
    {
        $this->_productsCount = $count;
        return $this;
    }

    public function getProductsCount()
    {
        if (null == $this->_productsCount) {
            $this->_productsCount = self::DEFAULT_PRODUCTS_COUNT;
        }
        return $this->_productsCount;
    }
}

class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract
{
    protected $_productsCount = null;

    const DEFAULT_PRODUCTS_COUNT = 5;
    
    protected function _beforeToHtml()
    {
        $collection = Mage::getResourceModel('catalog/product_collection');
        
        Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection);
        Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection);
        
        $collection = $this->_addProductAttributesAndPrices($collection)
            ->addStoreFilter(Mage::app()->getStore()->getId())
            ->addAttributeToFilter('featured', array('Yes' => true))
            ->setPageSize($this->getProductsCount())
            ->setCurPage(1);

        $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml');
        $this->setProductCollection($collection);
        
        return parent::_beforeToHtml();
    }
    
    public function setProductsCount($count)
    {
        $this->_productsCount = $count;
        return $this;
    }

    public function getProductsCount()
    {
        if (null == $this->_productsCount) {
            $this->_productsCount = self::DEFAULT_PRODUCTS_COUNT;
        }
        return $this->_productsCount;
    }
}

class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract
{
    protected $_productsCount = null;

    const DEFAULT_PRODUCTS_COUNT = 5;
    
    protected function _beforeToHtml()
    {
        $collection = Mage::getResourceModel('catalog/product_collection');
        
        Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection);
        Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection);
        
        $collection = $this->_addProductAttributesAndPrices($collection)
            ->addStoreFilter(Mage::app()->getStore()->getId())
            ->addAttributeToFilter('featured', array('Yes' => true))
            ->setPageSize($this->getProductsCount())
            ->setCurPage(1);

        $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml');
        $this->setProductCollection($collection);
        
        return parent::_beforeToHtml();
    }
    
    public function setProductsCount($count)
    {
        $this->_productsCount = $count;
        return $this;
    }

    public function getProductsCount()
    {
        if (null == $this->_productsCount) {
            $this->_productsCount = self::DEFAULT_PRODUCTS_COUNT;
        }
        return $this->_productsCount;
    }
}

class TM_Featured_Block_Featured extends Mage_Catalog_Block_Product_Abstract
{
    protected $_productsCount = null;

    const DEFAULT_PRODUCTS_COUNT = 5;
    
    protected function _beforeToHtml()
    {
        $collection = Mage::getResourceModel('catalog/product_collection');
        
        Mage::getSingleton('catalog/product_status')->addVisibleFilterToCollection($collection);
        Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($collection);
        
        $collection = $this->_addProductAttributesAndPrices($collection)
            ->addStoreFilter(Mage::app()->getStore()->getId())
            ->addAttributeToFilter('featured', array('Yes' => true))
            ->setPageSize($this->getProductsCount())
            ->setCurPage(1);

        $this->addPriceBlockType('bundle', 'bundle/catalog_product_price', 'bundle/catalog/product/price.phtml');
        $this->setProductCollection($collection);
        
        return parent::_beforeToHtml();
    }
    
    public function setProductsCount($count)
    {
        $this->_productsCount = $count;
        return $this;
    }

    public function getProductsCount()
    {
        if (null == $this->_productsCount) {
            $this->_productsCount = self::DEFAULT_PRODUCTS_COUNT;
        }
        return $this->_productsCount;
    }
}
?>

und hier die darus resultierende Meldung vom Browser:
Fatal error: Cannot redeclare class TM_Featured_Block_Featured in /var/www/app/code/local/TM/Featured/Block/Featured.php on line 81

Ich vermute, dass ich nun wieder etwas zuviel gelöscht habe, kann aber nicht sagen, was.

Danke, für's zeitnehmehmen!

Und ja, ich wer mir gleichzeitig auch den Link mit den php_Grundlagen durchsehen, nur dauert das halt bis man da mal durchblickt, deswegen wäre ich euch dankbar, wenn ihr mir da weiterhelft! Aber Danke auch für den Link! ;-)

Doitsu · 11.01.2011

AW: php Syntax-error

Die Fehlermeldung sagt doch schon alles?! Hast du dir mal die Zeile angeschaut?
Du hast die Klasse 'TM_Featured_Block_Featured' 5x drin, eine Klasse darf aber nur ein mal definiert werden.

saila · 11.01.2011

AW: php Syntax-error

Hi,

du nutzt das Megento-Shop-System. In dieses Shop-System kann man sogenannte Plugins bzw. Erweiterungen einbinden. Allerdings gibt es für diese Erweiterungen auch entsprechende Dokumentationen.

Abgesehen von der ersten Fehlermeldung hast du das Plugin (nach den hier vorliegenden Scriptcodes) an der falschen Stellen eingebunden. Abgesehen davon kommen Klassen in einen speziellen Ordner für Erweiterungen des Magento-System.

Also - gehe am besten zurck auf Start (ähm..... quatsch) auf die Seite, von welcher du das Plugin/Erweiterung hast und lese dir noch mal genau durch, was da wie zu tun ist, um die Erweiterung auch korrekt in das Magento-System einzubinden.

Ansonsten zeige den Link zum Plugin und zeige die Stellen, in welchen du rumgewurschtelt hast.

Auf diesem Wege wie bisher ist hier kein Lösungsansatz möglich, da nicht bekannt ist, in welchen Files du was eingebunden/geändert hast.