jafl/yui3-gallery — pull request #2 by kvikas (2011-11-07 14:33:37+00:00)
Title: Update src/gallery-quickedit/js/QuickEdit.js

Consider a case where a field is both, editable and in always include list. In this case the expected behavior should be that user edit should be preferred over existing value.

Example:

date = [
{id: 1, state: 'CA', city: 'xyz'},
{id: 2, state: 'AZ', city: 'abc'},
]

'state' and 'city' are editable. However there can be same city name across two states hence both state and city is required by backend to change location of the entries. Therefore 'state' is part of always inlude list along with 'id'. Now this works for cases where only 'city' is edited. But if cases where both state and city are changed, present code overwrites user entered value for state.

Please review my change to fix this.

Thanks.