• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

Algo-Web / POData-Laravel / 497
100%

Build:
DEFAULT BRANCH: master
Ran 25 Sep 2017 03:59PM UTC
Jobs 4
Files 25
Run time 49s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
497

push

travis-ci

web-flow
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.

2421 of 2421 relevant lines covered (100.0%)

137.36 hits per line

Jobs
ID Job ID Ran Files Coverage
1 497.1 (LARAVEL_DEP=5.1.*) 25 Sep 2017 03:58PM UTC 0
99.79
Travis Job 497.1
2 497.2 (LARAVEL_DEP=">5.1.11") 25 Sep 2017 03:58PM UTC 0
99.92
Travis Job 497.2
4 497.4 (LARAVEL_DEP=">5.1.11") 25 Sep 2017 03:57PM UTC 0
100.0
Travis Job 497.4
6 497.6 (LARAVEL_DEP=">5.1.11") 25 Sep 2017 03:58PM UTC 0
100.0
Travis Job 497.6
Source Files on build 497
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #497
  • 9e0fa341 on github
  • Prev Build on master (#494)
  • Next Build on master (#499)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc