{"comment":{"url":"https://api.github.com/repos/AOEpeople/Aoe_Scheduler/comments/10134970","html_url":"https://github.com/AOEpeople/Aoe_Scheduler/commit/2a690deff17bf3b902da92a7effa7820110a2fe2#commitcomment-10134970","id":10134970,"user":{"login":"fbrnc","id":468820,"avatar_url":"https://avatars.githubusercontent.com/u/468820?v=3","gravatar_id":"","url":"https://api.github.com/users/fbrnc","html_url":"https://github.com/fbrnc","followers_url":"https://api.github.com/users/fbrnc/followers","following_url":"https://api.github.com/users/fbrnc/following{/other_user}","gists_url":"https://api.github.com/users/fbrnc/gists{/gist_id}","starred_url":"https://api.github.com/users/fbrnc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fbrnc/subscriptions","organizations_url":"https://api.github.com/users/fbrnc/orgs","repos_url":"https://api.github.com/users/fbrnc/repos","events_url":"https://api.github.com/users/fbrnc/events{/privacy}","received_events_url":"https://api.github.com/users/fbrnc/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"2a690deff17bf3b902da92a7effa7820110a2fe2","created_at":"2015-03-11T04:16:48Z","updated_at":"2015-03-11T04:16:48Z","body":"Note (to myself): Looks like a smart thing to do since there's no reason to keep track of the $exists array outside of this method. Also the $exists variable wasn't passed as reference anyway so it didn't solve the purpose it was probably introduced in the first place (this is a leftover from Mage_Cron). Only draw back is the now multiple db queries will be made which is probably neglect-able for the low number of schedules."}}