CloudPG v8 Upgrade #62
Conversation
* updated version in build.gradle.kts * added new typescript module * bump java target form 11 to 17 * remove experimental opt-ins and comment out various error sources for now * replace String representation by Direction value and remove warning in check * simplify equality check * replace String value by Direction * replace String value by Direction * removed imports * fixed many incompatibilities between Name and String by reading the localName and creating new Name objects without parents where needed * added FIXME statements * fix all remaining errors regarding Name and String types * fix the creation of new Statements in all Handlers * apply spotless * fixed rest of the references in the handlers * bumped CPG from 6.2.0 to 6.2.2 to solve issue of TranslationResult.additionalNodes not being mutable * fix inheritance of the frontend * trying to fix Azure.kt by replacing every instance of CallExpression.base by CallExpression.callee * immediately bump CPG version to 7.0.0 in order to save redundant work on passes * adapt all passes to the new structure in CPG 7.0.0 * fix pass registration to work with CPG 7.0.0 * add RubyLanguage.kt * bump neo4j to version 4.0.5 to fix NoSuchMethod-Errors * manually register the default Languages * correcly overwrite visit in all IVisitors. Also stop trying to get "additionalNodes" from scratch since they are now a part of the result * split the searches again to prevent any order errors * make spotless * add comments * apply various fixes from oxisto's branch "cpg-v6" * fix order of evaluation leading to NPE * made code resistant against unsafe casts, null values, empty lists. Also added comments in these places. * remove unused import * replace calls of deprecated TranslationResult.translationUnits * remove heaps of warnings concerning unused variables/parameters and deprecated calls * correctly override additional "visit" functions * fix typo * manage debugging comments * add full request name parsing that was not working in previous versions * add comments regarding (missing) functionality * return all visitors into its original form, this prevents any subclasses from being visited as well * manually register the GoExtraPass * move defaultPasses after language registrations
…ponding VariableDeclaration
|
Main problems at the moment seem to be:
In both locations, trying to dereference a DeclaredReferenceExpression fails as the "refersTo" field is always null. Edit: |
|
The problem in the RequestPass (for python requests) can be fixed by making it depend on the GoExtraPass. |
This sound strange. I suspect that the general order of passes ist still incorrect. I can try to have a look later. |
After a quick look, I would suggest that all of these special passes should depend on at least the We should probably add a new base class that all of those passes derive from and add the annotation to that base class. |
… DeclarationStatement
…"plain_text" to the "message" (one of it via its inclusion in "signature"). This doubles the original two results (there are two HttpEndpoint nodes)
…nked to the "identifier" variable as well as the url.Values CallExpression
…newer CPG versions
…rty-graph into cpg-version-update
This PR aims to upgrade the CPG-dependency to version 7 while still maintaining the full functionality of the CloudPG.
The work done in this PR also includes changes from @oxisto to #58.
This PR is still WIP, necessary steps include: