Two first chapters having the same id

[grandt]

Two first chapters having the same id

[Grandt]

How are you adding chapters?

Because, at a glance I don't see a way addChapter can do that, meaning you are obviously doing something I didn't anticipate (or overlooked). :P
I'm not saying you are wrong, as you obviously have the problem.

[tcitworld]

The cover is added by this line :

$book->addChapter('Notices', 'Cover2.html', $content_start.$this->getExportInformation('PHPePub').$bookEnd);

and the chapters are added through a loop :

foreach ($this->entries as $entry) {
    foreach ($this->tags as $tag) {
        $book->setSubject($tag['value']);
     }
     $chapter = $content_start.$entry->getContent().$bookEnd;
     $book->addChapter($entry->getTitle(), htmlspecialchars($entry->getTitle()).'.html', $chapter, true, EPub::EXTERNAL_REF_ADD);
}

[Grandt]

Can I get you to send me an example of a defective book to [email protected]?

[Grandt]

Boy, does that bug make me look "stoopid"

In addChapter I increase chapterCount. That function calls a few others of course, in the course of which the code checks if the book has been initialized yet, if it haven't is does so. Initialize Sets up the zip file and adding the preliminary metadata depending on the setup of the book. For instance the ePub version.
And that initialization sets chapterCount to 0, AFTER I just increased it to 1...

I need more coffee.

[Grandt]

P.S.: For a minimum impact of your current code, at least until I'm ready to release 4.0.6, you can just uncomment the $this->chapterCount = 0; line in the initialize function.

[tcitworld]

No worries. Thanks for your time.

For a minimum impact of your current code, at least until I'm ready to release 4.0.6

If we can get it through composer before end of March, it's perfectly fine. :)

[Grandt]

I need to test 4.0.6 a little more, it's on github master branch now,
but I expect to release it before the end of February.

On 21-02-2016 15:44, Thomas Citharel wrote:

No worries. Thanks for your time.

For a minimum impact of your current code, at least until I'm
ready to release 4.0.6

If we can get it through composer before end of March, it's perfectly
fine. :)

—
Reply to this email directly or view it on GitHub
#36 (comment).

[tcitworld]

Perfect !

[nicosomb]

Hello @Grandt!

Do you plan to release a new version in next days?