Hi eaton,

I've made a (faithful) D6 port of this module as I needed it for a project at work and didn't really like or need the alternatives.

It is almost exactly the same, but replaces unavailable functions with their D6 equivalents:

drupal_static -> static keyword
DBTNG queries -> standard queries with db_query
hook_node_view -> nodeapi
hook_fields_extra_fields -> hook_content_extra_fields

and some theming-related changes in nodeapi because, unlike D7, D6 doesn't delay rendering of content.

If you are happy to include this module please let me know how I can share it (commit access to a d6 branch? or whatever suits you best).

You can see the code here on github:

https://github.com/nicl/Flippy-D6

It passes Coder review.

ps. this module backport was sponsored by my employer Actency.

pps. unfortunately, the D6 version has a cck dependency (to facilitate an easy/direct port).

CommentFileSizeAuthor
#5 D6_port-1468566-5.patch21.15 KBlolandese
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Katharine_Gates’s picture

Hello I am using this D6 port and it works beautifully.
Thank you so much!

quotientix’s picture

Works like a charm (with Views 3, "Custom Pagers" doesn't). Took my a while to find this release though. Please add it as a D6 release. Thanks nicl for your hard work!

lolandese’s picture

Status: Needs review » Reviewed & tested by the community

Works as expected. Don't forget to add the extension .txt to the README file, but that's the only minor I found.

OFF TOPIC:
Of course you wanted to follow the D7 version as faithful as possible, but personally I wouldn't include '‹' and '›' symbols for the navigation links in the module code. Putting it in the flippy.css file instead, it can be overridden with custom CSS, for example:

.flippy .previous a:before {
  content: '◄ ';
}

.flippy .next a:after {
  content: ' ►';
}

That applies also on the D7 version.

Thanks.

lolandese’s picture

Title: I have made a D6 port of this module. Would you be happy to add it to the project ? » Flippy: D6 port

Changed the title so it's better recognizable in the "Track posts" block on the Dashboard.

lolandese’s picture

FileSize
21.15 KB

Rolled a patch against 7-dev combining all of the above.

lolandese’s picture

Issue summary: View changes

added pps. on new dependency.

Anybody’s picture

Issue summary: View changes
Status: Reviewed & tested by the community » Closed (outdated)