jReviews :: Customer support forum

I can add and edit an entry and everything works fine as long as there are no images attached to the listing. If the listing has an image when it adds ok, but when I try to edit it I get a blank page and this error is written to my log file:

PHP Parse error:  syntax error, unexpected T_ENDIF in /htdocs/test/components/com_jreviews/tmpl/default/image_manager.ctp on line 30, referer: http://test.castawaysclub.com/content/view/77/

Already did that. Just thought I'd let you know about the error.

Are you sure you are editing the correct theme template. I mean default vs. store?

Even with this missing semicolon, I can edit images on my dev site. Please submit a ticket with access to see the problem and ftp to troubleshoot it.

Im running an offline "test" version of my joomla site on my LAN.  I upgraded the offline to jReviews 1.2.10 over the weekend and am testing it.

I have the same problem with this T_ENDIF line30.

Is there a fix ?

We had a problem with the images on the my listings or edit my listings page. When they were displayed they broke the css so menus appeared off the page.

The url of the edit my listings page by the way is (where yourdomain.com is the name of your site):
http://www.yourdomain.com/index.php?option=com_jreviews&task=edititem&id=319

Here is how to fix it.

Using ftp open image_manager.ctp in a text editor.
The path to this file should be something like:
/components/com_jreviews/tmpl/store/image_manager.ctp

If you are using the default instead of store it will be:
/components/com_jreviews/tmpl/default/image_manager.ctp


The file will look like this:
<?php
/**
 * jReviews - Reviews Extension
 * Copyright (C) 2006 Alejandro Schmeichler
 * This is not free software, do not distribute it.
 * For licencing information visit http://www.reviewsforjoomla.com
 * or contact sales@reviewsforjoomla.com
 *
 * Template used inside the content editing form to allow setting intro image and deletion of images
 **/

// no direct access
defined( '_VALID_MOS' ) or die( 'Restricted access' );

?>
<div id="content_images" style="margin-top:5px; margin-bottom:5px;">   
<span class="label"><?php echo _JR_IMAGE_MANAGER_LABEL?></span>
<br />

<?php $i = 0; foreach($thumbs->thumbs AS $tn):?>

   <div id="image<?php echo $i?>" style="margin:1px;padding-left:3px;padding-top:2px;border:1px solid #000;float:left;">
   
      <img style="border:1px solid #BFBFBF; width:60px;height:60px;" src="<?php echo $thumbs->site_tn.$tn['image_thumb']?>"  />
      <br />
      
      <? if(count($thumbs->thumbs)>1):?>
      <input style="cursor: pointer;" id="intro<?php echo $i?>" name="intro[]" type="radio" value="<?php echo $i?>" <?php echo $i == 0 ? 'checked="checked"' : ''?> onclick="xajax.$('image_edit').style.display = '';xajax_xajaxDispatch('listings','imageSetMain',<?php echo $content_id?>,'<?php echo $tn['image_target']?>');" />
      &nbsp;&nbsp;
      <?php endif; ?>
      
      <img style="cursor: pointer;" src="<?php echo _TMPL_SITE_JREVIEWS?>/images/trash.png"
         onclick="if(confirm('<?php echo _JR_IMAGE_CONFIRM_DELETE?>')) {xajax.$('image_edit').style.display = '';xajax_xajaxDispatch('listings','imageDelete',<?echo $content_id?>,<?php echo $i?>,'<?php echo $tn['image_target']?>');};" />
   
   </div>
<?php $i++; endforeach;?>

<div style="clear:both;"></div>
<img id="image_edit" style="display: none;" src="<?php echo _TMPL_SITE_JREVIEWS?>/images/loading.gif" />



Now change the file and include an additional </div> after the line:
<?php $i++; endforeach;?>


So it reads:
<?php $i++; endforeach;?>
</div>

<div style="clear:both;"></div>
<img id="image_edit" style="display: none;" src="<?php echo _TMPL_SITE_JREVIEWS?>/images/loading.gif" />



That should sort things out.

i hope this helps and was clear enough for people to understand.

HarrisW