[Bugs] #1262 UNSP: Process non-ds object in the right way in Journal

[SugarLabs Bugs]

#1262: Process non-ds object in the right way in Journal
------------------------------------------+---------------------------------
    Reporter:  alsroot                    |          Owner:  tomeu            
        Type:  defect                     |         Status:  new              
    Priority:  Unspecified by Maintainer  |      Milestone:  0.86             
   Component:  sugar                      |        Version:  Git as of bugdate
    Severity:  Unspecified                |     Resolution:                   
    Keywords:  r!                         |   Distribution:  Unspecified      
Status_field:  Unconfirmed                |  
------------------------------------------+---------------------------------

Comment(by alsroot):

 Replying to [comment:3 tomeu]:
 > The patch looks good, just a couple of questions:
 >
 > {{{
 > 374           def find(query, page_size):
 >       374     def find(query_, page_size):
 > }}}

 because it changes query argument(by poping 'mountpoints'), so next
 invoking of ListModel() in listview.py:287 will not contain 'mountpoints'

 > Why do we need to make this change?
 >
 > {{{
 >         mountpoints = metadata.get('mountpoints')
 > }}}
 >
 > When do we have a 'mountpoints' property? Didn't knew about it.

 model.is_editable() is used for query as well(when we don't have any row,
 see the patch)

-- 
Ticket URL: <http://dev.sugarlabs.org/ticket/1262#comment:4>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system