Hi Christian,
We should change that as we want to store the
signature to the file directly from the context menu. As sometimes a
file might be read-only it is a good idea to have alternative
locations to store signatures.
Do we need to append signature directly to the file to which it is getting applied? If so, won't user be able to manipulate it even unintentionally?
What I propose is:
- We store only signature to 'state location' of plugin so that user cannot directly access it. See [1] on stack overflow regarding state location.
- We provide a context menu 'Extract Signature' which will be active only when signature is present.
- Later on, user can chose to extract signature using 'Extract Signature' context menu to his preferred location.
- We can also provide user with an option to remove signature using context menu.
I have updated bug related to "Unable to access state location". Signature will now go to caller of wizard using instance of GetInfo class, in our case to PerformSignature handler class. See updated patch at [2].
[1]: http://stackoverflow.com/questions/37361643/save-file-info-for-processing-not-seen-by-user-in-eclipse
[2]: https://git.eclipse.org/r/#/c/76997/
Thanks,