Closed (fixed)
Project:
Amazon associate tools
Version:
5.x-1.5
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Nov 2007 at 00:41 UTC
Updated:
1 Oct 2008 at 20:52 UTC
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) {
| Comment | File | Size | Author |
|---|---|---|---|
| amazontools-fix.patch | 557 bytes | mcurry |
Comments
Comment #1
mcurry commentedOops, wrong module. Fixing.
Comment #2
goddess commentedThank 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!
Comment #3
wayland76 commentedComment #4
MGParisi commentedstill not applied, this module is dead? Is the author going to apply the patches?
Comment #5
MGParisi commentedThe patch is applied to http://drupal.org/node/289780#comment-945539
Comment #6
MGParisi commentedComment #7
MGParisi commentedPatched should be in Dev
Comment #8
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.