Column names with spaces about django-import-export HOT 5 CLOSED

Hi, it looks like we should use attribute_name instead of column_name. attribute_name:

method = getattr(self, 'dehydrate_%s' % field.column_name, None)

I have to test this, but would appreciate if you can test that this work for your case.

regards

from django-import-export.

That was my initial thought, too, but attribute_name might be blank, and that led me down the road of testing for a value. Given that I wasn't too familiar with the internals of import_export, the replace seemed an easier fix at the time.

Also, the use case for me is generating an export report (of sorts) for non-technical users, and being able to specify the column name in a very verbose way (i.e. more descriptive than what I name my attributes) is a requirement.

from django-import-export.

@pwr905 I am almost sure that attribute_name should be used in dehydratebut I will have catch some time to dig in code and test it does not break anything.

from django-import-export.

In https://github.com/bmihelac/django-import-export/compare/fix-36-use-field-name-for-dehydration branch is a change that uses field_name for dehydrating fields.

field_name is assigned when Resource is declared.

Example of use is in tests:

https://github.com/bmihelac/django-import-export/blob/d73c05641eafe52d7904ec0c53737ce195795a00/tests/core/tests/resources_tests.py#L181

Does this solve issue stated above?

from django-import-export.

Sorry for the delay, been really busy. Will test it out in the next few days and let you know.

from django-import-export.