[#67] Fix rollup with titles

Author: Bjwebb

flattentool/json_input.py

@@ -140,7 +140,12 @@ def parse_json_dict(self, json_dict, sheet, json_key=None, parent_name='', flatt
                     if self.rollup and parent_name == '': # Rollup only currently possible to main sheet
                         if len(value) == 1:
                             for k, v in value[0].items():
-                                if parent_name+key+'/0/'+k in self.schema_parser.main_sheet:
+                                if self.use_titles and parent_name+key+'/0/'+k in self.schema_parser.main_sheet.titles:
+                                    if type(v) in BASIC_TYPES:
+                                        flattened_dict[sheet_key_title(sheet, parent_name+key+'/0/'+k)] = v
+                                    else:
+                                        raise ValueError('Rolled up values must be basic types')
+                                elif not self.use_titles and parent_name+key+'/0/'+k in self.schema_parser.main_sheet:
                                     if type(v) in BASIC_TYPES:
                                         flattened_dict[sheet_key(sheet, parent_name+key+'/0/'+k)] = v
                                     else:

flattentool/tests/test_input_SpreadsheetInput_unflatten.py

@@ -590,19 +590,15 @@ def create_schema(root_id):
                     }
                 }
             },
-            'testR': {
-                'title': 'R title',
+            'testArr': {
+                'title': 'Arr title',
                 'type': 'array',
-                'rollUp': ['id', 'testB'],
                 'items': {
                     'type': 'object',
                     'properties': {
                         'id': {
                             'title': 'Identifier',
                             'type': 'string',
-                            # 'type': 'integer',
-                            # integer does not work, as testB:integer is not
-                            # in the rollUp
                         },
                         'testB': {
                             'title': 'B title',
@@ -612,13 +608,6 @@ def create_schema(root_id):
                             'title': 'C title',
                             'type': 'string',
                         },
-                        'testSA': {
-                            'title': 'SA title',
-                            'type': 'array',
-                            'items': {
-                                'type': 'string'
-                            }
-                        },
                         'testNest': {
                             'title': 'Nest title',
                             'type': 'array',
@@ -653,6 +642,38 @@ def create_schema(root_id):
                     }
                 }
             },
+            'testR': {
+                'title': 'R title',
+                'type': 'array',
+                'rollUp': ['id', 'testB'],
+                'items': {
+                    'type': 'object',
+                    'properties': {
+                        'id': {
+                            'title': 'Identifier',
+                            'type': 'string',
+                            # 'type': 'integer',
+                            # integer does not work, as testB:integer is not
+                            # in the rollUp
+                        },
+                        'testB': {
+                            'title': 'B title',
+                            'type': 'string',
+                        },
+                        'testC': {
+                            'title': 'C title',
+                            'type': 'string',
+                        },
+                        'testSA': {
+                            'title': 'SA title',
+                            'type': 'array',
+                            'items': {
+                                'type': 'string'
+                            }
+                        },
+                    }
+                }
+            },
             'testU': {
                 'title': UNICODE_TEST_STRING,
                 'type': 'string',