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

lookitsatravis / listify / 68
100%

Build:
DEFAULT BRANCH: master
Ran 07 Feb 2017 01:28AM UTC
Jobs 2
Files 1
Run time 5s
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
68

Pull #22

travis-ci

web-flow
Bugfix: Move item to bottom of list on scope change

The `addToListBottom` function was only setting the new list position if the item was not already in the list. This created an issue when updating an existing record to have a new scope (via the `checkScope` function)... because the record already exists so it was already in the list, and hence it was never getting its position number updated within the new scope.

So I removed the `if($this->isNotInList())` check in the `addToListBottom` function, and it works now.
Pull Request #22: Bugfix: Move item to bottom of list on scope change

1 of 1 new or added line in 1 file covered. (100.0%)

117 of 278 relevant lines covered (42.09%)

2.93 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
161
100.0
src/Lookitsatravis/Listify/Listify.php
Jobs
ID Job ID Ran Files Coverage
1 68.1 07 Feb 2017 01:29AM UTC 0
44.83
Travis Job 68.1
3 68.3 07 Feb 2017 01:28AM UTC 0
41.88
Travis Job 68.3
Source Files on build 68
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #68
  • Pull Request #22
  • PR Base - master (#65)
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

© 2025 Coveralls, Inc