Browse Source

Fix regression in handling of URL changes, fixes #947

Jordi Boggiano 6 years ago
parent
commit
752151d5aa

+ 1 - 1
src/Packagist/WebBundle/Controller/PackageController.php

@@ -550,7 +550,7 @@ class PackageController extends Controller
 
         $update = $req->request->get('update', $req->query->get('update'));
         $autoUpdated = $req->request->get('autoUpdated', $req->query->get('autoUpdated'));
-        $updateEqualRefs = $req->request->get('updateAll', $req->query->get('updateAll'));
+        $updateEqualRefs = (bool) $req->request->get('updateAll', $req->query->get('updateAll'));
 
         $user = $this->getUser() ?: $doctrine
             ->getRepository('PackagistWebBundle:User')

+ 3 - 0
src/Packagist/WebBundle/Service/Scheduler.php

@@ -21,6 +21,9 @@ class Scheduler
 
     public function scheduleUpdate($packageOrId, $updateEqualRefs = false, $deleteBefore = false, $executeAfter = null): Job
     {
+        $updateEqualRefs = (bool) $updateEqualRefs;
+        $deleteBefore = (bool) $deleteBefore;
+
         if ($packageOrId instanceof Package) {
             $packageOrId = $packageOrId->getId();
         } elseif (!is_int($packageOrId)) {