Bogue de l'application Express.js: la méthode validationResult (req) ne fonctionne pas

Peut-être devriez-vous commencer par ce https://express-validator.github.io/docs/ puis personnalisez-le progressivement en fonction de vos besoins afin de détecter toute erreur en cours de route.

Vous effectuez le rendu d'un modèle, vous essayez d'afficher Flash, puis de rediriger. Changez-le en

exports.addPost = (req, res, next) => {
   // Form validation rules
    req.check('title').not().isEmpty().withMessage("The title field is mandatory");
    req.check('body').not().isEmpty().withMessage("The full text field is mandatory");

   const errors = req.validationErrors();

Oubliez le rendu ... Cela n'a aucun sens pour vous de l'avoir là-bas. Ce que fait le rendu, il rend et renvoie un modèle. Par conséquent, votre req.flash et votre redirection ne se produisent jamais ou cela se produit après que l'en-tête a déjà été envoyé.

définition de res.render ():

Rend une vue et envoie la chaîne HTML rendue au client. Paramètres facultatifs:

locals, un objet dont les propriétés définissent des variables locales pour le vue. callback, une fonction de rappel. S'il est fourni, la méthode retourne à la fois l'erreur possible et la chaîne rendue, mais n'effectue pas réponse automatisée. Lorsqu'une erreur se produit, la méthode invoque next (err) en interne.

AND

req.flash('danger', errors);
req.session.save(() => res.redirect('/dashboard'));

essayez de changer votre instruction if de ceci:

if (!errors.isEmpty()) {
           return res.send(errors.array());
         // can also send an status and catch it
        // by sending res.status(400).send(errors.array());
        }

à ceci:

    exports.addPost = (req, res, next) => {
        // Form validation rules
        check('title', '<your error message>')
          .not()
          .isEmpty();
        check('excerpt', '<your error message>')
          .not()
          .isEmpty();
       check('body', '<your error message>')
          .not()
          .isEmpty();

        const errors = validationResult(req);

        const errors = validationResult(req);
        if (!errors.isEmpty()) {
           console.log(errors.array());
        }
}

Modifier

Si vous souhaitez envoyer une réponse à votre front-end, remplacez la commande console.log () dans res.send () puis analysez la réponse dans votre front-end comme ça:

   if (!errors.isEmpty()) {
        console.log('there are no validation errors');
    } else {
        console.log(errors);
    }
}

J'espère que cela a du sens

d'après ce que je vois dans la documentation de express-validator vous devez fournir un tableau de règles de validation (ces vérifications en haut de votre contrôleur) lorsque vous définissez l'itinéraire. Cela n'a pas beaucoup de sens pour eux d'être en haut du gestionnaire de requêtes car le validateur express ne pourra pas accéder au contexte qui fournit la requête à valider.

Donc, dans le routeur, vous avez besoin de quelque chose comme ceci:

router / front-end / posts.js

exports.addPost = (req, res, next) => {

        const errors = validationResult(req);

        if (!errors.isEmpty()) {
            console.log(errors.array());
     }

        if (!errors.isEmpty()) {
            res.render('admin/addpost', {
            layout: 'admin/layout',
             website_name: 'MEAN Blog',
             page_heading: 'Dashboard',
             page_subheading: 'Add New Post',
             errors: errors
            });
            req.flash('danger', errors);
            req.session.save(() => res.redirect('/dashboard'));
        } else {
                const post = new Post();
                    post.title = req.body.title;
                    post.short_description = req.body.excerpt
                    post.full_text = req.body.body;

                post.save(function(err){
                       if(err){
                          console.log(err);
                          return;
                        } else {
                          req.flash('success', "The post was successfully added");
                          req.session.save(() => res.redirect('/dashboard'));
                        }
                });
        }
}

controllers / front-end / posts.js

const validationRules = [// Form validation rules
        check('title', 'The title field id required')
        .not()
        .isEmpty(),
    check('excerpt', 'The excerpt field id required')
        .not()
        .isEmpty(),
 check('body', 'The full text field id required')
        .not()
        .isEmpty()];
// create new post
router.post('/', validationRules, postsController.addPost);

Tout le reste semble correct, du moins d'après le code que vous avez publié.

utilisez ce code dans index.js pour obtenir des messages d'erreur flash localement dans EJS,

app.use(function (req, res, next) {
    res.locals.messages = req.flash();
    next();
});

ce n'est pas un bogue avec les validateurs express, c'est la façon dont les validateurs fonctionnent dans le cas des middlewares.

Au niveau racine, créez un répertoire appelé utils et à l'intérieur du répertoire un fichier validation.js et ajoutez votre code de validation dedans:

utils / validation.js

Change Line No: 2
From: 
const { check, validationResult } = require('express-validator');
To:
const { validationResult } = require('express-validator');

Remove Line Nos 29 to 39.

Dans les routes / dashboard.js incluent la validation. js

const validator = require('../../utils/validation.js');

Change Line No: 16
From: 
router.post('/post/add', dashboardController.addPost);
To:
router.post('/post/add', validator.addPostCheck, dashboardController.addPost);

Dans les contrôleurs / admin / dashboard.js

const { check } = require('express-validator');
exports.addPostCheck = [
    check('title', 'The title field id required')
    .not()
    .isEmpty(),
    check('excerpt', 'The excerpt field id required')
    .not()
    .isEmpty(),
    check('body', 'The full text field id required')
    .not()
    .isEmpty()
];

Référence

<div id="messages" class="text-center">
<% Object.keys(messages).forEach(function (type) { %>
    <% messages[type].forEach(function (message) { %>
        <% if (type === 'errors') {%>
            <div class="alert alert-<%= type %>"><%= message.msg %></div>
        <%} else { %>
            <div class="alert alert-<%= type %>"><%= message %></div>
        <% } %>
    <% }) %>
<% }) %>

J'ai appliqué la solution fournie par Saravanakumar TN avec une petite modification dans messages.ejs .

J'ai: ceci dans le contrôleur:

<div id="messages" class="text-center">
    <% Object.keys(messages).forEach(function (type) { %>
            <% messages[type].forEach(function (message) { %>
                    <div class="alert alert-success <% if (type === 'danger') { %> alert-dismissible <% } %> alert-<%= type %>">
                        <% if (type === 'danger') { %>
                            <button type="button" class="close" data-dismiss="alert">&times;</button>
                            <%= message.msg %>
                        <%} else { %>
                            <%= message %>
                        <% } %>
                </div>
            <% }) %>
    <% }) %>
</div>