Sauvegarde du brouillon d'un nouveau sujet

[Rhidian (Rhidian Bramley)]

Priorité : Faible

Plateforme : Discourse mobile iOS (non testé sur d’autres plateformes). Reproduit sur Discourse meta et try.Discourse.org

Description : Sauvegarder un brouillon de Nouveau Sujet lorsqu’il est demandé. Le bouton Ouvrir le brouillon ne fonctionne pas.

Étapes de reproduction :

1. Accédez à la page d’accueil de Discourse.
2. Cliquez sur + Nouveau Sujet.
3. Saisissez l’objet « Test des brouillons ». Saisissez « Test » dans le corps du compositeur. Laissez-le sans catégorie.
4. Cliquez sur Annuler (icône de corbeille sur Discourse mobile). On vous demande si vous souhaitez Sauvegarder un brouillon pour plus tard — cliquez sur cette option.
5. Le bouton + Nouveau Sujet change en + Ouvrir le brouillon.
6. Cliquez sur + Ouvrir le brouillon. Rien ne se produit.
7. Vérifiez le dossier Brouillons — aucun brouillon n’est enregistré.

Conséquence : Cela m’a temporairement empêché de créer un nouveau Sujet.

Contournement : Cliquer à plusieurs reprises sur le bouton inactif + Ouvrir le brouillon finit par faire planter le site Discourse dans l’application mobile et retourne à la sélection de domaine. Retournez sur Discourse — le bouton + Ouvrir le brouillon a été effacé et est redevenu + Nouveau Sujet.

[jean.perez (Jean)]

Le titre n’est pas assez long pour passer cette validation
https://github.com/discourse/discourse/blob/master/app/assets/javascripts/discourse/app/models/composer.js#L1184
Mais je me demande pourquoi le code résout la promesse au lieu de la rejeter, puis de capturer les erreurs.

Je pense que les erreurs peuvent être capturées ici :

https://github.com/discourse/discourse/blob/master/app/assets/javascripts/discourse/app/controllers/composer.js#L1244

quelque chose dans ce genre :

this._saveDraftPromise = model.saveDraft().then(() => {
    this._lastDraftSaved = Date.now();
    ...
}, error => {
    // gestion des erreurs ici
}).finally(() => {
    this._saveDraftPromise = null;
});

@eviltrout, qu’en penses-tu ?

[eviltrout (Robin Ward)]

Ce code d’enregistrement fait malheureusement partie de nos plus anciens, et souvent de nos plus fragiles. Au fil des ans, nous y avons ajouté de nombreuses fonctionnalités, et certaines choses comme l’enregistrement de brouillons, qui peuvent réussir ou échouer, nécessitent leur propre logique personnalisée, ce qui ne fait qu’ajouter encore plus de complexité.

Je pense que votre intuition est probablement juste ici, mais il m’est impossible de le confirmer sans voir comment s’exécutent tous nos tests. Si vous avez le temps de proposer une pull request à l’avenir, ce serait formidable.

[SonsOfAhriman (Ryan Huddleston)]

J’ai pu reproduire ce bug sur Chrome sous MacOS sur Discourse meta en suivant les étapes de Rhidian ci-dessous :

@Rhidian

Étapes pour reproduire le bug :

1. Accédez à la page d’accueil de Discourse.
2. Cliquez sur + Nouveau sujet.
3. Saisissez « Testing Drafts » dans la ligne d’objet et « Testing » dans le corps du compositeur. Laissez-le sans catégorie.
4. Cliquez sur Annuler (icône de corbeille sur Discourse mobile). On vous demande si vous souhaitez enregistrer un brouillon pour plus tard — choisissez cette option.
5. Le bouton + Nouveau sujet se transforme en + Ouvrir un brouillon.
6. Cliquez sur + Ouvrir un brouillon. Rien ne se passe.
7. Vérifiez le dossier Brouillons — aucun brouillon n’est enregistré.

[nbianca (Bianca)]

Bonjour,

Merci pour votre rapport de bug. Il a été corrigé dans la dernière version de Discourse. Le problème venait du fait que l’outil proposait d’enregistrer des brouillons très courts, même lorsque cela ne devrait pas se produire.

[nbianca (Bianca) A fermé ce sujet ()]

Ce sujet a été automatiquement fermé après 3 jours. Les nouvelles réponses ne sont plus autorisées.