<p>Aha, now I think I know why <a href="https://github.com/eht16" class="user-mention">@eht16</a> did it - just noticed that this signal is also used in the filebrowser plugin. When "use the project's base directory" is enabled, the "project-save" signal is used to update the directory in the file browser (this signal is probably used so it gets updated also when base path changes in project's config). But you don't want to change the directory when closing the project.</p>
<p>One way to fix it with the new behavior is to have a flag that is set in "project-before-close" and reset in "project-close" handler to indicate whether project closing happens.</p>
<p>But maybe it would be even better to simply remove the "project-save" handler and do the directory change only in "project-open" - I'm not sure if it's a good idea to change the directory when project settings gets updated (imagine you changed the directory to some subdirectory of the project because you work there and just want to update a command-line flag of the compiler in settings - and BAM, your directory is changed).</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/1400#issuecomment-281097024">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ9twUNlIKqQvJJYDADEro_3dmKNHks5reabTgaJpZM4MFGGL">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ3IjETofYvWd70WIQ1uOrnoQaRKcks5reabTgaJpZM4MFGGL.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/1400#issuecomment-281097024"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@techee in #1400: Aha, now I think I know why @eht16 did it - just noticed that this signal is also used in the filebrowser plugin. When \"use the project's base directory\" is enabled, the \"project-save\" signal is used to update the directory in the file browser (this signal is probably used so it gets updated also when base path changes in project's config). But you don't want to change the directory when closing the project.\r\n\r\nOne way to fix it with the new behavior is to have a flag that is set in \"project-before-close\" and reset in \"project-close\" handler to indicate whether project closing happens.\r\n\r\nBut maybe it would be even better to simply remove the \"project-save\" handler and do the directory change only in \"project-open\" - I'm not sure if it's a good idea to change the directory when project settings gets updated (imagine you changed the directory to some subdirectory of the project because you work there and just want to update a command-line flag of the compiler in settings - and BAM, your directory is changed)."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1400#issuecomment-281097024"}}}</script>