34ff3903e7811c07f8b6c1ed9123cd75a1d3c865
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 2
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Move Method | 1 | ||
Move Method public resolve(fromModulePath String, relativeToModulePath String) : String from class com.google.javascript.jscomp.ClosureRewriteModule.GlobalRewriteState to private resolve(fromModulePath String, relativeToModulePath String) : String from class com.google.javascript.jscomp.ClosureRewriteModule | From | To | |
Rename Variable | 1 | ||
Rename Variable targetIsALegacyScript : boolean to requiredScript : Optional<ScriptDescription> in method private recordGoogRequire(t NodeTraversal, call Node, mustBeOrdered boolean) : void in class com.google.javascript.jscomp.ClosureRewriteModule | From | To | |
Change Return Type | 3 | ||
Change Return Type String to Optional<String> in method package getExportedNamespace() : Optional<String> in class com.google.javascript.jscomp.ClosureRewriteModule.ScriptDescription | From | To | |
Change Return Type String to Optional<String> in method package getBinaryNamespace() : Optional<String> in class com.google.javascript.jscomp.ClosureRewriteModule.ScriptDescription | From | To | |
Change Return Type String to Optional<String> in method private getExportedNamespaceOrScript(legacyNamespace String) : Optional<String> in class com.google.javascript.jscomp.ClosureRewriteModule.GlobalRewriteState | From | To | |
Merge Variable | 1 | ||
Merge Variable [targetGoogModuleExists : boolean, targetLegacyScriptExists : boolean] to requiredScript : Optional<ScriptDescription> in method private reportUnrecognizedRequires() : void in class com.google.javascript.jscomp.ClosureRewriteModule | From | To | |
Rename Attribute | 1 | ||
Rename Attribute scriptDescriptionsByGoogModuleNamespace : Map<String,ScriptDescription> to namespaceToScript : Map<String,ScriptDescription> in class com.google.javascript.jscomp.ClosureRewriteModule.GlobalRewriteState | From | To | |
Extract Variable | 2 | ||
Extract Variable legacyNamespacesToRemove : Collection<String> in method package removeRoot(toRemove Node) : void from class com.google.javascript.jscomp.ClosureRewriteModule.GlobalRewriteState | From | To | |
Extract Variable binaryNamespace : Optional<String> in method private maybeUpdateTopLevelName(t NodeTraversal, nameNode Node) : void from class com.google.javascript.jscomp.ClosureRewriteModule | From | To | |
Change Variable Type | 2 | ||
Change Variable Type exportedNamespace : String to exportedNamespace : Optional<String> in method private updateGoogRequire(t NodeTraversal, call Node) : void in class com.google.javascript.jscomp.ClosureRewriteModule | From | To | |
Change Variable Type targetIsALegacyScript : boolean to requiredScript : Optional<ScriptDescription> in method private recordGoogRequire(t NodeTraversal, call Node, mustBeOrdered boolean) : void in class com.google.javascript.jscomp.ClosureRewriteModule | From | To | |
Extract And Move Method | 1 | ||
Extract And Move Method package addNamespaceToScript(namespace String, script ScriptDescription) : void extracted from private recordGoogModule(t NodeTraversal, call Node) : void in class com.google.javascript.jscomp.ClosureRewriteModule & moved to class com.google.javascript.jscomp.ClosureRewriteModule.GlobalRewriteState | From | To |