It would be good if the pages and copyright fields in bookreview were text rather than integer. Some (older) books don't have a copyright date, and it is common to put "n/d" for these. Many book reviews/descriptions like to include the binding with the pages, so something like "253, hardcover" would be common. I've modified the fields in my database and it doesn't seem to cause any major problems (although it's hard to tell since i'm experiencing the same problem as http://drupal.org/node/11447).

CommentFileSizeAuthor
#3 bookreview.pgsql.patch460 bytespaulgear
#2 bookreview.mysql.patch510 bytespaulgear
#1 bookreview.module.patch23.79 KBpaulgear

Comments

paulgear’s picture

paulgear commented
StatusFileSize
new bookreview.module.patch23.79 KB

Here are some updated patches to provide this feature. Wouldn't want to make a guess as to how to convert peoples' databases, though. :-)

paulgear’s picture

paulgear commented
StatusFileSize
new bookreview.mysql.patch510 bytes
paulgear’s picture

paulgear commented
StatusFileSize
new bookreview.pgsql.patch460 bytes
paulgear’s picture

paulgear commented

And for the record the problems in http://drupal.org/node/11447 appear to be unrelated.

jeremy’s picture

jeremy commented

I'll look into applying this or a similar change shortly. This module is in need of some maintenance...

paulgear’s picture

paulgear commented

Anything particular that needs doing? I'm fairly happy with the results it produces and will probably use it quite a bit, so if it needs some tweaking here & there, i'm happy to look at it.

jeremy’s picture

jeremy commented
Assigned: Unassigned » jeremy

The pages and copyright fields have been converted from int to text. (See changelog for commands to execute to convert your database -- no data will be lost)

Anonymous’s picture

(not verified) commented