6
51%
master: 51%

Ran 22 Dec 2014 02:53AM UTC

Files 106

Run time 45s

Badge

Embed ▾

Committed 22 Dec 2014 01:54AM UTC coverage: 48.69% (+0.008%) from 48.682%

Job # 326.6

Build Type

push

travis-ci

Committed by

vanstyn

Commit Message

Re-enable storeId, but append req_num + JS on destroy cleanup - #100

In the previous commit the setting of 'storeId' was disabled, however,
there are some code paths which rely on storeId to resolve/access a data
store by its id, specifically related to componenets which make use of
the 'defer_DataStore' API. The design of this case should probably be
revisited, however, in the meantime to preserve the existing API and
also fix the caching bug that is present in perl 5.18 and later, the
storeId now has the request number appended. This ensures that the same
componenent will always have a unique storeId when opened multiple
times on the client side, but getStore lookup code which is generated
within the same request will still work.

Also added extra cleanup code to the DataStorePlus javascript plugin to
lookup and remove the store by Id from Ext.StoreMgr on destroy of the
parent component. This prevents store objects from piling up in the
global StoreMgr collection in the client machine for components which
may have already been closed/destroyed.

Coverage Stats

3084 of 6334 relevant lines covered (48.69%)

221.57 hits per line

vanstyn / RapidApp / 326 / 6
51%
master: 51%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 326.6

vanstyn / RapidApp / 326 / 6 51% master: 51%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 326.6

vanstyn / RapidApp / 326 / 6
51%
master: 51%

README BADGES
x