don't overwrite reflection cache, when analyzing only a single file in phpstorm

[staabm]

references #443 (comment)

related phpstan discussion phpstan/phpstan#8259

[staabm]

@noemi-salaun could you please give this change a try. I think it should prevent phpstorm-dba to write/change the result cache file, when the phpstan analysis was started from within phpstorn

[noemi-salaun]

@staabm It looks good. If I update a query, the new query is added in the cache file but everything else is left untouched in the cache file 👍
If I keep modifying the same request, every change add a new query in the cache file. Obsolete queries are keeped in the cache file even when I run phpstan manually. I need to remove the cache and re-run phpstan to get a clean cache without obsolete queries.

[staabm]

Obsolete queries are keeped in the cache file even when I run phpstan manually. I need to remove the cache and re-run phpstan to get a clean cache without obsolete queries.

yeah, thats a known issue, see #310
it shouldn't be a problem though, when obsolete queries are contained in the cache file

[staabm]

I can no longer reproduce the underlying issue. we need a failing unit test first