← Back to team overview

zeitgeist team mailing list archive

[Bug 407369] Re: Define content and source types in one central place

 

** Changed in: zeitgeist
       Status: New => Fix Committed

** Changed in: zeitgeist/0.3
       Status: New => Fix Committed

-- 
Define content and source types in one central place
https://bugs.launchpad.net/bugs/407369
You received this bug notification because you are a member of
Zeitgeist-Engine, which is the registrant for Zeitgeist Engine.

Status in Zeitgeist Engine: Fix Committed
Status in Zeitgeist Engine 0.3 series: Fix Committed

Bug description:
currently there are two places where content and source objects are defined
 * zeitgeist/datamodel.py (definition and URI abstraction)
 * _zeitgeist/engine/base.py (mapping, lookup, and caching content/source URI -> database ID)

We should change this to
 * have the URI (etc.) definition in one central place
 * make it easily possible to added new content and source types
 * allow easy lookup in the database implementation
 * there should still be a datamodel module in the public API, which knows about all definitions but nothing about the database



References