#18: crash in /yourgroups related to unwashed arguments
-------------------------+--------------------------------------------------
  Reporter:  simko       |       Owner:  jmartinm
      Type:  defect      |      Status:  reopened
  Priority:  major       |   Milestone:          
 Component:  WebSession  |     Version:          
Resolution:              |    Keywords:          
-------------------------+--------------------------------------------------
Changes (by simko):

  * status:  closed => reopened
  * resolution:  fixed =>


Comment:

 In some places, the code comments say:

 {{{
    'grpID' *string* - ID of the group(not null in case of group editing)
 }}}

 or this:

 {{{
    if grpID != "":
 }}}

 or this:

 {{{
    grpID = int(grpID)
 }}}

 So some parts of the code appear to assume grpID to be a string.

 Is changing {{{wash_urlargd()}}} types enough to cover all group
 facility related use cases?  Can you please check, and correct
 int/str as appropriate, and modify docstrings?

-- 
Ticket URL: <http://cdswaredev.cern.ch/invenio/ticket/18#comment:3>
CDS Invenio <http://cdswaredev.cern.ch/invenio>
CDS Invenio digital library software.

Reply via email to