2
100%
master: 100%

Ran 25 Sep 2017 04:00PM UTC

Files 25

Run time 1s

Badge

Embed ▾

Committed 25 Sep 2017 03:56PM UTC coverage: 99.917% (-0.08%) from 100.0%

Job # LARAVEL_DEP=">5.1.11"

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.

Coverage Stats

2396 of 2398 relevant lines covered (99.92%)

36.72 hits per line

Algo-Web / POData-Laravel / 497 / 2
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 497.2 (LARAVEL_DEP=">5.1.11")

Algo-Web / POData-Laravel / 497 / 2 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 497.2 (LARAVEL_DEP=">5.1.11")

Algo-Web / POData-Laravel / 497 / 2
100%
master: 100%

README BADGES
x