Issue Details (XML | Word | Printable)

Key: CHEF-1341
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Blocker Blocker
Assignee: Unassigned
Reporter: Daniel DeLeo
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Chef

rewrite the handler API

Created: 16/Jun/10 05:42 PM   Updated: 16/Jun/10 06:13 PM   Resolved: 16/Jun/10 06:13 PM
Component/s: Chef Client
Affects Version/s: 0.9.0
Fix Version/s: 0.9.0

Triage Status: Triaged


 Description  « Hide

The handler API requires users to implement a method with six arguments to receive run status data. This is bad because it is inflexible, we could never add new data to the handler without forcing everyone to rewrite all of their existing ones (otherwise they'd get errors like, "wrong number of arguments 7 for 6"). This also forces complexity on the user, when it can be elegantly dealt with internally in chef instead. Also, the JSON File handler uses methods that don't exist, so it doesn't work, and some implementation that is specific to the JSON file handler is in the base handler class.

If we fix all of these right now, we can have an extensible API that will be more amenable to adding features at a later time without causing the trauma of forcing people to rewrite a bunch of code.



Sort Order: Ascending order - Click to sort in descending order