Controller for removing an entity not fetching the id












0














I have this controller method



        [HttpGet]
public IActionResult DeleteMake(int id)
{
var data = _dbContext.VehicleMake.Find(id);
return View(data);
}


I generated a delete view for it. I assumed that based on the entity I clicked delete on, it would get it's information. It doesn't. I don't know why. The id in the parameter is always 0. Interesting enough, if you type in something like .../DeleteMake/1, then it works on the id of 1. Just doesn't work with the view.



This is the view



@model Data.Entities.VehicleMakeEntity

@{
ViewData["Title"] = "DeleteMake";
}
<h2>DeleteMake</h2>
<h3>Are you sure you want to delete this?</h3>
<div>
<h4>VehicleMakeEntity</h4>
<hr />
<dl class="dl-horizontal">
<dt>
@Html.DisplayNameFor(model => model.Id)
</dt>
<dd>
@Html.DisplayFor(model => model.Id)
</dd>
<dt>
@Html.DisplayNameFor(model => model.Name)
</dt>
<dd>
@Html.DisplayFor(model => model.Name)
</dd>
<dt>
@Html.DisplayNameFor(model => model.Abrv)
</dt>
<dd>
@Html.DisplayFor(model => model.Abrv)
</dd>
</dl>

<form asp-action="DeleteMake">
<input type="submit" value="Delete" class="btn btn-default" /> |
<a asp-action="Index">Back to List</a>
</form>




And the method I use to actually list all the items



                private readonly IQueryable<VehicleMakeEntity> _source;

public MakeRepository(ProjectDbContext context)
{
this.context = context;
_source = this.context.VehicleMake;
}
public IEnumerable<VehicleMakeEntity> SelectListMake(int index, int count, Expression<Func<VehicleMakeEntity, int>> orderLambda)
{
return _source.Skip(index * count).Take(count).OrderBy(orderLambda);
}









share|improve this question


















  • 2




    Your form does not post back anything. Either add a route value for the ID in the <form> or add a hidden input for the ID
    – user3559349
    Nov 20 '18 at 9:19










  • I don't see in the code where the view is passing the ID to delete....
    – Tim
    Nov 20 '18 at 9:20










  • How would I do that @StephenMuecke?
    – TimmyNeutron
    Nov 20 '18 at 9:25










  • Your could add asp-route-id="@Model.Id" in the <form> element, or add a hidden input - <input type="hidden" asp-for="Id" /> or @Html.HiddenFor(m => m.Id)
    – user3559349
    Nov 20 '18 at 9:28










  • I just tried it. The third one doesn't error out but the issue is still the same. ` <form asp-action="DeleteMake"> <input type="submit" value="Delete" asp-route-id="@Html.HiddenFor(m => m.Id)" class="btn btn-default" /> | <a asp-action="Index">Back to List </a> </form>`
    – TimmyNeutron
    Nov 20 '18 at 9:34


















0














I have this controller method



        [HttpGet]
public IActionResult DeleteMake(int id)
{
var data = _dbContext.VehicleMake.Find(id);
return View(data);
}


I generated a delete view for it. I assumed that based on the entity I clicked delete on, it would get it's information. It doesn't. I don't know why. The id in the parameter is always 0. Interesting enough, if you type in something like .../DeleteMake/1, then it works on the id of 1. Just doesn't work with the view.



This is the view



@model Data.Entities.VehicleMakeEntity

@{
ViewData["Title"] = "DeleteMake";
}
<h2>DeleteMake</h2>
<h3>Are you sure you want to delete this?</h3>
<div>
<h4>VehicleMakeEntity</h4>
<hr />
<dl class="dl-horizontal">
<dt>
@Html.DisplayNameFor(model => model.Id)
</dt>
<dd>
@Html.DisplayFor(model => model.Id)
</dd>
<dt>
@Html.DisplayNameFor(model => model.Name)
</dt>
<dd>
@Html.DisplayFor(model => model.Name)
</dd>
<dt>
@Html.DisplayNameFor(model => model.Abrv)
</dt>
<dd>
@Html.DisplayFor(model => model.Abrv)
</dd>
</dl>

<form asp-action="DeleteMake">
<input type="submit" value="Delete" class="btn btn-default" /> |
<a asp-action="Index">Back to List</a>
</form>




And the method I use to actually list all the items



                private readonly IQueryable<VehicleMakeEntity> _source;

public MakeRepository(ProjectDbContext context)
{
this.context = context;
_source = this.context.VehicleMake;
}
public IEnumerable<VehicleMakeEntity> SelectListMake(int index, int count, Expression<Func<VehicleMakeEntity, int>> orderLambda)
{
return _source.Skip(index * count).Take(count).OrderBy(orderLambda);
}









