#720: BibFormat: release output format naming convention constraints
------------------------+----------------------
  Reporter:  simko      |      Owner:  rajimene
      Type:  defect     |     Status:  assigned
  Priority:  major      |  Milestone:
 Component:  BibFormat  |    Version:
Resolution:             |   Keywords:
------------------------+----------------------
Changes (by simko):

 * status:  in_merge => assigned


Comment:

 You need to respect also the old convention for text/plain formats, i.e.
 edit bibformat.py `get_output_format_content_type()` migration part to say
 something like:

 {{{
         if content_type == '':
             # let's check if of starts with 'x' or 't' which was the
             # old output format naming convention, in order to speed
             # up migrations:
             if of.startswith('x'):
                 content_type = 'text/xml'
             elif of.startswith('t'):
                 content_type = 'text/plain'
             else:
                 content_type = 'text/html'
 }}}

 (Currently only `x` is checked there.)

 However, even if you do this, URLs like `/search?p=ellis&of=tm&ot=100,700`
 do not behave properly when compared to master.  See for example ` sudo -u
 www-data python ./modules/websearc /lib/websearch_regression_tests.py
 --yes-i-know`. Please check and amend as necessary.

-- 
Ticket URL: <http://invenio-software.org/ticket/720#comment:4>
Invenio <http://invenio-software.org>

Reply via email to