added request for metadata calls to be able to handle errors in messages

[myslice.git] / manifold / metadata.py

diff --git a/manifold/metadata.py b/manifold/metadata.py
index 1b0e270..40a88ff 100644 (file)
--- a/manifold/metadata.py
+++ b/manifold/metadata.py
@@ -1,8 +1,11 @@
 import json 
+import os.path
 
 from manifold.manifoldresult import ManifoldResult
 from manifold.manifoldapi import ManifoldAPI
 
+from django.contrib                     import messages
+
 debug=False
 debug=True
 
@@ -16,8 +19,8 @@ class MetaData:
         self.auth=auth
         self.hash_by_object={}
 
-    def fetch (self):
-        offline_filename="offline-metadata.json"
+    def fetch (self, request):
+        offline_filename="%s/../offline-metadata.json"%os.path.dirname(__file__)
         if work_offline:
             try:
                 with file(offline_metadata) as f:
@@ -26,24 +29,26 @@ class MetaData:
             except:
                 print "metadata.work_offline: failed to decode %s"%offline_filename
         manifold_api = ManifoldAPI(self.auth)
-        fields = ['table', 'column.column',
-                  'column.description','column.header', 'column.title',
-                  'column.unit', 'column.info_type',
-                  'column.resource_type', 'column.value_type',
-                  'column.allowed_values', 'column.platforms.platform',
-                  'column.platforms.platform_url']
-        rows_result = manifold_api.Get({
+        fields = ['table', 'column.name', 'column.qualifier', 'column.type', 'column.is_array', 'column.description', 'column.default', 'key', 'capability']
+        #fields = ['table', 'column.column',
+        #          'column.description','column.header', 'column.title',
+        #          'column.unit', 'column.info_type',
+        #          'column.resource_type', 'column.value_type',
+        #          'column.allowed_values', 'column.platforms.platform',
+        #          'column.platforms.platform_url']
+        result = manifold_api.forward({
+            'action': 'get',
             'object': 'local:object', # proposed to replace metadata:table
             'fields':     fields 
         })
 
-        if row_results['code'] == 1: # warning
+        if result['code'] == 1: # warning
             messages.warning(request, result['description'])
-        elif row_results['code'] == 2:
+        elif result['code'] == 2:
             messages.error(request, result['description'])
             # XXX FAIL HERE XXX
 
-        rows = rows_result.ok_value()
+        rows = result.ok_value()
 # API errors will be handled by the outer logic
 #        if not rows:
 #            print "Failed to retrieve metadata",rows_result.error()
@@ -62,3 +67,7 @@ class MetaData:
 
     def sorted_fields_by_object (self, object):
         return self.hash_by_object[object]['column'].sort()
+
+    def get_field_type(self, object, field):
+        print "Temp fix for metadata::get_field_type() -> consider moving to manifold.core.metadata soon"
+        return field