https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292016-08-12T15:15:52ZRedmineRedmine - Patch #23519: Don't preload projects and roles on Principal#memberships associationhttps://www.redmine.org/issues/23519?journal_id=726672016-08-12T15:15:52ZOndřej Ezr
<ul><li><strong>File</strong> <a href="/attachments/16505">memberships_performance.diff</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/16505/memberships_performance.diff">memberships_performance.diff</a> added</li></ul><p>No reason for preload in membership(...)</p> Redmine - Patch #23519: Don't preload projects and roles on Principal#memberships associationhttps://www.redmine.org/issues/23519?journal_id=730092016-08-27T04:32:07ZToshi MARUYAMA
<ul><li><strong>Target version</strong> set to <i>3.4.0</i></li></ul> Redmine - Patch #23519: Don't preload projects and roles on Principal#memberships associationhttps://www.redmine.org/issues/23519?journal_id=757172017-01-07T11:40:16ZJean-Philippe Langjp_lang@yahoo.fr
<ul><li><strong>Subject</strong> changed from <i>Performance for memberships</i> to <i>Don't preload projects and roles on Principal#memberships association</i></li><li><strong>Status</strong> changed from <i>New</i> to <i>Closed</i></li><li><strong>Assignee</strong> set to <i>Jean-Philippe Lang</i></li></ul><p>Patch committed, thanks. FTR, the proposed patch was not working as expected and broke a few tests, <code>#roles</code> has to be called explicitly (using <code>self.roles</code>) due to a variable naming clash in <code>#allowed_to?</code>.</p>