noteweb: Rendering bird field notes for the Web

In YearCollection.addYear(), the call to the YearRow constructor is missing the second (txny) argument, because it was written before the constructor was fleshed out.

        yearRow  =  YearRow ( self, yyyy )

should be:

        yearRow  =  YearRow ( self, txny, yyyy )

which means of course that the YearCollection.addYear() method also needs this argument, and the call to that method in findYears() must also supply it.

In YearRow.readOneMonth(), the reference to the BirdNoteSet constructor was not properly qualified with its containing module name.

 # [
            birdNoteSet := a new BirdNoteSet instance with #
            taxonomy self.txny ] birdNoteSet = BirdNoteSet ( txny
            )

should be:

        # [ birdNoteSet  :=  a new birdnotes.BirdNoteSet instance
        #       with taxonomy self.txny ]
        birdNoteSet  =  birdnotes.BirdNoteSet ( txny )

In the same line as the previous defect, txny should be self.txny.

        # [ birdNoteSet  :=  a new birdnotes.BirdNoteSet instance
        #       with taxonomy self.txny ]
        birdNoteSet  =  birdnotes.BirdNoteSet ( self.txny )

The declaration of MONTH_SEASON_MAP was cut and pasted from MONTH_NAME_MAP, and I changed the content but forgot to change the name.

Spelling error in YearRow.successor():

    def succesor ( self, mm ):

Try:

    def successor ( self, mm ):

In YearCollection.__findNext(), this line:

        pos  =  yyyyList.index ( yyyyy )

should be:

        pos  =  yyyyList.index ( yyyy )

Method MonthCell.__pageFrame() was declared with two arguments prevURL and nextURL, but the caller was supplying only the “yyyy-mm” key of those months, and the URLs of the target pages were developed inside the method. Change the method's starting lines from:

    def __pageFrame ( self, prevURL, nextURL ):
        '''Set up an empty tccpage2.TCCPage instance.

          [ (prevURL is the URL of the previous month page or None) and
          [ (nextURL is the URL of the next month page or None) ->
              return a new TCCPage instance with navigation links
              prevURL and nextURL ]
        '''

to:

    def __pageFrame ( self, prev, next ):
        '''Set up an empty tccpage2.TCCPage instance.

          [ (prev is the 'yyyy-mm' of the previous month or None) and
          [ (next is the 'yyyy-mm' of the next month or None) ->
              return a new TCCPage instance with navigation links
              previous=(prev's page) and next=(next's page) ]
        '''

Symmetric errors in both YearRow.firstMonth() and YearRow.lastMonth(): they were supposed to return the month key 'mm', not the entire MonthCell. Here is the fix for YearRow.firstMonth():

        return self[monthKeyList[0]]

should be:

        return self[monthKeyList[0]].mm

MonthCell.__renderPage() was declared as MonthCell.renderPage(). This happened during a general renaming of methods of this class so that all the internal methods started with “__”.

Left out the self argument in MonthCell.__renderPage() and also in MonthCell.__pageTOC().

In MonthCell.__pageTOC, the intended function was correct, but two lines were missing from the code:

        #-- 1 --
        # [ parent  :=  parent with a new 'ul' child element added
        #   ul  :=  that child
        #   anchorSet  :=  a new, empty set
        #   anchorMap  :=  a new, empty dictionary ]
        ul  =  et.SubElement ( parent, 'ul' )

Here are the missing lines:

        anchorSet  =  set()
        anchorMap  =  {}

In MonthCell.__dayTOC(), these lines:

        newAnchor  =  self.birdNoteSet.date
        suffix  =  'a'
        while newAnchor in anchorSet:
            newAnchor  =  self.birdNoteSet.date+suffix
            suffix  =  chr(ord(suffix)+1)

were trying to extract the date from the wrong place. It should be:

        newAnchor  =  dayNotes.date
        suffix  =  'a'
        while newAnchor in anchorSet:
            newAnchor  =  dayNotes.date+suffix
            suffix  =  chr(ord(suffix)+1)

In MonthCell.__dayTOC(), this line:

        a  =  et.SubElement ( 'a', href=('#'+newAnchor) )

omitted the first argument, and should be:

        a  =  et.SubElement ( li, 'a', href=('#'+newAnchor) )

In MonthCell.__notablesBlock(), this line:

        div  =  et.SubElement ( parent )

should be:

        div  =  et.SubElement ( parent, 'div' )

The declaration of DAY_SUMMARY_CLASS omitted the actual declaration.

The declaration of MonthCell.__locDef() was erroneously coded as .__locDefs().

In MonthCell.__paragraph(), this line:

                div.addTextMixed ( div, s )

should be:

                tccpage2.addTextMixed ( div, s )

In MonthCell.__singleSighting(), there were four references to div that should have been to parent.

In MonthCell-ageSexGroup(), I blithely assumed that GENUS_CLASS had already been defined during the coding of MonthCell.__paragraph(), but it hadn't: in the latter routine, the class name comes from the input. Solution: define it.

In buildMonthCell(), omitted the parent argument from SubElement():

    a  =  et.SubElement ( 'a', href=monthFileName )

should be:

    a  =  et.SubElement ( td, 'a', href=monthFileName )

In YearRow.predecessor(), after finding the position of the given month in the list mmList, the value mm was compared, not its position. So in this code:

        #-- 3 --
        if mm >= len(mmList):
            raise KeyError
        else:
            return mmList[pos+1]

the first line should be:

        if pos+1 > len(mmList):

Also, thanks to the wonders of cut'n'paste, YearRow.successor() had the same error.