Rather, display that not all items were saved.

2010/2/2 Naftoli Gugenheim <[email protected]>:
> Additionally ItemsListEditor will catch SQLExceptions in ItemsList.save and 
> display them.
>
> On Tue, Feb 2, 2010 at 8:26 PM, Naftoli Gugenheim <[email protected]> 
> wrote:
>> I have a patch on Review Board. I haven't asked on the list for the
>> committers to review it because I've needed to perfect it still. It
>> occurred to me that some of the tickets might not have been discussed
>> on the list, so I want to do so now in case I didn't yet, although the
>> enhancements included are trivial and pretty basic.
>> As you may know, the net.liftweb.mapper.view package contains some
>> useful Mapper-related components. One is TableEditor, which provides a
>> very simple snippet to edit a Mapper table--useful for very basic
>> lists. It's based on ItemsListEditor, which allows for similar
>> functionality in a much more configurable way. It in turn uses
>> ItemsList, which is a convenient way to hold a list of Mapper entities
>> and keep track of new items pending being saved, and items pending
>> removal. This way a user can cancel their edits, as long as they have
>> not clicked Save.
>> They tickets are:
>> 299 - ItemsList should be have refresh method to clear added/removed
>> without requerying database
>>  Until now it only had a reload method, which reset the state to that
>> of the database, clearing pending additions and deletions. This adds a
>> refresh method, which only reloads items but remembers pending
>> additions and deletions.
>> 300 - ItemsList.save unremoves removed unsaved items
>>  This is a defect. After a number of attempts, I made a more
>> substantial change to fix it.
>> 301 - ItemsListEditor should allow custom columns
>>  Provide a hook so user code can add additional columns, e.g., calculated 
>> data.
>> 302 - ItemsListEditor should display items pending removal, albeit in
>> strikeout font
>>  Previously when you click 'Remove' the item disappears, and only
>> reappears if you reload it without saving first. This places the items
>> at the end of the list, uneditable and in a strikeout font.
>>
>> I would like to add a new enhancement as well, namely to generate
>> javascript to prompt you if you try to leave the page with unsaved
>> items.
>> Thoughts?
>>
>> Thanks.
>>
>
>

-- 
You received this message because you are subscribed to the Google Groups 
"Lift" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en.

Reply via email to