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 internal representation #23

Open
iemejia opened this issue Feb 27, 2013 · 3 comments
Open

Refactor internal representation #23

iemejia opened this issue Feb 27, 2013 · 3 comments
Assignees
@iemejia
Labels
FIX

Comments

@iemejia
Copy link
Owner

iemejia commented Feb 27, 2013

Remove the global catalogs variable and put it in an object
Create class for the DAO operations (maybe the same)
@iemejia
Copy link
Owner Author

iemejia commented Feb 28, 2013

The idea now is to have one class DAO like for the DB operations
And a second one for the concrete filesystem operations
Refactor must pass the test suite (and rewrite the tests if necessary)

@ghost ghost assigned rgamez Feb 28, 2013
@iemejia
Copy link
Owner Author

iemejia commented Feb 28, 2013

I have just pushed a commit of a refactor that vastly helps to simplify this.

@ghost ghost assigned iemejia Mar 4, 2013
@iemejia
Copy link
Owner Author

iemejia commented Mar 4, 2013

We have to discuss how to provide this as a reusable API for other services (e.g. a webpage)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@iemejia iemejia

Labels
FIX
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rgamez @iemejia