MVC CRUD: going from create directly to udpate

Question

I have a relatively simple flow:

posting an item -> preview of post -> option A: edit | option B: publish

Everything is working as expected, using standard CRUD whenever possible. However, in the scenario where an item is being posted, then going to preview screen and the user would like to publish (which essentially is just changing a bool from false to true in the DB) I get the following error message:

Validation failed for one or more entities. See 'EntityValidationErrors' property for more details. Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.

Exception Details: System.Data.Entity.Validation.DbEntityValidationException: Validation failed for one or more entities. See 'EntityValidationErrors' property for more details.

As in standard CRUD, I call the ID of the item when editing, but when moving from create to preview, there seems to be no logic way to call the ID as it is just being created. However, with the extra step in between, I think the issue lies with me trying to edit without having a ID specified in the preview view.

Here my controller code for the publishing part:

    //POST: /errandom/errandPreview
    public ActionResult ConfirmErrand()
    {
        // define variables for reuse
        var userID = User.Identity.GetUserId();
        DateTime nowUTC = DateTime.Now.ToUniversalTime();
        DateTime nowLocal = DateTime.Now.ToLocalTime();
        // define errand
        var errand = new errands
        {
            // form
            Published = true
        };
        // save errand to DB
        ERRANDOM.Entry(errand).State = EntityState.Modified;
        ERRANDOM.SaveChanges();
        return RedirectToAction("Index");
    }

Plus the view for completeness:


    @using (Html.BeginForm("errandPreviewForm", "errandom", FormMethod.Post, new { @id = "errandPreviewForm", @class = "form-horizontal", @role = "form" }))
    {
        @Html.AntiForgeryToken()
        @Html.HiddenFor(m => m.LatitudePosted, new { @id = "errandPreviewLatitudePosted", @class = "userActivity" })
        @Html.HiddenFor(m => m.LongitudePosted, new { @id = "errandPreviewLongitudePosted", @class = "userActivity" })
        @Html.HiddenFor(m => m.LocationPosted, new { @id = "errandPreviewUserLocationPosted", @class = "userActivity" })
        @Html.HiddenFor(m => m.ID, new { @id = "errandPreviewID", @class = "errandPreviewID" })
    
        
            @Html.DisplayFor(m => m.Title)
        
        
            
                @Html.DisplayNameFor(m => m.DateTimePosted)
            
            
                @Html.DisplayFor(m => m.DateTimePosted)
            
            
                @Html.DisplayNameFor(m => m.Category)
            
            
                @Html.DisplayNameFor(m => m.SubCategory)
            
            
                @Html.DisplayFor(m => m.Category)
            
            
                @Html.DisplayFor(m => m.SubCategory)
            
        
        
        
            
                
            
            
                @Html.DisplayFor(m => m.Description)
            
        
        
        
            
                @Html.DisplayNameFor(m => m.Location)
            
            
                @Html.DisplayFor(m => m.Location)
                @Html.HiddenFor(m => m.Location, new { @id = "errandPreviewLocationDisplayValue", @class = "manageDisplay" })
            
            
                
                
            
        
        
        
            
                @Html.DisplayNameFor(m => m.ASAP)
            
            
                @Html.DisplayFor(m => m.ASAP)
            
            
                @Html.DisplayNameFor(m => m.StartDateTime)
            
            
                @Html.DisplayFor(m => m.StartDateTime)
            
            
                @Html.DisplayNameFor(m => m.EndDateTime)
            
            
                @Html.DisplayFor(m => m.EndDateTime)
            
            
                @Html.DisplayNameFor(m => m.DurationInHours)
            
            
                @Html.DisplayFor(m => m.DurationInHours)
            
        
        
        
            
                @Html.DisplayNameFor(m => m.Offering)
            
            
                @Html.DisplayNameFor(m => m.PaymentMethod)
            
            
                @Html.DisplayFor(m => m.Currency)
            
            
                @Html.DisplayFor(m => m.Offering)
            
            
                @Html.DisplayFor(m => m.PaymentMethod)
            
            
                @Html.DisplayNameFor(m => m.Price)
            
            
                @Html.DisplayFor(m => m.Price)
            
        
        
        
            
                
            
            
                @Html.DisplayFor(m => m.FirstName)
                @*link to contact for questions*@
                @Html.DisplayFor(m => m.Rating)
            
        
    
        
            
                @Html.ActionLink("Edit", "EditErrand", "errandom", new { ID = Model.ID }, htmlAttributes: new { @id = "errandPreviewEditButton", @class = "manageButton col-xs-5 col-sm-5 col-md-5 col-lg-5" })
                
                @Html.ActionLink("Post", "ConfirmErrand", "errandom", new { ID = Model.ID }, htmlAttributes: new { @id = "errandPreviewPostButton", @class = "manageButton col-xs-5 col-sm-5 col-md-5 col-lg-5" })
            
        
    }

Any idea where I am going wrong? As said, the goal should be to go from CREATE directly to PREVIEW (which essentially is a trimmed down UPDATE) and then back to INDEX.

Willy David Jr · Accepted Answer

As far as I understand your problem, you want to preview before publishing, and if you already saved the record, you can get the ID using this:

//POST: /errandom/errandPreview
public ActionResult ConfirmErrand(int? ID) //Add this parameter to get record ID
{
    //Your other logic here..

    var errand = ERRANDOM.errands.Find(ID); //Find your record 
    errand.Published = true; 

    //Update errand to DB
    ERRANDOM.Entry(errand).State = EntityState.Modified;
    ERRANDOM.SaveChanges();
    return RedirectToAction("Index");
}

You can get the record using the Find method, since you are passing the ID on your code on your existing code of your View here:

@Html.ActionLink("Post", "ConfirmErrand", "errandom", new { ID = Model.ID }, htmlAttributes: new { @id = "errandPreviewPostButton", @class = "manageButton col-xs-5 col-sm-5 col-md-5 col-lg-5" })

MVC CRUD: going from create directly to udpate

Answers (1)

Related Questions