patch for contrib/isn
hi all,
currently i am working on a big project for a german bookseller and
publisher. one of the requirements was correct hyphenation of ISBN-13
for about 14.400.000 books in postgresql database. so added support
for hyphenating isbn with the new 979-prefix and additionally added all
missing ranges since year 2006 for isbn with 978-prefix.
patch against HEAD is attached and validated against a lot of previously
wrong and correct hyphenated isbn.
regards, jan
Attachments:
contrib_isn-1.patchapplication/octet-stream; name=contrib_isn-1.patchDownload+385-227
On Wed, 2010-08-04 at 19:32 +0200, Jan Otto wrote:
hi all,
currently i am working on a big project for a german bookseller and
publisher. one of the requirements was correct hyphenation of ISBN-13
for about 14.400.000 books in postgresql database. so added support
for hyphenating isbn with the new 979-prefix and additionally added all
missing ranges since year 2006 for isbn with 978-prefix.patch against HEAD is attached and validated against a lot of previously
wrong and correct hyphenated isbn.regards, jan
Great! Thanks. We will get it on the review list.
Joshua D. Drake
--
PostgreSQL.org Major Contributor
Command Prompt, Inc: http://www.commandprompt.com/ - 509.416.6579
Consulting, Training, Support, Custom Development, Engineering
http://twitter.com/cmdpromptinc | http://identi.ca/commandprompt
"Joshua D. Drake" <jd@commandprompt.com> wrote:
On Wed, 2010-08-04 at 19:32 +0200, Jan Otto wrote:
patch against HEAD is attached and validated against a lot of
previously wrong and correct hyphenated isbn.
Great! Thanks. We will get it on the review list.
I added it as "isbn update" to the 2010-09 CommitFest page.
-Kevin
On Wed, Aug 4, 2010 at 2:58 PM, Kevin Grittner
<Kevin.Grittner@wicourts.gov> wrote:
"Joshua D. Drake" <jd@commandprompt.com> wrote:
On Wed, 2010-08-04 at 19:32 +0200, Jan Otto wrote:
patch against HEAD is attached and validated against a lot of
previously wrong and correct hyphenated isbn.Great! Thanks. We will get it on the review list.
I added it as "isbn update" to the 2010-09 CommitFest page.
Please put "contrib/isn" in the name somewhere so that there is some
overlap between the subject line and the CF entry. I can't tell you
how many times I've gone "oh, crud, what was the subject line for the
latest version of the \ef / \sf patch?" in the last two weeks.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company
Robert Haas <robertmhaas@gmail.com> wrote:
Please put "contrib/isn" in the name somewhere so that there is
some overlap between the subject line and the CF entry.
It is now "contrib/isn isbn update".
-Kevin
On ons, 2010-08-04 at 19:32 +0200, Jan Otto wrote:
patch against HEAD is attached and validated against a lot of
previously wrong and correct hyphenated isbn.
I think this module could use a regression test.
On Aug 6, 2010, at 11:08 PM, Peter Eisentraut wrote:
On ons, 2010-08-04 at 19:32 +0200, Jan Otto wrote:
patch against HEAD is attached and validated against a lot of
previously wrong and correct hyphenated isbn.I think this module could use a regression test.
i'll take a look at this within next days.
regards, jan