497
100%

Ran 25 Sep 2017 03:59PM UTC

Jobs 4

Files 25

Run time 49s

Badge

Embed ▾

pending completion

Build # 497

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Fix PrimaryKey substitution in LaravelReadQuery (#117)

When applying a non-trivial orderBy object that contained PrimaryKey
field, getResourceSet was ignoring previously-retrieved copy of target
model's primary key and trying to dig it out anew.  This doesn't work
too well on query builders, as getKeyName doesn't exist.

The simple and obvious fix is to simply cut the orderBy handling over
to use the previously-retrieved copy of keyName, and let it worry about
whether we're hanging off a model, relation, etc.

Run Details

2421 of 2421 relevant lines covered (100.0%)

137.36 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	*497.1 (LARAVEL_DEP=5.1.)**	25 Sep 2017 03:58PM UTC	99.79	Travis Job 497.1
2	497.2 (LARAVEL_DEP=">5.1.11")	25 Sep 2017 03:58PM UTC	99.92	Travis Job 497.2
4	497.4 (LARAVEL_DEP=">5.1.11")	25 Sep 2017 03:57PM UTC	100.0	Travis Job 497.4
6	497.6 (LARAVEL_DEP=">5.1.11")	25 Sep 2017 03:58PM UTC	100.0	Travis Job 497.6

Source Files on build 497

Detailed source file information is not available for this build.

Algo-Web / POData-Laravel / 497
100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 497

Algo-Web / POData-Laravel / 497 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 497

Algo-Web / POData-Laravel / 497
100%

README BADGES
x