1520
95%
master: 87%

Ran 28 Sep 2016 06:48PM UTC

Jobs 1

Files 106

Run time 7s

Badge

Embed ▾

pending completion

Build # 1520

Build Type

push

travis-ci

Committed by

jgondron

Commit Message

AIMS-358:
-Problem was due to the usage of Batch.current_match. It made an assumption about which match should be modified. If the user clicked on it multiple times, such as in the case of an impatient user clicking the button again before receiving a response, it would receive all of the requests and cause each match in order to be accepted, without the user being aware. Changed it to expect the form to post the match id as a param so that it's explicitly modifying the correct match.
-This still does not prevent the user from clicking multiple times when binning an item. It merely prevents it from modifying the wrong matches when this happens. At worst, it will just make the activity log look odd by having multiple "AssociatedItemAndBin" activities.
-There are additional problems in other parts of the workflow that will be affected by this reliance on Batch.current_match. This commit specifically addresses the part of the workflow affected in AIMS-358. Future commits on this branch will be to correct others.

Run Details

2105 of 2246 relevant lines covered (93.72%)

26.29 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	1520.1 (2.1.4)	28 Sep 2016 06:48PM UTC	0	93.72	Travis Job 1520.1

Source Files on build 1520

Detailed source file information is not available for this build.

ndlib / annex-ims / 1520
95%
master: 87%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1520

ndlib / annex-ims / 1520 95% master: 87%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1520

ndlib / annex-ims / 1520
95%
master: 87%

README BADGES
x