Copy to json manually #6061
Copy to json manually #6061
Conversation
TypeError: cannot deepcopy this pattern object
| if ds.get_id == config_id: | ||
| return ds | ||
| raise BadSpecError(_('The data source referenced by this report could ' | ||
| 'not be found.')) |
There was a problem hiding this comment.
this should fail hard, if I understand its usage correctly.
|
This looks good to go from my perspective. Sorry for the jsonobject bug! |
|
@esoergel sorry i wasn't more available today on this, but I don't think this is the right fix. I think the right thing to do is just not call the two failing lines if it is a |
|
I've got another branch implementing Cory's suggestion, which seems less impactful, so I'll close this one. (I can't push to github due to the DDoS at this time though) |
wait on build
http://manage.dimagi.com/default.asp?162940
@NoahCarnahan @czue @dannyroberts
It errors on these two lines
The error we were originally getting was because the corresponding config wasn't found, but I think that was a transient issue. When I try to run the task manually, I get a
TypeError: cannot deepcopy this pattern objecton a save attempt. This appears to be a standing issue with jsonobject, as Danny pointed out. Until a fix for that gets rolled out, this does at least make the config save properly, which lets the task run.