<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000;text-align: left" dir="ltr">
                                        Hi Levente --<div><br></div><div>Okay. I will put this back on my plate and deal with it tomorrow. Let's see whether I can manage to remove that Collections -> System dependency altogether.</div><div><br></div><div>Thanks for your review!</div><div><br></div><div>Best,</div><div>Marcel</div><div class="mb_sig"></div>
                                        <blockquote class="history_container" type="cite" style="border-left-style: solid;border-width: 1px;margin-top: 20px;margin-left: 0px;padding-left: 10px;min-width: 500px">
                        <p style="color: #AAAAAA; margin-top: 10px;">Am 27.11.2021 09:56:39 schrieb Levente Uzonyi <leves@caesar.elte.hu>:</p><div style="font-family:Arial,Helvetica,sans-serif">Hi Marcel,<br><br>This change adds another (2nd) dependency between Collections and System.<br>I think it would be better if this method were in another package with <br>initialAnswer: being a parameter.<br><br>I also think that the methods with similar name of Model in the Kernel <br>package do not belong there. They are entirely related to files and not <br>the Model itself.<br>Model's subclasses can override #suggestedFileNameForSave, but that <br>abstraction is clearly not sufficient in case of TranscriptStream, hence <br>this method. It's also not used at all in Trunk.<br><br>I also think there's also a small bug with this method: Model's <br>implementation handles the case where the selected filename is nil, and I <br>don't see why that cannot happen here but this method assumes it is always <br>a string.<br><br><br>Levente<br><br>P.S.: I vaguely remember complaining when the first dependency between <br>Colllections and System appeared back in 2020 (HtmlReadWriter -> <br>RealEstateAgent).<br><br>On Thu, 25 Nov 2021, commits@source.squeak.org wrote:<br><br>> Marcel Taeumel uploaded a new version of Collections to project The Trunk:<br>> http://source.squeak.org/trunk/Collections-mt.962.mcz<br>><br>> ==================== Summary ====================<br>><br>> Name: Collections-mt.962<br>> Author: mt<br>> Time: 25 November 2021, 11:50:55.017354 am<br>> UUID: 3826da22-e39d-2a4a-ada1-95f8f67c85fd<br>> Ancestors: Collections-pre.961<br>><br>> Tools-mt.1075<br>><br>> =============== Diff against Collections-pre.961 ===============<br>><br>> Item was added:<br>> + ----- Method: TranscriptStream>>saveContents:accessMode: (in category 'model protocol') -----<br>> + saveContents: stringContents accessMode: accessMode<br>> +     "Interactive callback from TextEditor. See commentary in Model class."<br>> + <br>> +       ^ (Project uiManager<br>> +            saveFilenameRequest: 'Save text contents in file...'<br>> +            initialAnswer: 'Transcript.text')<br>> +                       ifEmpty: [false]<br>> +                        ifNotEmpty: [:fileName |<br>> +                                Model new<br>> +                                       saveContents: stringContents<br>> +                                    onFileNamed: fileName<br>> +                                   accessMode: accessMode].!<br><br></div></blockquote></div>