Issue #1808 has been updated by luke.

Status changed from Accepted to Closed

This is now fixed, as of commit:"0cf9decfeab5a45f8457af2e51633fd6e0f877fc".
----------------------------------------
Refactor #1808: There should be a common Resource class
http://projects.reductivelabs.com/issues/show/1808

Author: luke
Status: Closed
Priority: High
Assigned to: luke
Category: plumbing
Target version: 0.25.0
Affected version: 0.24.6


This speaks toward long-term goals of refactoring the RAL and more, but as a 
short-term goal, it would provide a common class for at least two classes:

* Parser resources
* Transportable resources

These are currently handled by Puppet::Parser::Resource and 
Puppet::TransObject, respectively.  The TransObject class is one of the 
earliest classes in Puppet and has long been in need of a refactor.  The 
Parser::Resource class duplicates some significant amount of TransObject's 
functionality, but it adds a lot of its own functionality.

Further, in the future we'd like to move RAL resources to use a simpler 
Resource class that could then be associated with the RAL types, rather than 
being instances of them.  We would plan initial work focusing on the parsing 
and transportation, and leave the RAL code for later.

This makes sense to do as part of the catalog/REST conversion because 
TransObjects are currently what pass over the wire, and we need something 
maintainable in the long term.  So, the first stage in this ticket is to 
replace TransObject with a new Puppet::Resource class (retaining TransObject 
for use when xmlrpc is still used, for backward compatibility).  The second 
stage is to convert Puppet::Parser::Resource into a wrapper around the 
Puppet::Resource class (yes, a wrapper, not a subclass -- that way the catalog 
of resources doesn't need any conversion before transportation or whatever else 
is done).


----------------------------------------
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://reductivelabs.com/redmine/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
-~----------~----~----~----~------~----~------~--~---

Reply via email to