This patch includes support for controlling not only whether you see amazon related links attached to full nodes or teasers, but also provides a weight option for controlling where it is displayed. This is controlled on a per-node basis, and it's in with the other settings in the content type admin.

This patch also include rudimentary CSS support. I plan to continue working on the CSS feature, which is mandatory if you want to be able to hide portions of the amazon information on a per-node basis (you can use it with node style, for example, or just handle it in CSS.) This portion of the patch is very much a work in progress, but it's necessary for the way I use the module and I don't have time now to separate out those edits. Sorry :(

CommentFileSizeAuthor
amazontools.hyperlogos.1.patch.gz2.2 KBhyperlogos

Comments

hyperlogos’s picture

Also note that you can go in at about line 358 (post-patch) and comment out

    if (strpos($node_type, 'amazon') === FALSE) {

AND THE MATCHING CLOSE BRACE (}) AS WELL, DON'T FORGET! You can find it in the vi editor with the % key (when the cursor is on the open brace.)

In order to allow attaching additional, related links to nodes of type 'amazon'. This is not guaranteed to work correctly (yet?) but it can be inoffensive so far here with proper CSS and not showing the related links in teasers. I do think there may be a problem with the overuse of the variable "asin" in forms. Sorry.

MGParisi’s picture

Do you have a screen shot? mgparisi-=at=-rochester.rr.com

I may want to include this into my module addon!

MGParisi’s picture

CSS support has been added (not with this patch) to dev, and I will look at the weight fix when I get a moment to sit down! I have changed the weight, but it would be nice to have an admin option.

MGParisi’s picture

Status: Needs review » Fixed

Fixed and tested in head, needs to publish!

Status: Fixed » Closed (fixed)

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