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.

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

convert_cloudformation_types breaks nested dictionaries #86

Closed
DaveWK opened this issue Sep 7, 2016 · 0 comments
Closed

convert_cloudformation_types breaks nested dictionaries #86

DaveWK opened this issue Sep 7, 2016 · 0 comments

Comments

@DaveWK
Copy link
Contributor

DaveWK commented Sep 7, 2016

Hi,

I noticed that after updating to the latest gordon, I no longer have .context files which include nested elements in my common.yml

in example, for my common.yml:

parent:
  child1: 60
  child2: 60
  child3: 60

used to give me an array like:
parent[child1]
parent[child2]
parent[child3]

where all the values would be 60.

Now, I only get:
parent[0] = child1
parent[1] = child2
parent[2] = child3

and my value of 60 is nowhere to be found.

After some digging I discovered that the offending change is the convert_cloudformation_types in utils.py which wraps the load_settings call on line 470 of core.py

I wasn't sure what syntax rules this was attempting to adjust for, so I just added a simple
return data
on line 187 of util.py to bypass it and get myself back to a working state.

jorgebastida pushed a commit that referenced this issue Apr 6, 2017
Adding in sanitization for ints in iterable objects. Fix for issue #86
@DaveWK DaveWK closed this as completed May 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant