The voting works on plain smackdown nodes, but when I try to theme the nodes by calling the fields,

print $node->field_name[0]['view'];

The voting no longer works when I click on the content title/image. Is there some way kind of code that I need to add to tell it to vote?

Or how can I theme these nodes so that they look the way I want, but still allow voting?

CommentFileSizeAuthor
#7 node-battle.tpl_.php_.zip265 bytesjomg

Comments

sirkitree’s picture

sirkitree commented
Status: Active » Closed (fixed)

If you take a look at the javascript (Drupal.behaviors.smackdown) you'll see it is targeting specific classes. You must make sure that these are present for the voting to work.

jomg’s picture

jomg commented

Ah thank you!

jomg’s picture

jomg commented

Sorry I'm really new to this. I still can't seem to get it to work...
The link doesn't get processed as a vote, and only goes to the node.

I tried adding class="smackdown-processed" and a lot of the higher class tags as well.

It works simply with print $content ... but I have no idea what it's finding. I have tried using firebug and dev module.

I can't seem to spot where there are any other classes in the javascript:

/**
* Attach the smackdown behavior to nodereferences on the page.
*
* @param context
* The jQuery object to apply the behaviors to.
*/
Drupal.behaviors.smackdown = function(context) {
if (typeof Drupal.settings.smackdown !== 'undefined') {
var field_1 = '.field-' + Drupal.settings.smackdown.field1.name.replace(/_/g, '-');
var field_2 = '.field-' + Drupal.settings.smackdown.field2.name.replace(/_/g, '-');
if (Drupal.settings.smackdownPerm == 1) {
Drupal.smackdown.attachVote(context, field_1 + ' .field-item a');
$(field_1 + ' .field-item a').attr('rel', Drupal.settings.smackdown.field1.nid).addClass('smackdown-processed');
Drupal.smackdown.attachVote(context, field_2 + ' .field-item a');
$(field_2 + ' .field-item a').attr('rel', Drupal.settings.smackdown.field2.nid).addClass('smackdown-processed');
}
else {
Drupal.smackdown.attachNotice(context, field_1 + ' .field-item a');
Drupal.smackdown.attachNotice(context, field_2 + ' .field-item a');
}
}
};

sirkitree’s picture

sirkitree commented
Status: Closed (fixed) » Active

You first have to know what the variable here is. What is your field name? Smackdown stores the field name in Drupal.settings.smackdown.field1.name and Drupal.settings.smackdown.field2.name - the then var field_1 and var field_2 become field-[your-field-name]. So if you field ( in cck ) is called field_smackdown_1, Drupal.settings.smackdown.field1.name should have a value of smackdown_1 and then var field_1 will hold the value '.field-smackdown-1'.

Smackdown then attaches the voting function to $('.field-smackdown-1 .field-item a'). So you must make sure this is how your DOM is setup in your final theming. It then sets the rel property of this DOM element to the nid, which should be recorded in Drupal.settings.smackdown.field1.nid, and adds the class '.smackdown-processed'.

Please let me know if this helps.

jomg’s picture

jomg commented

Thanks for your response. I think I can understand how the javascript function works in that sense, but I don't now how it call the function.

I'll try to explain better what I'm doing.

My field name is field_war1 (in cck).

I created a new template for node-smackdown.tpl.php (which is being used properly by the content type), inside which I am trying to call specific pieces of the node referenced in the smackdown node so I can arrange/theme them:

$war1 = $field_war1[0]['nid'];

print $war1->title

Then I turn the title into a link so it can be used to vote. But then there is no part that tells smackdown.js to do anything I guess, which is why it doesn't work (?). I'm not sure what to add. I guess my DOM is net setup properl, but I'm not totally sure I understand what that's about.

Or even easier, I want to show both nodes as they are but have them side-by-side rather than vertically stacked as they show in default with:

print $content

Ideally I want to create a smackdown node like this.

| ------------ | | --------------|
| NODE1 |vs| NODE2 |
|-------------| |--------------|
[Vote1] [Vote2]

Thanks in advance for any help.

sirkitree’s picture

sirkitree commented

Maybe if you attach your actual .tpl file I can help a bit more.

jomg’s picture

jomg commented
StatusFileSize
new node-battle.tpl_.php_.zip265 bytes

Thanks. I have attached the tpl. I really appreciate the help.

sirkitree’s picture

sirkitree commented

You are giving the anchor tag the 'smackdown-processed' class. This is incorrect. 'smackdown-processed' should _only_ be given by smackdown's javascript. It's simply a way for the javascript to know that it has already attached itself to an element.

Beyond that, I can't really tell what you're trying to accomplish with this .tpl file. I assume that your 'smackdown' node type is called 'battle' and you are then trying to theme this node type.

It would probably be much easier for you to utilize the template_preprocess_node() in your template file that to try to create a tpl file for this node type.

You say that the $content variable is working just fine, so I suggest that you mimic the HTML structure that it is producing, changing as little as possible. I'm not sure I can really help you much more than that.

jomg’s picture

jomg commented

Ok. Thanks for the help. I'll see what I can do.

sirkitree’s picture

sirkitree commented
Status: Active » Closed (fixed)