Produits Entr'ouvert » Passerelle

On peut aussi s'arranger pour ne pas conserver l'ancien fichier sur une mise à jour, ça évitera dans un premier temps de créer tant de fichiers orphelins.

Benjamin Dauvergne a écrit :

On peut aussi s'arranger pour ne pas conserver l'ancien fichier sur une mise à jour, ça évitera dans un premier temps de créer tant de fichiers orphelins.

Yep, j'ai pensé à ça (à faire lors du save()) mais je me disais que c'était assez "agréable" de conserver les quelques dernières versions... une sorte de backup du pauvre. Et comme de toute façon il va falloir écrire le code pour nettoyer les /media/csv actuels...

C'est bizarre je ne vois pas de migration AlterField pour la modification à upload_to.

--

Le tenant s'obtient via tenant = getattr(django.db.connection, 'tenant', None) mais en fait ça ne me parait pas nécessaire, le storage attaché au field devrait tout seul trouver le bon répertoire (ils sont tenant "aware") donc un simple accès à instance.csv_file.path donnera le bon chemin, et pour trouver le répertoire de base model._meta.get_field('csv_file').storage.path('').

• modifier tous les upload_to sans réfléchir (1 migration)
• repasser sur tous les champs csv_file et faire

  old_name = instance.csv_file.name
  old_path = instance.csv_file.path
  if not old_name.startswith('csv/'):
      continue
  filename = os.path.basename(old_name)
  instance.csv_file.save(filename, instance.csv_file)
  unlink(old_path)
  

  
  car upload_to n'est utilisé qu'à la première sauvegarde pour générer name, ensuite il respecte name absolument.

PS: édité.

Non, lors d'une migration on a un FakeTenant sur connection.tenant, qui ne contient que schema_name, et ne nous permet pas de retrouver le path jusqu'au dossier media, d'où ma tambouille.
Si on essaie d'accéder à csv_file.path lors d'une migration, on a une erreur ici:

class TenantFileSystemStorage(FileSystemStorage, TenantStorageMixin):
    '''Lookup files first in $TENANT_BASE/<tenant.schema>/media/ then in default location'''
    def path(self, name):
        if connection.tenant:
            location = safe_join(settings.TENANT_BASE, connection.tenant.domain_url, 'media')  # <= ici
        else:
            location = self.location
        try:
            path = safe_join(location, name)
        except ValueError:
            raise SuspiciousOperation("Attempted access to '%s' denied." % name)
        return os.path.normpath(path)

Car sur un FakeTenant il n'y a pas de domain_url
Le storage fonctionne mal sur une migration

Les AlterField sont dans les migrations 0019 (csvdatasource) et 0004 (nancypoll)

Lauréline Guerin a écrit :

Non, lors d'une migration on a un FakeTenant sur connection.tenant, qui ne contient que schema_name, et ne nous permet pas de retrouver le path jusqu'au dossier media, d'où ma tambouille.

Mouais c'est tout pourri, il faudrait d'abord corriger ça.

Lauréline Guerin a écrit :

(ne fonctionnera qu'avec la modif de #48071)

J'ai un doute: st_ctime ou st_mtime ?
On utilise déjà st_ctime dans le code pour afficher la date d'ajout du fichier, mais en local la commande de nettoyage fonctionne mieux (avec les fichiers que j'avais déjà dans mon rep media) avec st_mtime (st_mtime semble donner la même date qu'un ls -l en shell)

Je dirai mtime, c'est ce que touch modifie si jamais on veut conserver un fichier plus longtemps (dans les faits ça n'a pas beaucoup d'importance, juste que sur un unlink/link/chmod/chown ça va faire ctime <- now(), est-ce qu'on veut vraiment que ça prolonge la vie du fichier ?).