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

sonata-project / SonataMediaBundle / 3810
51%

Build:
DEFAULT BRANCH: 3.x
Ran 02 Aug 2019 08:35AM UTC
Jobs 9
Files 137
Run time 12min
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
3810

cron

travis-ci

core23
Fixed deletion of images with id greater than 1000 (#1592)

* Fixed deletion of images with id greater than 1000

Inside the $this->getReferenceImage() method, the id of the $media object is used, but at the time of the postRemove event, the $media object is detached from an EntityManager and its id is null. During the postRemove event, the $media->getId() method returns null, and the $this->getReferenceImage() method returns the wrong path and the file remains in the file system as garbage. My changes fix this.

* Add testPostRemove().

Test for "Fix deletion of images with id greater than 1000".

* Update testPostRemove().

* The visibility of the property "clones" is changed to private.

* Adapted test "testPostRemove" to check the data in the private property "clones" instead of the protected.

* Update src/Provider/BaseProvider.php

Co-Authored-By: Javier Spagnoletti <phansys@gmail.com>

* Update src/Provider/BaseProvider.php

Co-Authored-By: Javier Spagnoletti <phansys@gmail.com>

* Update tests/Provider/BaseProviderTest.php

Co-Authored-By: Javier Spagnoletti <phansys@gmail.com>

* Adding a separate version of the testPostRemove(), which can be run in the current branch 3, and in the proposed me.

* Revert "Adding a separate version of the testPostRemove(), which can be run in the current branch 3, and in the proposed me."

This reverts commit bab31c8e3.

* Adding a separate version of the testPostRemove(), which can be run in the current branch 3, and in the proposed me.

* Remove unnecessary comment.

* Removed unnecessary comment

* Reorganization of tests for compliance with the requirements.

* Update tests/Provider/BaseProviderTest.php

Co-Authored-By: Javier Spagnoletti <phansys@gmail.com>

* Fixes for FlintCI.

* Fixed deletion of images with id greater than 1000

Inside the $this->getReferenceImage() method, the id of the $media object is used, but at the ti... (continued)

2169 of 4214 relevant lines covered (51.47%)

37.74 hits per line

Jobs
ID Job ID Ran Files Coverage
1 3810.1 02 Aug 2019 08:34AM UTC 0
51.43
Travis Job 3810.1
2 3810.2 02 Aug 2019 08:36AM UTC 0
51.4
Travis Job 3810.2
3 3810.3 02 Aug 2019 08:36AM UTC 0
51.21
Travis Job 3810.3
6 3810.6 (COMPOSER_FLAGS="--prefer-lowest") 02 Aug 2019 08:39AM UTC 0
48.97
Travis Job 3810.6
7 3810.7 (SYMFONY=3.4.*) 02 Aug 2019 08:39AM UTC 0
51.21
Travis Job 3810.7
8 3810.8 (DOCTRINE_ODM=1.*) 02 Aug 2019 08:40AM UTC 0
51.21
Travis Job 3810.8
9 3810.9 (SONATA_CORE=3.*) 02 Aug 2019 08:40AM UTC 0
51.21
Travis Job 3810.9
10 3810.10 (SONATA_ADMIN=3.*) 02 Aug 2019 08:40AM UTC 0
51.21
Travis Job 3810.10
13 3810.13 (DOCTRINE_ODM='dev-master as 1.x-dev') 02 Aug 2019 08:46AM UTC 0
51.21
Travis Job 3810.13
Source Files on build 3810
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #3810
  • f25bc3f4 on github
  • Prev Build on 3.x (#3809)
  • Next Build on 3.x (#3813)
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