Hi,<br>
<br>
do we really need to reset the author initials when one opens the image under a different path? This just stroke me a bit when I was trying to prepare an image that does not ask the user for any initials, but the receiver was still told to enter them again.<br>
<br>
Also, note that string comparison is not sufficient for comparing paths. For instance, there are multiple ways under Windows to express the same path (UNC paths, virtual drive letters, ...), or even the WSL path when I open the same image, without moving it, from WSL rather from Windows.<br>
<br>
Last but not least, this is a really unexpected side effect and I only found this via a sender search.<br>
<br>
Is this really necessary? :-)<br>
<br>
Best,<br>
Christoph<br>
<br>
<font color="#808080">---<br>
</font><font color="#808080"><i>Sent from </i></font><font color="#808080"><i><a href="https://github.com/hpi-swa-lab/squeak-inbox-talk"><u><font color="#808080">Squeak Inbox Talk</font></u></a></i></font><br>
<br>
On 2018-04-16T08:31:43+00:00, commits@source.squeak.org wrote:<br>
<br>
> Marcel Taeumel uploaded a new version of System to project The Trunk:<br>
> http://source.squeak.org/trunk/System-mt.1018.mcz<br>
> <br>
> ==================== Summary ====================<br>
> <br>
> Name: System-mt.1018<br>
> Author: mt<br>
> Time: 16 April 2018, 10:31:28.239614 am<br>
> UUID: e0350560-a5a5-5a40-b31b-d13f55162021<br>
> Ancestors: System-mt.1017<br>
> <br>
> #setAuthorInitials: -> #authorInitials:<br>
> <br>
> =============== Diff against System-mt.1017 ===============<br>
> <br>
> Item was changed:<br>
>   ----- Method: SmalltalkImage>>openSourceFiles (in category 'sources, changes log') -----<br>
>   openSourceFiles<br>
>   <br>
>       self imageName = LastImageName ifFalse:<br>
>           ["Reset the author initials to blank when the image gets moved"<br>
>           LastImageName := self imageName.<br>
> +         Utilities authorInitials: ''].<br>
> -         Utilities setAuthorInitials: ''].<br>
>       FileDirectory<br>
>           openSources: self sourcesName<br>
>           andChanges: self changesName<br>
>           forImage: LastImageName.<br>
>       SourceFileArray install!<br>
> <br>
>