Shashank
Shashank

Reputation: 339

MEAN : Getting Error 400 (Bad Request) on POST

I am fairly new to the MEAN Stack, and after following this tutorial : http://www.bradoncode.com/tutorials/mean-stack-tutorial-part-4-angularjs/, I was trying to make my own web application to send and receive invoices. I have a localhost:3000/invoices page set up which is currently empty ([ ]). I am trying to fill out a form and post to it localhost:3000/invoices so that the MongoDB database is populated. My MongoDB process is constantly running. This is the Model outlining all the fields I require in a record :

'use strict';

/**
 * Module dependencies.
 */
var mongoose = require('mongoose'),
    Schema = mongoose.Schema,
    validation = require('./validation.server.model');

/**
 * Invoices Schema
 */
var InvoiceSchema = new Schema({
    invoice_date: {
        type: Date,
        default: Date.now
    },
    from_entity: {
        type: String,
        default: '',
        trim: true,
        required: 'legal entity cannot be blank'
    },
    from_service: {
        type: String,
        default: '',
        trim: true,     
        required: 'service line cannot be blank'
    },

    country: {
        type: String,
        default: '',
        trim: true,
        required: 'country cannot be blank'
    },

    to_entity: {
        type: String,
        default: '',
        trim: true,
        required: 'legal entity cannot be blank'
    },

    to_service: {
        type: String,
        default: '',
        trim: true,     
        required: 'service line cannot be blank'

    },

    partner: {
        type: String,
        default: '',
        trim: true,     
        required: 'partner cannot be blank'
    },

    invoice_number: {
        type: String,
        default: '',
        trim: true,     
        unique: true,
        required: 'invoice number cannot be blank'
    },

    currency: {
        type: String,
        default: '',
        trim: true,
        required: 'currency cannot be blank'
    },


    amount: {
        type: Number,
        default: '',
        trim: true,
        required: 'amount cannot be blank'

    }

});

mongoose.model('Invoice', InvoiceSchema);

This is my invoice client side controller :

'use strict';

// Invoices controller
angular.module('invoices').controller('InvoicesController', ['$scope', '$stateParams', '$location', 'Authentication', 'Invoices', '$filter',
    function($scope, $stateParams, $location, Authentication, Invoices, $filter) {
        $scope.authentication = Authentication;
        $scope.currentPage = 1;
        $scope.pageSize = 10;
        $scope.offset = 0;

        // Page changed handler
        $scope.pageChanged = function() {
            $scope.offset = ($scope.currentPage - 1) * $scope.pageSize;
        };

        // Create new Invoice
        $scope.create = function() {
            var invoice = new Invoices ({
                    amount: this.amount,
                    invoice_number: this.invoice_number,
                    partner: this.partner,
                    to_service: this.to_service,
                    to_entity: this.to_entity,
                    country: this.country,
                    from_service: this.from_service,
                    from_entity: this.from_entity
            });

            // Redirect after save
            invoice.$save(function(response) {
                $location.path('invoices/' + response._id);

                // Clear form fields
                $scope.name = '';
            }, function(errorResponse) {
                $scope.error = errorResponse.data.message;
            });
        };

        // Remove existing Invoice
        $scope.remove = function(invoice) {
            if (invoice) {
                invoice.$remove();

                for (var i in $scope.invoices) {
                    if ($scope.invoices [i] === invoice) {
                        $scope.invoices.splice(i, 1);
                    }
                }
            } else {
                $scope.invoice.$remove(function() {
                    $location.path('invoices');
                });
            }
        };

        // Update existing Invoice
        $scope.update = function() {
            var invoice = $scope.invoice;

            invoice.$update(function() {
                $location.path('invoices/' + invoice._id);
            }, function(errorResponse) {
                $scope.error = errorResponse.data.message;
            });
        };


        // Find a list of Invoices
        $scope.find = function() {
                $scope.invoices = Invoices.query();
        };

        // Find existing Invoice
        $scope.findOne = function() {
            $scope.invoice = Invoices.get({ 
                invoiceId: $stateParams.invoiceId
            });
        };


        // Search for a Invoice
        $scope.invoiceSearch = function(invoice) {
            $location.path('invoices/' + invoice._id);
        };
    }
]);

