Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor DefinedContent class #2

Open
DaveGreasley opened this issue Apr 5, 2015 · 0 comments
Open

Refactor DefinedContent class #2

DaveGreasley opened this issue Apr 5, 2015 · 0 comments
Assignees
@DaveGreasley
Labels
enhancement
Milestone
Second Release

Comments

@DaveGreasley
Copy link
Collaborator

The main DefinedContent class is now pretty much fully functional but should really be split out as it has far too many responsibilities and is quite complicated.

Suggested classes:

  • DefindContent - Keeps singleton and static methods and a reference to a cache object.
  • DefinedContentCache - The cache, responsible for building it's self
  • DefinedContentCreator - Responsible for creating new items
  • DefinedContentItemSerializer - Responsible for serializing / de-serializing DefinedContentItems to XML
@DaveGreasley DaveGreasley self-assigned this Apr 5, 2015
@DaveGreasley DaveGreasley added this to the Second Release milestone Apr 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DaveGreasley DaveGreasley

Labels
enhancement
Projects
None yet
Milestone
Second Release
Development

No branches or pull requests
1 participant
@DaveGreasley