Sign in to follow this  

Sending an array via AJAX to MVC Controller

Recommended Posts

+FloatingFatMan    13,150

First, let me start by saying that, though I'm a seasoned developer with 30 years behind the keyboard, I've only been working with web development for about a year, so there are things I don't know.  However, this one really has me confused!

 

All I want to do is send an array of integers to an MVC controller using an AJAX call.  The code below works absolutely perfectly for me every single time I run it in my development environment (Windows 10, VS2017), BUT, when I roll it out to the production server (running IIS 7.5), I get an error 500, internal server error.  To make matters worse, this code WORKED on the production environment 1 week ago, and yesterday it decided "screw you!" and started to fail.

 

Note, when it errors, it jumps into the error part of the Ajax call; it never gets into the controller method, I guess because it doesn't like the int[] parameter anymore...

 

Here is my AJAX call:

	var selectedItems = new Array();
	$("input:checkbox[name=selectPrice]:checked").each(function () {
		selectedItems.push(this.id);
	});

	if (selectedItems.length > 0) {
		$.ajax({
			url: "/ProductPrice/CanApprove",
			type: "post",
			dataType: "json",
			contextType: "application/json",
			data: { selectedIds: selectedItems },
			traditional: true,
			success: function (result) {
				if (result.success) {
					//Do something useful
				} else {
					//It all went wrong
				}
			},
			error: function (_err) {
				//It all went REALLY wrong
			}
		});
	}

 

And here is my controller method:

[Authorize]
[HttpPost]
public JsonResult CanApprove(int[] selectedIds)
{
	bool canApprove = service.CanApprove(selectedIds, out string reason, out int[] approvableIDs);
	return Json(new { success = canApprove, ex = reason, approvableIDList = approvableIDs }, JsonRequestBehavior.AllowGet);
}

 

I've had to change over to just passing a delimited string for now, and converting it back into an array inside my controller method (which is just urgh!), but I'm REALLY struggling to understand why this just stopped working?!  Is there anyone here with more javascript experience than me that can explain what could have caused this?

 

 

Share this post


Link to post
Share on other sites
Seahorsepip    539

Your ajax code is not sending an int[] array but an object with an int[] array:

 

What you expect it to send:

[1,2,3,4]

 

What it actually sends:

{"selectedIds": [1,2,3,4]}

 

Change data property in your ajax call to:

data: selectedItems,

 

That should probably work if my assumptions were correct :p

 

It's been a while since I worked with MVC controllers but as far I remember you don't need to send the controller parameter key in the object data if there is a single parameter in the controller method.

Share this post


Link to post
Share on other sites
+FloatingFatMan    13,150

Yeah, I tried that, and other variants. Without the name of the parameter, all the controller receives is NULL. :(

 

Anyway... I forgot to post, I actually found a solution to the problem late yesterday afternoon, after much pulling out of hair, and commenced to kicking myself repeatedly once I'd found it!

 

I was on the verge of just giving up completely when I decided that, even though the controller code was simple, I'd stick it inside a try/catch (which I should have done from the start anyway, because y'know, DUH!).  And when I did that, I was finally able to get a sensible error message in the AJAX call.  It seems that for some reason, my call was having trouble opening a database connection as the error complained that the connection as in use!

 

Ah ha! says I. I've seen that before and immediately went to look at the webconfig file, followed immediately by going over to our DBA and punching him in the side of the head.  It seems that when he last updated the webconfig when they changed the password a few days ago, he REMOVED the command "MultipleActiveResultSets=True" from the connection string.

 

IDIOT! IDIOT! IDIOT!

 

I'm amazed the app was functioning at ALL with that missing!

 

Moral of this story folks... IT'S THE DBA'S FAULT! :p

 

 

  • Haha 1

Share this post


Link to post
Share on other sites
+Sledge    132

I think moral of the story is poor error handling.

 

Should always be the first thing to code so when things do go wrong (which they always do) you can check a log and point a finger.

 

 

Share this post


Link to post
Share on other sites
Seahorsepip    539
2 hours ago, Sledge said:

I think moral of the story is poor error handling.

 

 

On 9/23/2017 at 0:38 PM, FloatingFatMan said:

 

Moral of this story folks... IT'S THE DBA'S FAULT! :p

 

Seems the DBA is here and throwing the blame back at you :rofl:

 

But I have to agree with the DBA on this one, too many times I programmed a try catch without outputting the error in the catch resulting in missing the error and wondering what i did wrong :pinch:

Share this post


Link to post
Share on other sites
+Sledge    132
1 hour ago, Seahorsepip said:

 

Seems the DBA is here and throwing the blame back at you :rofl:

 

But I have to agree with the DBA on this one, too many times I programmed a try catch without outputting the error in the catch resulting in missing the error and wondering what i did wrong :pinch:

Hehe - nah just too many years covering my ass

Share this post


Link to post
Share on other sites
+FloatingFatMan    13,150
12 hours ago, Sledge said:

I think moral of the story is poor error handling.

 

Should always be the first thing to code so when things do go wrong (which they always do) you can check a log and point a finger.

 

 

Yeah, yeah, I know.. Like I said.  Idiot!! :p 

 

I'm still blaming the DBA though, as he did the connection string for the production server! :p 

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

Sign in to follow this  

  • Recently Browsing   0 members

    No registered users viewing this page.