share|improve this question


















  • 2




    Your form does not post back anything. Either add a route value for the ID in the <form> or add a hidden input for the ID
    – user3559349
    Nov 20 '18 at 9:19










  • I don't see in the code where the view is passing the ID to delete....
    – Tim
    Nov 20 '18 at 9:20










  • How would I do that @StephenMuecke?
    – TimmyNeutron
    Nov 20 '18 at 9:25










  • Your could add asp-route-id="@Model.Id" in the <form> element, or add a hidden input - <input type="hidden" asp-for="Id" /> or @Html.HiddenFor(m => m.Id)
    – user3559349
    Nov 20 '18 at 9:28










  • I just tried it. The third one doesn't error out but the issue is still the same. ` <form asp-action="DeleteMake"> <input type="submit" value="Delete" asp-route-id="@Html.HiddenFor(m => m.Id)" class="btn btn-default" /> | <a asp-action="Index">Back to List </a> </form>`
    – TimmyNeutron
    Nov 20 '18 at 9:34
















0












0








0







I have this controller method



        [HttpGet]
public IActionResult DeleteMake(int id)
{
var data = _dbContext.VehicleMake.Find(id);
return View(data);
}


I generated a delete view for it. I assumed that based on the entity I clicked delete on, it would get it's information. It doesn't. I don't know why. The id in the parameter is always 0. Interesting enough, if you type in something like .../DeleteMake/1, then it works on the id of 1. Just doesn't work with the view.



This is the view



@model Data.Entities.VehicleMakeEntity

@{
ViewData["Title"] = "DeleteMake";
}
<h2>DeleteMake</h2>
<h3>Are you sure you want to delete this?</h3>
<div>
<h4>VehicleMakeEntity</h4>
<hr />
<dl class="dl-horizontal">
<dt>
@Html.DisplayNameFor(model => model.Id)
</dt>
<dd>
@Html.DisplayFor(model => model.Id)
</dd>
<dt>
@Html.DisplayNameFor(model => model.Name)
</dt>
<dd>
@Html.DisplayFor(model => model.Name)
</dd>
<dt>
@Html.DisplayNameFor(model => model.Abrv)
</dt>
<dd>
@Html.DisplayFor(model => model.Abrv)
</dd>
</dl>

<form asp-action="DeleteMake">
<input type="submit" value="Delete" class="btn btn-default" /> |
<a asp-action="Index">Back to List</a>
</form>




And the method I use to actually list all the items



                private readonly IQueryable<VehicleMakeEntity> _source;

public MakeRepository(ProjectDbContext context)
{
this.context = context;
_source = this.context.VehicleMake;
}
public IEnumerable<VehicleMakeEntity> SelectListMake(int index, int count, Expression<Func<VehicleMakeEntity, int>> orderLambda)
{
return _source.Skip(index * count).Take(count).OrderBy(orderLambda);
}









share|improve this question













I have this controller method



        [HttpGet]
public IActionResult DeleteMake(int id)
{
var data = _dbContext.VehicleMake.Find(id);
return View(data);
}


I generated a delete view for it. I assumed that based on the entity I clicked delete on, it would get it's information. It doesn't. I don't know why. The id in the parameter is always 0. Interesting enough, if you type in something like .../DeleteMake/1, then it works on the id of 1. Just doesn't work with the view.



This is the view



@model Data.Entities.VehicleMakeEntity

@{
ViewData["Title"] = "DeleteMake";
}
<h2>DeleteMake</h2>
<h3>Are you sure you want to delete this?</h3>
<div>
<h4>VehicleMakeEntity</h4>
<hr />
<dl class="dl-horizontal">
<dt>
@Html.DisplayNameFor(model => model.Id)
</dt>
<dd>
@Html.DisplayFor(model => model.Id)
</dd>
<dt>
@Html.DisplayNameFor(model => model.Name)
</dt>
<dd>
@Html.DisplayFor(model => model.Name)
</dd>
<dt>
@Html.DisplayNameFor(model => model.Abrv)
</dt>
<dd>
@Html.DisplayFor(model => model.Abrv)
</dd>
</dl>

<form asp-action="DeleteMake">
<input type="submit" value="Delete" class="btn btn-default" /> |
<a asp-action="Index">Back to List</a>
</form>




And the method I use to actually list all the items



                private readonly IQueryable<VehicleMakeEntity> _source;

public MakeRepository(ProjectDbContext context)
{
this.context = context;
_source = this.context.VehicleMake;
}
public IEnumerable<VehicleMakeEntity> SelectListMake(int index, int count, Expression<Func<VehicleMakeEntity, int>> orderLambda)
{
return _source.Skip(index * count).Take(count).OrderBy(orderLambda);
}






c# entity-framework dependency-injection asp.net-core-mvc






share|improve this question













share|improve this question











share|improve this question




share|improve this question










asked Nov 20 '18 at 9:16









TimmyNeutron

258




