Skip to content

Instantly share code, notes, and snippets.

@jeantil
Last active April 1, 2016 15:10
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save jeantil/91a39517c3cd61e84c1b0ff8c85d5381 to your computer and use it in GitHub Desktop.
Save jeantil/91a39517c3cd61e84c1b0ff8c85d5381 to your computer and use it in GitHub Desktop.
Notes on some intellij issues I want to tackle

SCL-xxxxx

There are a couple more issues I am interested in fixing but I haven't found where to look yet, pointer would be nice the first is fairly recent (haven't opened the youtrack yet) given the following code

class A()

trying to trigger the move refactor when the cursor is after A results in an error popup which says that Move method is not supported for constructors while it would make sense if the cursor was after the ( I think that before the paren it should trigger a move class not a move method. I have no idea where to look though

niktrop 12:52 For the first issue you can start from MoveHandler.invoke() method. Probably you'll need to modify >ScalaMoveClassesOrPackagesHandler to work with primary constructors. https://gitter.im/JetBrains/intellij-scala?at=56fe52eee4a8384a1bbdeb79

SCL-9300

Still in the move refactoring category (or related) is https://youtrack.jetbrains.com/issue/SCL-9300

Inspection move xxx to package yyy doesn't offer to create package in current source directory if it doesn't exist Any pointer for starting to tackle these issues would be very appreciated

niktrop 12:52 For the second, check ScalaMoveToPackageQuickFix. https://gitter.im/JetBrains/intellij-scala?at=56fe52eee4a8384a1bbdeb79

#SCL-9296 (link with IDEA-152227)

starting point in org.jetbrains.jps.model.serialization.PathMacroUtil#getModuleDir and com.intellij.execution.JavaTestFrameworkRunnableState

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment