search

julien-c / epub

node.js epub reader
https://www.npmjs.com/package/epub
Other
333 stars 307 forks source link

Removed ':' from metadata tags #18

Closed johanderohan closed 9 years ago

johanderohan commented 9 years ago

Then we can use metadata from calibre library like ‘calibre:series’, ‘calibre:series_index’ or ‘calibre:rating’

julien-c commented 9 years ago

I think the intended behavior is for the lib to output exactly what's in the metadata files. So I'm rather reluctant to accepting your PR, unless you have stronger reasons why. Thanks!

johanderohan commented 9 years ago

The problem is that from Calibre metadata like:

< meta content="01" name="calibre:series_index" / >

It generate an estructure like :

{ "calibre:series_index":"1" }

And is impossible to get the value easily from the index: 'epub.calibe:series_index'

But you are right, is more a Calibre transformation issue, not an epub format issue.

Thanks for your time!

julien-c commented 9 years ago

No problem! Thanks for using epub