Download & Extend

Display of & (and possibly other special chars) isn't handled correctly

Project:Amazon Module
Version:6.x-1.0-rc2
Component:User interface
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

To reproduce this issue, create an amazon item using ISBN 0945747306

Ampersands (&) display as & on the front end.

Comments

#1

I don't know if this is a solution or not, but I could get away from the issue by editing amazon-item.tpl.php like the following.

<div class="<?php print $classes; ?>">
<?php print $smallimage; ?>
<div><strong><a href="<?php print $detailpageurl; ?>"><?php print $title; ?></a></strong></div>
</div>

#2

Status:active» fixed

Fixed in http://drupal.org/cvs?commit=270996 and it will be in the next release. Should be in tomorrow's dev release, and I'd much appreciate you testing it.

Thanks,
-Randy

#3

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

#4

Version:6.x-1.0-beta5» 6.x-1.0-rc2
Status:closed (fixed)» active

This issue appears to still exist in 6.x-1.0-rc2

In the meantime will try the fix mentioned above.

#5

Actually the reason why it still doesn't work is the new link code needs to be added to all the other built-in amazon templates, i.e. amazon-item-book-details.tpl.php and the others. but in the meantime, it's easy enough to override those templates as needed.

#6

Status:active» fixed

Ah, it got fixed in the small view but not in the full view. The same change needs to be made in amazon-item-details.tpl.php.

Fixing the template should do the job for you.

Fix committed: http://drupal.org/cvs?commit=311648 and will be in the next dev.

AttachmentSize
amazon.amp_revisited_350320_05.patch 1.34 KB

#7

Missed the media module templates: http://drupal.org/cvs?commit=311696. Thanks for pointing that out bradezone.

AttachmentSize
amazon.amp_revisited_350320_06.patch 3.38 KB

#8

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here