Hi,
Is it possible to have the code upaded to 4.7? I tried uncommenting the two lines in the module which are for 4.7 and removing the other two preceeding lines but there are still errors.
Many thanks
Rachel
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | nodevote_4.7_forms_noWOOT.diff | 5.14 KB | telex4 |
| #4 | node_vote_fapi.diff | 6.21 KB | Bèr Kessels |
| #3 | nodevote_4.7_forms.patch | 4.97 KB | telex4 |
Comments
Comment #1
kbahey commentedThis is not enough, there are other changes for the new form API.
All my modules need to be updated at some point, and I will wait for a stable 4.7 release candidate before I start.
If you want it sooner for a business site, please contact me for a consultation assignment.
Thanks
Comment #2
cre8d commentedThanks. I have started making some adaptions using the form updater module and have got rid of the errors but it's not listing the node types in the settings menu option. I'll see if I can get that updated :)
Unfortunately it's just a test site for me not a commercial project with a budget.
Comment #3
telex4 commentedI've been working on this... I've so far got the forms showing up OK, and the settings form works fine. The only problem is that voting doesn't seem to work - I submit the form and just come back to the same page, no vote made, and the score defaulting to what I submitted.
I've probably missed something obvious because I'm tired at the mo, so it shouldn't be too hard to fix this up into a fully working patch.
Comment #4
Bèr Kessels commentedthis patch has the voting fixed too
Comment #5
telex4 commentedHere's the same patch but without Ber's addition of 'WOOT' to the rating button :-) It all now works fine on http://www.remixcommons.org
Comment #6
naudefj commentedHi Telex4,
Your patch seems to work well with Drupal 4.7, with the following small exception:
After casting a vote, the page is redirected to /node/view/nid instead of just /node/nid. Is this intentional?
If not, we can just change this line:
drupal_goto("node/view/" . $nid);
to
drupal_goto("node/" . $nid);
and set the status to ready to be committed.
Best regards.
Frank
Comment #7
kbahey commentedA new 4.7 version has been committed, and it contains some new features as well.
Comment #8
(not verified) commented