Comments (8)
mihaeu
commented on May 23, 2024
1
@smuuf thanks for helping me analyze this. Instead of trying to be really smart about it I instead opted to ignore this problem because really that code does not make sense from the point of view of a static analyzer.
from dephpend.
mihaeu
commented on May 23, 2024
Oh that looks interesting. Can you share the file where this occurs?
…On 17.09.19 13:20, Martín Moreno wrote:
I'm running this command
|docker run --rm -v <abs_path>:/inspect mihaeu/dephpend:latest text
/inspect|
and I'm getting this output
`Something went wrong, this shouldn't happen. Please take a minute and
report this issue: https://github.com/mihaeu/dephpend/issues
Argument 1 passed to
Mihaeu\PhpDependencies\Analyser\DependencyInspectionVisitor::addName()
must be an instance of PhpParser\Node\Name, instance of
PhpParser\Node\Expr\PropertyFetch given, called in
/dephpend/src/Analyser/DependencyInspectionVisitor.php on line 89
[/dephpend/src/Analyser/DependencyInspectionVisitor.php at line 97]`
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#56?email_source=notifications&email_token=AAQRO7OKDR5CIBWA3WD52BLQKC4PTA5CNFSM4IXONRXKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HL2NHUQ>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAQRO7LOXFB66DLUTG3JB4TQKC4PTANCNFSM4IXONRXA>.
from dephpend.
155martinmoreno
commented on May 23, 2024
Well, it's a massive project, both in files count and size.
Sorry, can't share
Could size be related?
Also, is there any verbosity option that could narrow down the search?
If it's small enough maybe I can share an extract.
from dephpend.
mihaeu
commented on May 23, 2024
No it seems to be a parser problem. All files are syntactically correct?
…On 18.09.19 10:51, Martín Moreno wrote:
Well, it's a massive project, both in files count and size.
Sorry, can't share
Could size be related?
Also, is there any verbosity option that could narrow down the search?
If it's small enough maybe I can share an extract.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#56?email_source=notifications&email_token=AAQRO7MGNH7F5GIJFA5S6NLQKHTXVA5CNFSM4IXONRXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD67KMKA#issuecomment-532588072>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAQRO7MCGV6XIPIGQGJFIODQKHTXVANCNFSM4IXONRXA>.
from dephpend.
155martinmoreno
commented on May 23, 2024
Would say yes, but I'm still trying to get my bearings in the full code.
Actually, that's why I'm using this tool.
Will try to split runs in several to see if I can narrow it down.
I'll let you know.
from dephpend.
mihaeu
commented on May 23, 2024
Yeah that's what I was going to suggest, something like `for file in
**/*.php; do dephpend text $file; done`
…On 18.09.19 11:17, Martín Moreno wrote:
Would say yes, but I'm still trying to get my bearings in the full code.
Actually, that's why I'm using this tool.
Will try to split runs in several to see if I can narrow it down.
I'll let you know.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#56?email_source=notifications&email_token=AAQRO7KERLHZDSDGKHYNMZDQKHW2NA5CNFSM4IXONRXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD67M6AY#issuecomment-532598531>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAQRO7J55DOCHUJ7PTY3UGTQKHW2NANCNFSM4IXONRXA>.
from dephpend.
smuuf
commented on May 23, 2024
I have stumbled upon the same error and using the "separate files" technique mentioned above I have found out the culprit. And it is this piece of code:
$someObject->someOtherObject::SOME_CONSTANT;Which in and of itself is a valid way of how to access constants of some instance's class (I'm using PHP 7.2), but seems like it breaks some stuff in the analyzer.
Hope this helps.
from dephpend.
mihaeu
commented on May 23, 2024
Interesting, thanks for reporting this. I'll fix it this weekend.
…On 10.10.19 12:25, Přemysl Karbula wrote:
I have stumbled upon the same error and using the "separate files"
technique mentioned above I have found out the culprit. And it is this
piece of code:
$this->someInstance::SOME_CONSTANT
Which in and of itself is a valid way of how to access constants of
some instance's class /(I'm using |PHP 7.2|)/, but seems like it
breaks some stuff in the analyzer.
Hope this helps.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#56?email_source=notifications&email_token=AAQRO7LILCL24POYQNOARKLQN37KRA5CNFSM4IXONRXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEA3XL3A#issuecomment-540505580>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAQRO7MRDWJ2TFJM6W3U6RDQN37KRANCNFSM4IXONRXA>.
from dephpend.
Related Issues (20)
- Docker container rebuild HOT 2
- Release Artifact: Can't check signature: No public key HOT 16
- When reporting syntax errors, mention the file name
- Incompatible with symfony/event-dispatcher 4.3 HOT 3
- Write errors to stderr not stdout HOT 1
- Show global namespace in --no-classes HOT 1
- SSL_ERROR_BAD_CERT_DOMAIN when trying to download phar. HOT 5
- Wrong argument on ensureDestinationIsWritable() HOT 5
- Error in text mode HOT 5
- Update to Symfony 5. HOT 2
- Website down? HOT 3
- Mihaeu\PhpDependencies\Util\DI is missing HOT 2
- Unable to execute this tool HOT 2
- It cannot parse this php 8 code. HOT 3
- broken link to animated timeline in README.md HOT 4
- Expired certificate in verification signature
- Multiple derprecations in php 8.1 HOT 1
- unexpected T_STRING error reported for php 8.1 enum
- "PlantUML installation not found" despite PlantUML being globally installed HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from dephpend.