rubberduck-vba / Rubberduck

Every programmer needs a rubberduck. COM add-in for the VBA & VB6 IDE (VBE).
https://rubberduckvba.com
GNU General Public License v3.0
1.91k stars 299 forks source link

Default fix for synchronizing module attribute #4977

Open djrivers opened 5 years ago

djrivers commented 5 years ago

Version 2.4.1.4723 OS: Microsoft Windows NT 10.0.17763.0, x64 Host Product: Microsoft Office x64 Host Version: 16.0.11601.20204 Host Executable: MSACCESS.EXE

If the @ModuleDescription was already set at synchronized before then an edit was made changing the content of the description, is it by design that applying the "Fix all detected occurrences" will update the value of the @ModuleDescription to whatever the value of attribute VB_Description is?

If yes, would there be a way or an option to instead making the value of @ModuleDescription same as the value of VB_Description when the "Fix all detected occurrences" button was selected, be applied the other way? Update the value of VB_Description to whatever the value of @ModuleDescription.

Thanks, Darren

retailcoder commented 5 years ago

Hm, the default fix for any inspection result is whichever quickfix is listed first. Would it make more sense to just flip the order of applicable fixes for these inspection results? Thinking it's more likely you edit the annotation than the attribute (right?)...

djrivers commented 5 years ago

Well, for me, yes. :) I think it makes more sense that the reason for editing the annotation is to edit the attribute. :)