Extra

class=\"

in table output produces incorrect HTML output. Please see attached patch.

retrieving revision 1.63.2.7
diff -u -w -b -F^function -r1.63.2.7 amazon.module
--- amazon.module	15 Feb 2007 02:45:50 -0000	1.63.2.7
+++ amazon.module	26 Nov 2007 00:23:04 -0000
@@ -1467,7 +1467,7 @@ function theme_amazon_node($node) {
 
 function theme_amazon_related_link($node) {
   $output = '';
-  $output .= "<table class=\" class=\"amazontools_related\"><tr>";
+  $output .= "<table class=\"amazontools_related\"><tr>";
   $output .= "<td><a href=\"$node->detailpageurl\" target=\"_blank\">";
 	$module_dir = drupal_get_path('module', 'amazon');
   if ($node->smallimageurl) {
CommentFileSizeAuthor
amazontools-fix.patch557 bytesmcurry

Comments

mcurry’s picture

Project: Amazon Filter » Amazon associate tools
Version: 5.x-1.x-dev » 5.x-1.5

Oops, wrong module. Fixing.

goddess’s picture

Thank you; I came looking for a solution to this issue. Those applying the patch may need to be reminded that you must edit the amazon node and save it again (separate from the node it's attached to) in order for the changes to take effect. At least, I would have benefited from such a reminder. Ha!

Thanks!

wayland76’s picture

Status: Active » Needs review
MGParisi’s picture

still not applied, this module is dead? Is the author going to apply the patches?

MGParisi’s picture

MGParisi’s picture

Status: Needs review » Patch (to be ported)
MGParisi’s picture

Status: Patch (to be ported) » Fixed

Patched should be in Dev

Anonymous’s picture

Status: Fixed » Closed (fixed)

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