Search code examples
pythondjangoforeign-keysdjango-import-export

Dealing with import of foreignKeys in django-import-export


I don't understand how django-import-export module deals with ForeignKeys. Here is a simple exemple : models.py

class TFamilies(models.Model):
    id_fam = models.AutoField(primary_key=True, unique=True)
    name_fam = models.CharField(max_length=1024, blank=True,verbose_name='Famille')


class TGenus(models.Model):
    id_genus = models.AutoField(primary_key=True, unique=True)
    name_genus = models.CharField(max_length=1024,verbose_name='nom de genre')
    id_fam = models.ForeignKey(TFamilies, null=True, db_column='id_fam', blank=True, verbose_name='Famille')

I would like to allow people adding genus with family associated ! A CSV/XLS with only name_genus and name_fam... (and id left blank).

Family already exist in DB most of the time, Django juste have to find the right id number...

admin.py

class TGenusResource(resources.ModelResource):

    name_fam = fields.Field(widget=widgets.ForeignKeyWidget(TFamilies, 'name_fam'))

    class Meta:
        model = TGenus
        import_id_fields = ['id_genus']


class TGenusAdmin(ImportExportActionModelAdmin):
    form = TGenusAdminForm
    resource_class = TGenusResource
    pass

This configuration lead to error in import interface :

Line number: 1 - 'NoneType' object has no attribute 'name_fam'
Traceback (most recent call last):
File "/....../lib/python2.7/site-packages/import_export/resources.py", line 348, in import_data
row_result.object_repr = force_text(instance)
File "......./lib/python2.7/site-packages/django/utils/encoding.py", line 85, in force_text
s = six.text_type(s)
AttributeError: 'NoneType' object has no attribute 'name_fam'

I don't understand... I also tried answer there : django-import-export resource definition for foreignkey field? and sort of like there : Foreign Key in django migration using django-import-export

Do I have to use before_import to find myself the match ?


Solution

  • I figured out that you have to find yourself the match ! It's impossible to alterate values in a tablib dataset so you have to take entries make changes and put them back in a new line then erase the old one.

    My excel template contains columns id_genus (empty), name_genus and id_fam filled by the name of the family !

    For anyone who landed here I post my way :

    def before_import(self, dataset, dry_run):
            """
            Make standard corrections to the dataset before displaying to user
            """
    
            i = 0
            last = dataset.height - 1
    
            # for all lines, search for id of family given and add a new line at the bottom with it then delete the first one
            while i <= last:
                # Check if the Genus exist in DB
                if (TGenus.objects.filter(name_genus=dataset.get_col(2)[0].capitalize())):
                    id_genus = TGenus.objects.filter(name_genus=dataset.get_col(2)[0].capitalize())[0].id_genus
                else :
                    id_genus = ''
                # Check if the family exists in DB
                try:
                    TFamilies.objects.get(name_fam=dataset.get_col(2)[0].capitalize())
                except TFamilies.DoesNotExist:
                    raise Exception("Family not in DB !")                   
                except TFamilies.MultipleObjectsReturned:
                    pass
    
                # use of "filter" instead of "get" to prevent duplicate values, select the first one in all cases
                dataset.rpush((id_genus,
                    dataset.get_col(1)[0],
                    TFamilies.objects.filter(name_fam=dataset.get_col(2)[0].capitalize())[0].id_fam))
                dataset.lpop()
                i = i + 1
    

    My django admin can be used by non sys-admin so, they can and duplicate genus or families that aren't in DB... If someone have an idea to deal with errors better, I would like to read it ! Also, I would like to keep the name of the family in the preview, not only her id... If you know how, I have posted another question about that : Is-it possible to customize the template of preview in django import-export?