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.
