various fixes

[myslice.git] / manifold / metadata.py

diff --git a/manifold/metadata.py b/manifold/metadata.py
index 3475139..1d8d5e8 100644 (file)
--- a/manifold/metadata.py
+++ b/manifold/metadata.py
@@ -1,7 +1,11 @@
 import json 
 
+from manifold.manifoldresult import ManifoldResult
 from manifold.manifoldapi import ManifoldAPI
 
+debug=False
+debug=True
+
 # turn this on if you want to work offline
 work_offline=False
 #work_offline=True
@@ -10,14 +14,17 @@ class MetaData:
 
     def __init__ (self, auth):
         self.auth=auth
-        self.hash_by_subject={}
+        self.hash_by_object={}
+
+        # XXX Retrieve all metadata the first time we instanciate the class
+        self.fetch()
 
     def fetch (self):
-        offline_filename="offline_metadata.json"
+        offline_filename="offline-metadata.json"
         if work_offline:
             try:
                 with file(offline_metadata) as f:
-                    self.hash_by_subject=json.loads(f.read())
+                    self.hash_by_object=json.loads(f.read())
                 return
             except:
                 print "metadata.work_offline: failed to decode %s"%offline_filename
@@ -28,18 +35,26 @@ class MetaData:
                   'column.resource_type', 'column.value_type',
                   'column.allowed_values', 'column.platforms.platform',
                   'column.platforms.platform_url']
-        results = manifold_api.Get('metadata:table', [], [], fields)
-        self.hash_by_subject = dict ( [ (result['table'], result) for result in results ] )
+        rows_result = manifold_api.Get({
+            'object': 'local:objects', # proposed to replace metadata:table
+            'fields':     fields 
+        })
+#old#        rows_result = manifold_api.Get('metadata:table', [], [], fields)
+        rows = rows_result.ok_value()
+        if not rows:
+            print "Failed to retrieve metadata",rows_result.error()
+            rows=[]
+        self.hash_by_object = dict ( [ (row['table'], row) for row in rows ] )
         # save for next time we use offline mode
-        with file(offline_filename,'w') as f:
-            f.write(json.dumps(self.hash_by_subject))
+        if debug:
+            with file(offline_filename,'w') as f:
+                f.write(json.dumps(self.hash_by_object))
 
     def to_json(self):
-        return json.dumps(self.hash_by_subject)
-
-    def details_by_subject (self, subject):
-        return self.hash_by_subject[subject]
+        return json.dumps(self.hash_by_object)
 
-    def sorted_fields_by_subject (self, subject):
-        return self.hash_by_subject[subject]['columns'].sort()
+    def details_by_object (self, object):
+        return self.hash_by_object[object]
 
+    def sorted_fields_by_object (self, object):
+        return self.hash_by_object[object]['columns'].sort()