composer-patches icon indicating copy to clipboard operation
composer-patches copied to clipboard

Properly handle all patches being ignored

Open dotdash opened this issue 1 year ago • 0 comments

Description

The check to determine if there are any patches present for a given package currently yields true for an empty set of patches, which can happen if all patches provided by a package are marked as ignored. This causes the affected package to be re-installed everytime, because $has_patches is true while $has_applied_patches is false. The logic for $has_patches should also check for the number of patches being greater than 0, just like it is for $has_applied_patches.

Related tasks

  • [x] Does not break backwards compatibility OR a BC break has been discussed in the related issue(s).

dotdash avatar Jan 03 '24 10:01 dotdash