Issue #5079 has been updated by Matt Robinson. Status changed from Investigating to Rejected
After some discussion on the mailing list with Sandor we're rejecting this refactor since it doesn't add any functionality. Sandor will include this refactor in a future patch if he finds something he'd like to add or change that builds on the refactor. ---------------------------------------- Refactor #5079: MCX content provider https://projects.puppetlabs.com/issues/5079 Author: Sandor Szücs Status: Rejected Priority: Normal Assignee: Nigel Kersten Category: OSX Target version: Affected Puppet version: Branch: I refactored the MCX content provider to enhance code readability. This is the first step to understand the current implementation. I would like to add a best practice dslocal solution pointed out by the authors of "Enterprise Mac Managed Preferences". They wrote that you should not use /Local/Default as DB, because it is also used as local cache. They further said that you should add an own DB to circumvent ping-pong overwrites between central managed MCX and dslocal managed MCX. The patch includes a small change within the spec file, because the error handling should be more conform to other puppet providers. Instead to raise MCXContentProviderException I choose to raise Puppet::Error, because macauthorization provider does the same. I hope that is the right Error handling. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
