Hi!

my php abilities have progressed form the use of "patch -p1 -i ..." to CTRL+C, CTRL+V :)

This patch against cvs HEAD adds an option to enumerate the biblio entries using <ol> and <li>tags.

I know there is a bad interaction with the pager (entries numbering always starts at '1') but it can be handy anyways. An example is shown here :

http://www.creatis.insa-lyon.fr/site/biblio/author/4

Please comment!

Sebastien

Comments

rjerome’s picture

I'm not so sure about this one...

My first question is: what is the benefit to numbering them? Given it's just an ordered list, the number will change depending on the filters in place and additions or deletion of entries.

Also as you point out, for each "pager" page of entries, the count resets to 1 which will likely be the biggest complaint from the users. This is not insurmountable since the pager query updates a few globals which could be used to create contiguous numbering, but I'm still not convinced of the value.

Maybe some other users will chime in with comments.

Ron.

sebos69’s picture

My first question is: what is the benefit to numbering them?

It can be handy to just count the number of publications. We are moving from using bibtex2html which delivers this kind of output:
http://www.creatis.insa-lyon.fr/publications/Author/VALETTE-S.html

To limit the pager issue, I increased the size of the pager (to 250 lines, solves 90%+ of cases).

rjerome’s picture

What about putting the section totals in the section headers?

Book (3)
...

Conference Paper (10)

...

Journal Article (200)

sebos69’s picture

What about putting the section totals in the section headers?

It may be enough (I will check about it). But as I see the code layout, adding this feature is a bigger work than applying the proposed patch, but maybe I'm wrong...

rjerome’s picture

I haven't tried it yet, but I believe the totals could be obtained with a separate SQL query grouping by the type and counting the totals in each group.

sebos69’s picture

yep, so it's out of my abilities right now...

sebos69’s picture

StatusFileSize
new378 bytes

Hi ron!

here is just an updated patch against the current version (1.6) of the module. Maybe this could come handy for other users too.

The good news is that with your biblio.pages.inc rewrite this patch is much simpler than before (4 lines to change).

Greetings

liam morland’s picture

Issue summary: View changes
Status: Needs review » Closed (outdated)

This version is no longer maintained. If this issue is still relevant to the Drupal 7 version, please re-open and provide details.