I'm relatively new to php and any advice is appreciated.
I retrieve some data from an external source and process them in a foreach loop to display the items. Now I'm not sure if the way I choose is the right way or if there is a better way for a situation like this.
And here the code I use:
define ("WSDL_SERVER",
"XXX");
$oSmartFeed= new SoapClient(WSDL_SERVER);
$oSessionHash= $oSmartFeed->login('XXX', 'XXX');
if(!$oSessionHash->HasError){
$sSessionHash= $oSessionHash->Records['sessionHash'];
$aConfig= array('limit'=>5, 'offset'=>2, 'feeds'=>[XXX]);
$aResult= $oSmartFeed-> getProductData ($sSessionHash, $aConfig);
echo '<ul>';
foreach ($aResult->Records AS $aRecord) {
$produktName = $aRecord['productname'];
$imageUrl = $aRecord['imagebigurl'];
$price = $aRecord['currentprice'] . $aRecord['currency'];
$affLink = $aRecord['deeplinkurl'];
$productDescription = $aRecord['productdescriptionlong'];
$versandKosten = $aRecord['shipping'] . ' €';
?>
<li>
<a href="<?php echo $affLink;?>">
<img src="<?php echo $imageUrl; ?>" alt="<?php echo $produktName; ?>">
<h3>
<?php echo $produktName; ?>
</h3>
<p>
<?php echo $productDescription;?>
</p>
<aside>
<span>
Price: <?php echo $price; ?>
</span>
<br>
<span>
Shipping: <?php echo $versandKosten; ?>
</span>
</aside>
</a>
</li>
<?php
}
echo '</ul>';
$oSmartFeed->logout($sSessionHash);
if
statement is missing its closing brace.) \$\endgroup\$