Add support for renaming methods, classes and fields #794
Conversation
…yRealName() to JadxWrapper Those methods is like getFullName() and searchJavaClassByClassName(), but for class names without aliases. It is necessary for renaming classes/methods/fields.
This is necessary for renaming functionality
…ields) It allows user to rename classes, methods and fields. It updates deobfuscation map and reload file. This may be suboptimal, and maybe some RenameVisitor should be added. Deobfuscation should be enabled in order to allow this.
|
Hi, thanks for adding this feature! |
|
@ppwwyyxx That seems to be a problem with the GUI of the rename dialog. The dialog itself can be resized. If you do so and increase the height the text field where you can change the name becomes visible. The problem source seems to be this line: Comment it out and the dialog is usable. |
|
Just for anyone interested in the advanced rename feature - please check out the https://github.com/skylot/jadx/tree/rename branch to test it. Please report any issues to the #791 issue (if any). |
This request adds a Rename popup menu item that allows the user to rename classes, methods and fields.
It is useful for obfuscated code, it makes it much easier to read.
The implementation is very basic and perhaps not optimal, but it works.
I'm working for a better implementation which will not reload whole file at every rename, but it's still in progress.
This request is related to issue #791 but does not completely closes it - there is a place for improving this implementation,
However, no feature is worse than slow, but working feature - so, it's will be good to merge current implementation and then improve it.