Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Index creation with taking account language of each node
Comment | File | Size | Author |
---|---|---|---|
#10 | prev_next-language-1979744-10.patch | 1.72 KB | ckng |
Comments
Comment #1
Ilyna CreditAttribution: Ilyna commentedComment #2
Ilyna CreditAttribution: Ilyna commentedError in patch file
Comment #3
krystlc CreditAttribution: krystlc commented#2 patch still works but i had to patch it manually.
Comment #4
intergalactic overlords CreditAttribution: intergalactic overlords commentedRe-uploaded the patch. The path inside the patch was from root. Can now be applied automatically from inside the module folder.
Comment #5
intergalactic overlords CreditAttribution: intergalactic overlords commentedRe-uploaded the patch. The path inside the patch was from root. Can now be applied automatically from inside the module folder.
Comment #6
jaxxed CreditAttribution: jaxxed commentedThat last patch looks like a re-package. I properly packaged the above changes from the latest 7.x-1.x git branch as per patch submission requirements. This fixes the git --diff line at the top of the patch.
I took only 1 or two liberties in cleaning the code. The language helper function that returns the SQL condition for the language was pretty messy (extra semi-colon, double return, no validation on node language.) The modules could still use more SQL validation and should like use more db_select() instead of db_query().
I am not sure what impact the patch has/should have on the hook_cron() task.
Instructions for providing a patch to a Drupal project: https://drupal.org/node/707484
Comment #7
Sebb'a CreditAttribution: Sebb'a commented#6 patch has a bug in the first fixed line
+ $cond = _prev_next_node_types_sql($node_type) . _prev_next_node_lang;
you have to put ($nid) behind _prex_next_node_lang:
+ $cond = _prev_next_node_types_sql($node_type) . _prev_next_node_lang($nid);
Comment #8
Jorge Navarro CreditAttribution: Jorge Navarro commentedTested with #7 fix, it works!
Comment #9
marty.true CreditAttribution: marty.true commentedWhere does this issue/patch stand now that the module is in 7.x-2.x-dev and 7.x-1.x-dev is no longer supported?
Comment #10
ckngA new patch for 7.x-2.x.