Uh oh!
There was an error while loading. Please reload this page.
adt support - #487
Conversation
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
| Object part = partRef.getPart(false); | ||
| if (part instanceof ITextEditor) { | ||
| handleEditorChange((ITextEditor) part); | ||
| } else if (part instanceof IEditorPart) { |
There was a problem hiding this comment.
isnt ItextEditor as IEditorPart as well?
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
| } else if (editorInput instanceof IFileEditorInput fileEditorInput) { | ||
| return fileEditorInput.getFile().getRawLocation().toOSString(); | ||
| var file = fileEditorInput.getFile(); | ||
| if (file.getRawLocation() == null && AbapUtil.isAbapFile(file)) { |
There was a problem hiding this comment.
Also add a comment indicating that special handling is needed for abap files because we are referencing the cached location it is stored with ADT plugins.
Please add this.
| } | ||
| @Test | ||
| void isAdtEditorWithSapPackagePrefixReturnsTrue() { |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
| private void setupPlatformMock() { | ||
| setupPlatformMockWithPaths( | ||
| "file:///workspace/.metadata/.plugins/org.eclipse.core.resources.semantic/.cache/test/path", | ||
| "/workspace/.metadata/.plugins/org.eclipse.core.resources.semantic", |
There was a problem hiding this comment.
shouldnt this be c:/users instead?
Uh oh!
There was an error while loading. Please reload this page.
Merges feature branch containing improvements for ADT ABAP support into main. Relevant PRs * adt support #487 * Support updating remote when ABAP files are edited #495 * handle the null case for contentType in adt plugin environment #496 * Fix: Handle removing paint listeners with ADT viewer and allow multiline text in inline chat #500
Problems
The Amazon Q Eclipse plugin lacked proper support for ABAP files when used with the ADT (ABAP Development Tools) plugin. ABAP developers could not access inline completion suggestions or benefit from active file functionality that was available for other programming languages. The plugin was unable to recognize ABAP workspace structure and files, preventing Amazon Q features from working in ABAP development environments.
Solutions
This PR enables ABAP support by registering a new editor to handle ABAP files in the active file system and utilizing temporary folder management to make ABAP files accessible to Q chat functionality. The solution registers common ABAP files into the workspace so clients can utilize inline suggestions for ABAP development. This allows ABAP files to display the Q button for inline suggestions and be properly recognized by the Amazon Q system.
Further work
Future improvements should include
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.