search

fidley / ABAPQuickFix

ABAP Quick Fix
https://abapblog.com
MIT License
42 stars 9 forks source link

Replace read table to new syntax #26

Closed reyemsaibot closed 1 year ago

reyemsaibot commented 2 years ago

Hi,

first, thanks for this great tool. I discovered a bug in eclipse 2021-09 when I want to replace an old read table. Here is a short video

https://user-images.githubusercontent.com/6608522/138066924-9934c678-a94a-4647-9c47-89524f84fc04.mp4

fidley commented 2 years ago

Hi Tobias, thanks for kind words. This bug is nice one. As far as I remember this quick fix is regex based so when you have a comment with READ word inside it goes crazy. But as you have an example I will be able to reproduce it and fix it.

Thanks!

reyemsaibot commented 2 years ago

Yes I tried it without the comment and then it worked ;)

fidley commented 2 years ago

Anyway I need to secure that :)

reyemsaibot commented 1 year ago

Hi Lukasz,

has you a fix yet?

Best regards

Tobias

fidley commented 1 year ago

Hello Tobias, in fact I've corrected it , but not published as I also started to add the help contents to the plugin and I want to finish before I publish the new version. Hope you can wait a bit longer ;) Cheers Łukasz

fidley commented 1 year ago

Should be solved