Javadocs Feature#118
Conversation
Reaper-1709
commented
Mar 15, 2025
- Added see javadocs button in StatementBlock, ExpressionBlock, and PluginComponentBlock
- Added Javadocs tab in Plugin Settings
- Javadocs functionality for all components, and all methods is added and no other blocks supported as of now
1. Added see javadocs button in StatementBlock, ExpressionBlock, and PluginComponentBlock 2. Added Javadocs tab in Plugin Settings 3. Javadocs functionality for all components, and all methods is added and no other blocks supported as of now
|
there is a lot of code that repeats itself. wouldn't it be better to replace various pieces of code with one field and other pieces of code with a method? |
I'd work on optimisation once the project is finished maybe, I still have some ToDos: add javadocs reference for other statements, expressions and add a config file for explicitly redirecting any method and class. If you could point out the references where code can be summed up into a method, I'd really appreciate that. |
|
Please do not merge the PR yet, I mistakenly forgot to commit JavadocsManager.java |
1. roll-backed VisualBukkitApp.java due to presence of unnecessary changes 2. Commited JavadocsManager.java
|
@OfficialDonut looks good to me now, you can test it. |
|
the JavadocsWindow class is also missing |
That is an unnecessary one as well, please remove any code that mentions that class. |
- Added javadocs url for all statements and expressions - Added remap system to explicitly redirect to a different javadocs page for a given method - some optimizations in Project.java
|
I think it is fully completed now. |
- Added an overridable method as suggested - removed switches from StatementBlock.java and ExpressionBlock.java - changed java 8 urls to java 17 ones
|
It is now optimized as instructed. |
- commit revised JavadocsManager.java
|
this doesnt compile |
Update StatBungeeCordConnect.java Update gui.properties Co-Authored-By: OfficialDonut <donut42915@gmail.com>
|
If you click "see javadocs" on a method block where the class/method hasnt been selected it throws an npe |
|
Should be fixed now, along with similar errors which could have occoured with ExprMethod and ExprField. Thanks a lot for pointing out these things and sorry for my lack of testing skills. |