Project

General

Profile

Bug #463

TOC incorrect when undeclared divisions exist

Added by Luke Murphey almost 12 years ago. Updated almost 12 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Luke Murphey
Target version:
0.1
Start date:
12/01/2012
Due date:
% Done:

100%

Description

The table of contents include divisions that are not included on Perseus because they are not in the refs declaration.

Associated revisions

Revision 215 (diff)
Added by Luke Murphey almost 12 years ago

Added import transform that deletes unnecessary divisions. Closes #463.

Revision 215 (diff)
Added by Luke Murphey almost 12 years ago

Added import transform that deletes unnecessary divisions. Closes #463.

Revision 207 (diff)
Added by Luke Murphey almost 12 years ago

Added import transform that deletes unnecessary divisions. Closes #463.

History

#1 Updated by Luke Murphey almost 12 years ago

Filtering undeclared divisions made the TOC look closer to what is expected. However, a couple of issues were noted:

  1. There are several sections with the title "none" (such as under Δημοδόκου)
  2. Some of the sub-sections are missing. Like those under Κλεοβούλου:

*Κλεοβούλου ** section 1 ** ἐπίγραμμα καὶ Αἴνιγμα

#2 Updated by Luke Murphey almost 12 years ago

  • % Done changed from 0 to 20

#3 Updated by Luke Murphey almost 12 years ago

The sections with "none" come from divisions that literally say "none":

<div1 type="text" id="tlg-0245"><head>*dhmodo/kou</head>

<div2 type="section"><!--testimonia-->
<head><bibl><foreign lang="en">NONE</foreign></bibl></head><p></p></div2>

<div2 type="section"><!--fragments--><head>*)elegei/wn</head>

<div3 type="elegiac" n="1">
<cit id="tlg-0245.cit.1"><quote>o(/ti me\n ou)=n kaki/a h( a)krasi/a ou)k e)/sti, fanero/n. a)lla/ ph| i)/sws: to\ me\n ga\r para\ proai/resin to\ de\ kata\ proai/resi/n e)stin. ou) mh\n a)ll' o(/moio/n ge kata\ ta\s pra/ceis w(/sper to\ *dhmodo/kou ei)s *milhsi/ous:
<cit id="tlg-0245.cit.2"><quote>
<l>&lt;*kai\ to/de *dhmodo/kou&gt;:<!--note n="p.166.n.1"/--> *milh/sioi a)cu/netoi me\n</l>
<l>ou)k ei)si/n, drw=sin d' oi(/aper w(cu/netoi:<!--note n="p.166.n.2"/--></l></quote><bibl>CURFRAG.tlg-0245.1</bibl></cit>
<p>kai\ oi( a)kratei=s a)/dikoi me\n ou)k ei)si/n, a)dikou=si de/.</p> </quote> <bibl>Arist. <title>Eth. Nic.</title> 1151 a 5</bibl></cit>
</div3>

What is odd is that the section titled "*)elegei/wn" is skipped though it is also a section. The content of the "none" section is the contents of section 3:

ὅτι μὲν οὖν κακία ἡ ἀκρασία οὐκ ἔστι, φανερόν. ἀλλά πῃ ἴσως: τὸ μὲν γὰρ παρὰ προαίρεσιν τὸ δὲ κατὰ προαίρεσίν ἐστιν. οὐ μὴν ἀλλ' ὅμοιόν γε κατὰ τὰς πράξεις ὥσπερ τὸ Δημοδόκου εἰς Μιλησίους:

#4 Updated by Luke Murphey almost 12 years ago

I think the "none" sections are only shown on the TOC because they have no chapters and are thus not considered readable units.

#5 Updated by Luke Murphey almost 12 years ago

Actually, the "NONE" units are shown in Perseus. The other readable units are as well:

  • Δημοδόκου
    • NONE
    • ἐλεγείων
    • τετραμέτρων

#6 Updated by Luke Murphey almost 12 years ago

Not showing the sections under the divisions may be a blessing. You can always view them by clicking through the sections in the paginator. The problem is that I need a way to detect divisions that have no verses in them and flag them so that they can be filtered.

#7 Updated by Luke Murphey almost 12 years ago

  • % Done changed from 20 to 70

#8 Updated by Luke Murphey almost 12 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 70 to 100

Applied in changeset r215.

Also available in: Atom PDF