And this is a simple form in AngularJS on the front-end :

<section data-ng-controller="InvoicesController">
    <div class="page-header">
        <h1>New Invoice</h1>
    </div>
    <div class="col-md-12">
        <form class="form-horizontal" data-ng-submit="create()" novalidate>
            <fieldset>
                <div class="form-group">
                    <label class="control-label" for="invoice_number">Invoice Number</label>
                    <div class="controls">
                        <input type="text" data-ng-model="invoice.invoice_number" id="invoice_number" class="form-control" placeholder="Invoice Number" required>
                    </div>
                </div>

                <div class="form-group">
                    <label class="control-label" for="amount">Amount</label>
                    <div class="controls">
                        <input type="text" data-ng-model="invoice.amount" id="amount" class="form-control" placeholder="Amount" required>
                    </div>
                </div>

                <div class="form-group">
                    <label class="control-label" for="currency">Currency</label>
                    <div class="controls">
                        <input type="text" data-ng-model="invoice.currency" id="currency" class="form-control" placeholder="Currency" required>
                    </div>
                </div>

                <div class="form-group">
                    <label class="control-label" for="partner">Partner</label>
                    <div class="controls">
                        <input type="text" data-ng-model="invoice.partner" id="partner" class="form-control" placeholder="Partner" required>
                    </div>
                </div>

                <div class="form-group">
                    <label class="control-label" for="country">Country</label>
                    <div class="controls">
                        <input type="text" data-ng-model="invoice.country" id="name" class="form-control" placeholder="Country" required>
                    </div>
                </div>

                <div class="form-group">
                    <label class="control-label" for="to_service">Service Line (To:)</label>
                    <div class="controls">
                        <input type="text" data-ng-model="invoice.to_service" id="to_service" class="form-control" placeholder="Service Line (To:)" required>
                    </div>
                </div>

                <div class="form-group">
                    <label class="control-label" for="from_service">Service Line (From:)</label>
                    <div class="controls">
                        <input type="text" data-ng-model="invoice.from_service" id="from_service" class="form-control" placeholder="Service Line (From:)" required>
                    </div>
                </div>
                <div class="form-group">
                    <label class="control-label" for="to_entity">Entity(To:)</label>
                    <div class="controls">
                        <input type="text" data-ng-model="invoice.to_entity" id="to_entity" class="form-control" placeholder=" Entity (To:)" required>
                    </div>
                </div>

                <div class="form-group">
                    <label class="control-label" for="from_entity">Entity(From:)</label>
                    <div class="controls">
                        <input type="text" data-ng-model="invoice.from_entity" id="from_entity" class="form-control" placeholder=" Entity (From:)" required>
                    </div>
                </div>

                <div class="form-group">
                    <input type="submit" class="btn btn-default">
                </div>
                <div data-ng-show="error" class="text-danger">
                    <strong data-ng-bind="error"></strong>
                </div>
            </fieldset>
        </form>
    </div>
</section>

When I proceed and submit the form I get the following :

enter image description here

Amount is a required field as evident through the model, and I am always certain to fill in that value. I have typed numbers and decimals always, yet is says that field is required and displays Bad Request. Also whenever I do npm start, my application loads up without any errors. I have attempted watching several youtube tutorials, but none clarify the reason for this not working. Swift Help would be greatly appreciated :)

EDIT : When I remove the required from the Model, the transaction goes through, but none of the values inputted into the form are recorded, only the invoice date is.

Upvotes: 0

Views: 737

Answers (1)

TJ-
TJ-

Reputation: 14363

You get a date because there is a default set for that. Define $scope.invoice = {} in your controller so that it is bound to your view (2-way). Then, change all fetches from $scope.invoice. For example:

    $scope.invoice = {};
    $scope.create = function() {
        var invoice = new Invoices ({
                amount: $scope.invoice.amount,
                invoice_number: $scope.invoice.invoice_number,
                partner: $scope.invoice.partner,
                ...
        });
        ...
    };

Upvotes: 1

Related Questions