258








  • 2




    Your form does not post back anything. Either add a route value for the ID in the <form> or add a hidden input for the ID
    – user3559349
    Nov 20 '18 at 9:19










  • I don't see in the code where the view is passing the ID to delete....
    – Tim
    Nov 20 '18 at 9:20










  • How would I do that @StephenMuecke?
    – TimmyNeutron
    Nov 20 '18 at 9:25










  • Your could add asp-route-id="@Model.Id" in the <form> element, or add a hidden input - <input type="hidden" asp-for="Id" /> or @Html.HiddenFor(m => m.Id)
    – user3559349
    Nov 20 '18 at 9:28










  • I just tried it. The third one doesn't error out but the issue is still the same. ` <form asp-action="DeleteMake"> <input type="submit" value="Delete" asp-route-id="@Html.HiddenFor(m => m.Id)" class="btn btn-default" /> | <a asp-action="Index">Back to List </a> </form>`
    – TimmyNeutron
    Nov 20 '18 at 9:34
















  • 2




    Your form does not post back anything. Either add a route value for the ID in the <form> or add a hidden input for the ID
    – user3559349
    Nov 20 '18 at 9:19










  • I don't see in the code where the view is passing the ID to delete....
    – Tim
    Nov 20 '18 at 9:20










  • How would I do that @StephenMuecke?
    – TimmyNeutron
    Nov 20 '18 at 9:25










  • Your could add asp-route-id="@Model.Id" in the <form> element, or add a hidden input - <input type="hidden" asp-for="Id" /> or @Html.HiddenFor(m => m.Id)
    – user3559349
    Nov 20 '18 at 9:28










  • I just tried it. The third one doesn't error out but the issue is still the same. ` <form asp-action="DeleteMake"> <input type="submit" value="Delete" asp-route-id="@Html.HiddenFor(m => m.Id)" class="btn btn-default" /> | <a asp-action="Index">Back to List </a> </form>`
    – TimmyNeutron
    Nov 20 '18 at 9:34










2




2




Your form does not post back anything. Either add a route value for the ID in the <form> or add a hidden input for the ID
– user3559349
Nov 20 '18 at 9:19




Your form does not post back anything. Either add a route value for the ID in the <form> or add a hidden input for the ID
– user3559349
Nov 20 '18 at 9:19












I don't see in the code where the view is passing the ID to delete....
– Tim
Nov 20 '18 at 9:20




I don't see in the code where the view is passing the ID to delete....
– Tim
Nov 20 '18 at 9:20












How would I do that @StephenMuecke?
– TimmyNeutron
Nov 20 '18 at 9:25




How would I do that @StephenMuecke?
– TimmyNeutron
Nov 20 '18 at 9:25












Your could add asp-route-id="@Model.Id" in the <form> element, or add a hidden input - <input type="hidden" asp-for="Id" /> or @Html.HiddenFor(m => m.Id)
– user3559349
Nov 20 '18 at 9:28




Your could add asp-route-id="@Model.Id" in the <form> element, or add a hidden input - <input type="hidden" asp-for="Id" /> or @Html.HiddenFor(m => m.Id)
– user3559349
Nov 20 '18 at 9:28












I just tried it. The third one doesn't error out but the issue is still the same. ` <form asp-action="DeleteMake"> <input type="submit" value="Delete" asp-route-id="@Html.HiddenFor(m => m.Id)" class="btn btn-default" /> | <a asp-action="Index">Back to List </a> </form>`
– TimmyNeutron
Nov 20 '18 at 9:34






I just tried it. The third one doesn't error out but the issue is still the same. ` <form asp-action="DeleteMake"> <input type="submit" value="Delete" asp-route-id="@Html.HiddenFor(m => m.Id)" class="btn btn-default" /> | <a asp-action="Index">Back to List </a> </form>`
– TimmyNeutron
Nov 20 '18 at 9:34



















active

oldest

votes











Your Answer






StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");

StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "1"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});

function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});


}
});














draft saved

draft discarded


















StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53389698%2fcontroller-for-removing-an-entity-not-fetching-the-id%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown






























active

oldest

votes













active

oldest

votes









active

oldest

votes






active

oldest

votes
















draft saved

draft discarded




















































Thanks for contributing an answer to Stack Overflow!


  • Please be sure to answer the question. Provide details and share your research!

But avoid



  • Asking for help, clarification, or responding to other answers.

  • Making statements based on opinion; back them up with references or personal experience.


To learn more, see our tips on writing great answers.





Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


Please pay close attention to the following guidance:


  • Please be sure to answer the question. Provide details and share your research!

But avoid



  • Asking for help, clarification, or responding to other answers.

  • Making statements based on opinion; back them up with references or personal experience.


To learn more, see our tips on writing great answers.




draft saved


draft discarded














StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53389698%2fcontroller-for-removing-an-entity-not-fetching-the-id%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown





















































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown

































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown







Popular posts from this blog

"Incorrect syntax near the keyword 'ON'. (on update cascade, on delete cascade,)

Alcedinidae

Origin of the phrase “under your belt”?