table of contents diff is broken

[Edward Betts]

See http://openlibrary.org/b/OL18195253M/Using-mentors-to-change-problem-behaviour-in-primary-school-children?m=diff&b=2

Revision 1 and 2 are blank.

[Edward Betts (edwardbetts)]

We should fix this after the October release.

[George (george-archive)]

Still broken. Is it easier now to fix given the simpler TOC implementation?

[George (george-archive)]

Here's another example:

http://upstream.openlibrary.org/books/OL18929164M/Workshop_manual_for_B.M.C._2.2-litre_four-cylinder_high_speed_diesel_engine.?m=diff&b=2

Given the increased prominence of history/recent edits in the redesign, it would be good to fix this bug.

[George (george-archive)]

How is it possible to have a blank diff? E.g. http://upstream.openlibrary.org/books/OL17889439M/Introdu%C3%A7%C3%A3o_%C3%A0_hist%C3%B3ria_da_antropologia_ind%C3%ADgena_no_Brasil_s%C3%A9culo_16.?m=diff&b=4

[Anand Chitipothu (anandology)]

Fixed.

http://upstream.openlibrary.org/books/OL9706244M/Language_of_the_Earth?b=12&a=11&_compare=Compare&m=diff

[George (george-archive)]

Excellent!

[George (george-archive)]

Why is everything on separate lines? Readability?

[George (george-archive)]

And, it looks like your insertion of a blank TOC entry is counted as an update...

http://upstream.openlibrary.org/books/OL3819344M/Simulacres_et_simulation?m=diff&b=4

Would be good to either modify your addition of a blank "| |" or remove it from the diff.

[Anand Chitipothu (anandology)]

I think, the blank " | | " are coming from production. Added code to supress such entries from view, edit and diff.