Call for the Drupal community to fix prev/next in forum and in Drupal
Dear friends, I call for the Drupal community to fix prev/next scalability in Drupal forum. An open letter is available for Dries BuyTaert. Issue: http://drupal.org/node/559424 Thank you for your attention, Jean-Michel Pouré
Okay we get it you think it's an issue, so patch write a patch already! 2009/8/26 Jean-Michel Pouré <jm@poure.com>
Dear friends,
I call for the Drupal community to fix prev/next scalability in Drupal forum. An open letter is available for Dries BuyTaert.
Issue: http://drupal.org/node/559424
Thank you for your attention, Jean-Michel Pouré
Le mercredi 26 août 2009 à 16:29 -0400, Jim Taylor a écrit :
Okay we get it you think it's an issue, so patch write a patch already!
Thank you very much for your comprehension. Previous/Next link is an important issue because MySQL and PostgreSQL users usually do not have access to server-side initialization scripts and therefore may not log slow queries. Even using Devel module, it is hard to find as explained here: http://drupal.org/node/559320 The only way to find this bug was to read Drupal code and test queries server-side. This can only be done by developers. So many people may be suffering from this bug and YES it is important to fix it. Thank you for your attention. Kind regards, Jean-Michel
Jean-Michel Pouré wrote:
Dear friends,
I call for the Drupal community to fix prev/next scalability in Drupal forum. An open letter is available for Dries BuyTaert.
I tried to find a volunteer to fix this 2 years ago. Since you're not volunteering, you're just rehashing. Perhaps you should just roll the patch rather than telling other people what to do. It's also already been removed from core, so there's nothing to fix anymore: http://drupal.org/node/556136
Le mercredi 26 août 2009 à 13:46 -0700, Earl Miles a écrit :
It's also already been removed from core, so there's nothing to fix anymore: http://drupal.org/node/556136
I answered this issue. A new module called http://drupal.org/project/prev_next is available. Why not integrate it in core? Kind regards, Jean-Michel
Also, 2009/8/26 Jean-Michel Pouré <jm@poure.com>
Le mercredi 26 août 2009 à 13:46 -0700, Earl Miles a écrit :
It's also already been removed from core, so there's nothing to fix anymore: http://drupal.org/node/556136
I answered this issue.
A new module called http://drupal.org/project/prev_next is available.
I am the author of prev_next API. While it does a good job for large sites, it adds yet another table of redundant data to the schema. This type of data duplication is not necessary for most sites that do not have the size that mandates such a solution. So, for the vast majority of sites, this kind of duplication is not needed.
Why not integrate it in core?
As usual, we try to keep core small, and not add every single feature to it. We keep it extensible and people can come up with solutions like this module provides. Not every site will need such an API, and therefore it is not core justified. Another options it that you can also just remove the Next/Previous links by a simple change to your template.php and the problem goes away. function phptemplate_forum_topic_navigation($node) { return ''; } That is it. -- Khalid M. Baheyeldin 2bits.com, Inc. http://2bits.com Drupal optimization, development, customization and consulting. Simplicity is prerequisite for reliability. -- Edsger W.Dijkstra Simplicity is the ultimate sophistication. -- Leonardo da Vinci
----- "Jean-Michel Pouré" <jm@poure.com> wrote:
I call for the Drupal community to fix prev/next scalability in Drupal forum. An open letter is available for Dries BuyTaert.
Multiple people have censured you over the last few days -- with no apparent effect. You bring an unfortunate combination of insolence and incompetence to our issue queues, documentations pages, and, apparently, our mailing lists. Competence and cooperation are virtues we hope to foster in all community members over time, but your choices are no path to that. Be productive and civil -- or leave.
participants (5)
-
David Timothy Strauss -
Earl Miles -
Jean-Michel Pouré -
Jim Taylor -
Khalid Baheyeldin