Fixes #12 : Bug 577317 - [17] Provide quick fixes to add permitted

[ktatavarthi]

class/interface for sealed class/interface

Signed-off-by: Kalyan Prasad Tatavarthi kalyan_prasad@in.ibm.com

[SarikaSinha]

If I use quick fix to add class in same package, the "permits" is not coloured properly. After going back to the file and adding a "enter" applies the colouring.

[SarikaSinha]

For interface, we have more challenges.
We already had 2 quickfixes to create implementation class and to create new interface.
These 2 should not appear if sealed is used as this again leads to errors in both the files.

Also, we can explore keeping these quickfixes for class and interface even without errors to add new permits. We can discuss that in a separate issue.

[ktatavarthi]

If I use quick fix to add class in same package, the "permits" is not coloured properly. After going back to the file and adding a "enter" applies the colouring.

I am looking into this.

[ktatavarthi]

For interface, we have more challenges. We already had 2 quickfixes to create implementation class and to create new interface. These 2 should not appear if sealed is used as this again leads to errors in both the files.

Also, we can explore keeping these quickfixes for class and interface even without errors to add new permits. We can discuss that in a separate issue.

As currently there are no quick fixes available, I think these quick fixes are valid. There is no way to avoid error using sealed in the class creation wizard as there is no easy solution for this other than quick fixes. If we restrict creation of permits class or interface with final and non-sealed modifiers and not allow sealed modifier, then we restrict our Quick Fix to not create sub types which can also be sealed.

[SarikaSinha]

Created #25 as follow up.