-
Notifications
You must be signed in to change notification settings - Fork 666
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix #3117 - invalidate all caches when composer lockfile changes
- Loading branch information
Showing
18 changed files
with
274 additions
and
108 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,144 @@ | ||
| <?php | ||
| namespace Psalm\Internal\Provider; | ||
|
|
||
| use const DIRECTORY_SEPARATOR; | ||
| use function file_exists; | ||
| use function file_get_contents; | ||
| use function file_put_contents; | ||
| use function is_array; | ||
| use function is_readable; | ||
| use Psalm\Internal\Analyzer\IssueData; | ||
| use Psalm\Config; | ||
| use function serialize; | ||
| use function unserialize; | ||
|
|
||
| /** | ||
| * @psalm-type TaggedCodeType = array<int, array{0: int, 1: string}> | ||
| */ | ||
| /** | ||
| * Used to determine which files reference other files, necessary for using the --diff | ||
| * option from the command line. | ||
| */ | ||
| class ProjectCacheProvider | ||
| { | ||
| const GOOD_RUN_NAME = 'good_run'; | ||
| const COMPOSER_LOCK_HASH = 'composer_lock_hash'; | ||
|
|
||
| /** | ||
| * @var int|null | ||
| */ | ||
| private $last_run = null; | ||
|
|
||
| /** | ||
| * @var string|null | ||
| */ | ||
| private $composer_lock_hash = null; | ||
|
|
||
| private $composer_lock_location; | ||
|
|
||
| public function __construct(string $composer_lock_location) | ||
| { | ||
| $this->composer_lock_location = $composer_lock_location; | ||
| } | ||
|
|
||
| /** | ||
| * @return bool | ||
| */ | ||
| public function canDiffFiles() | ||
| { | ||
| $cache_directory = Config::getInstance()->getCacheDirectory(); | ||
|
|
||
| return $cache_directory && file_exists($cache_directory . DIRECTORY_SEPARATOR . self::GOOD_RUN_NAME); | ||
| } | ||
|
|
||
| /** | ||
| * @param float $start_time | ||
| * | ||
| * @return void | ||
| */ | ||
| public function processSuccessfulRun($start_time) | ||
| { | ||
| $cache_directory = Config::getInstance()->getCacheDirectory(); | ||
|
|
||
| if (!$cache_directory) { | ||
| return; | ||
| } | ||
|
|
||
| $run_cache_location = $cache_directory . DIRECTORY_SEPARATOR . self::GOOD_RUN_NAME; | ||
|
|
||
| \touch($run_cache_location, (int)$start_time); | ||
| } | ||
|
|
||
| /** | ||
| * @return int | ||
| */ | ||
| public function getLastRun() | ||
| { | ||
| if ($this->last_run === null) { | ||
| $cache_directory = Config::getInstance()->getCacheDirectory(); | ||
|
|
||
| if (file_exists($cache_directory . DIRECTORY_SEPARATOR . self::GOOD_RUN_NAME)) { | ||
| $this->last_run = \filemtime($cache_directory . DIRECTORY_SEPARATOR . self::GOOD_RUN_NAME); | ||
| } else { | ||
| $this->last_run = 0; | ||
| } | ||
| } | ||
|
|
||
| return $this->last_run; | ||
| } | ||
|
|
||
| public function hasLockfileChanged() : bool | ||
| { | ||
| if (!file_exists($this->composer_lock_location)) { | ||
| return true; | ||
| } | ||
|
|
||
| $lockfile_contents = file_get_contents($this->composer_lock_location); | ||
|
|
||
| if (!$lockfile_contents) { | ||
| return true; | ||
| } | ||
|
|
||
| $sha1 = \sha1($lockfile_contents); | ||
|
|
||
| $changed = $sha1 !== $this->getComposerLockHash(); | ||
|
|
||
| $this->composer_lock_hash = $sha1; | ||
|
|
||
| return $changed; | ||
| } | ||
|
|
||
| public function updateComposerLockHash() : void | ||
| { | ||
| $cache_directory = Config::getInstance()->getCacheDirectory(); | ||
|
|
||
| if (!$cache_directory || !$this->composer_lock_hash) { | ||
| return; | ||
| } | ||
|
|
||
| if (!file_exists($cache_directory)) { | ||
| \mkdir($cache_directory, 0777, true); | ||
| } | ||
|
|
||
| $lock_hash_location = $cache_directory . DIRECTORY_SEPARATOR . self::COMPOSER_LOCK_HASH; | ||
|
|
||
| file_put_contents($lock_hash_location, $this->composer_lock_hash); | ||
| } | ||
|
|
||
| protected function getComposerLockHash() : string | ||
| { | ||
| if ($this->composer_lock_hash === null) { | ||
| $cache_directory = Config::getInstance()->getCacheDirectory(); | ||
|
|
||
| $lock_hash_location = $cache_directory . DIRECTORY_SEPARATOR . self::COMPOSER_LOCK_HASH; | ||
|
|
||
| if (file_exists($lock_hash_location)) { | ||
| $this->composer_lock_hash = file_get_contents($lock_hash_location) ?: ''; | ||
| } else { | ||
| $this->composer_lock_hash = ''; | ||
| } | ||
| } | ||
|
|
||
| return $this->composer_lock_hash; | ||
| } | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.