A circular reference was detected while serializing an object of type 'System.Reflection

Question

I have an asp.net MVC 3 controller action method like this:

public JsonResult GetRecordingRates(int Id)
{            
    List<DefaultRateChart> defaultRateCharts = new List<DefaultRateChart>();
    using (IDefaultRateChartManager defaultRateChartManager = new ManagerFactory().GetDefaultRateChartManager()) {
       defaultRateCharts = defaultRateChartManager.GetAll().Where(rc => rc.Currency.Id == Id && (!rc.NumberPrefix.StartsWith("#") || rc.NumberPrefix.StartsWith("Default")) && rc.AccountCredit == "Credit").ToList();
    }
    return Json(defaultRateCharts);
}

I want to send this list to jquery ajax success method but I am getting 500 Internal Server Error

my ajax call is like this:

$.ajax({
type: "POST",
dataType: "json",
url: "/Home/GetRecordingRates",
data: {
    Id: $("#hdCurrencyId").val()                 
},
success: function (data) {
        alert(data);
   }
});

In firebug XHR under response tab, it says:

A circular reference was detected while serializing an object of type 'System.Reflection.RuntimeModule'.

[EDIT]

I changed the action method to this:

public JsonResult GetRecordingRates(int Id)
{
    List<DefaultRateChart> defaultRateCharts = new List<DefaultRateChart>();
    using (IDefaultRateChartManager defaultRateChartManager = new ManagerFactory().GetDefaultRateChartManager())
    {
        defaultRateCharts = defaultRateChartManager.GetAll().Where(rc => rc.Currency.Id == Id && (!rc.NumberPrefix.StartsWith("#") || rc.NumberPrefix.StartsWith("Default")) && rc.AccountCredit == "Credit").ToList();
    }

    return this.Json(
        new
        {
            Result = (from obj in defaultRateCharts select new { Id = obj.Id, DialPrefix = obj.NumberPrefix, Rate = obj.PurchaseRates })
        }
        , JsonRequestBehavior.AllowGet
     );
}

and I dont get that error now but how can I parse the collection in ajax success. I changed success to following but it is not adding rows in table.

success: function (data) {
            var row = $('<tr>');
            for(var i = 0; i < data.length; i++) {
                row.append($('<td>').html(data[i]));
            }   

            $('#results').append(row);                                      
            jQuery('#RecordingRates').dialog({ closeOnEscape: false });
            $(".ui-dialog-titlebar").hide();
            $("#RecordingRates").dialog({ dialogClass: 'transparent' });
            $('#RecordingRates').dialog('open');
           }
    });

In Firebug Net => XHR=> Json, It shows following JSON:

[Object { Id=

1

,  DialPrefix=

"1"

,  Rate=

2.6

}, Object { Id=

3

,  DialPrefix=

"2"

,  Rate=

2.6

}, Object { Id=

5

,  DialPrefix=

"7"

,  Rate=

3.5

}, 3 more...]


0
    Object { Id=

1

, DialPrefix=

"1"

, Rate=

2.6

}

1
    Object { Id=

3

, DialPrefix=

"2"

, Rate=

2.6

}

2
    Object { Id=

5

, DialPrefix=

"7"

, Rate=

3.5

}

3
    Object { Id=

7

, DialPrefix=

"8"

, Rate=

6

}

4
    Object { Id=

9

, DialPrefix=

"Default"

, Rate=

5

}

5
    Object { Id=

15

, DialPrefix=

"Subscription"

, Rate=

15

}

Have you tried hard-coding values for `DefaultRateChart` and removing the `using` part temporarily? Might not make sense to do that but it's a good way to isolate where the problem is. — von v., Apr 29 '13 at 13:25
Is this some kind of new method to write or am I right you are not closing the tr and the td's. What do you get when you inspect things. — Nick N., Apr 29 '13 at 13:34
@DotnetSparrow that is a totally new issue. But anyway see my answer below. — von v., Apr 29 '13 at 13:51

score 4 · Answer 1 · edited May 23 '17 at 12:16

This happens when two or more objects in your JSON points to each other, the serializer cannot reflect those into JSON. You should use the Newtonsoft serializer (It is explained here how to do it: Using JSON.NET as the default JSON serializer in ASP.NET MVC 3 - is it possible?)

There are several things you can do. The first was one is let the configuration manager know to ignore or preserve those references like this:

        config.Formatters.JsonFormatter.SerializerSettings.ReferenceLoopHandling = ReferenceLoopHandling.Ignore;
        config.Formatters.JsonFormatter.SerializerSettings.PreserveReferencesHandling = PreserveReferencesHandling.None;

or

you could annotate the properties you don't want to be serialized in the objects (So a circular reference won't exist). With the annotation [JsonIgnore] like this:

public class Order
{
    [JsonIgnore]
    public virtual ICollection<OrderLines> OrderLines{ get; set; } 
}

Thanks for the answer. Please see edited question. I need to use this collection/json to add rows in table — DotnetSparrow, Apr 29 '13 at 13:30
Ok, my answer does not correspond with the question anyway, since the question has been edited. — Nick N., Apr 29 '13 at 13:44

score 0 · Accepted Answer · answered Apr 29 '13 at 13:49

0

how can I parse the collection in ajax success. I changed success to following but it is not adding rows in table.

You are returning a collection with the following fields: Id, DialPrefix, Rate, and you are not using them.

var row = $('<tr>');
for(var i = 0; i < data.length; i++) {
    row.append($('<td>').html(data[i]));
}   

$('#results').append(row);

You need to add them to your loop:

row.append($('<td>').html(data[i].DialPrefix));
row.append($('<td>').html(data[i].Rate));

answered Apr 29 '13 at 13:49

von v.

16,868
4
60
84

Have you checked that you actually have data? Try printing in the console (`console.log(data)`). – von v. Apr 29 '13 at 13:55
alert won't help you, do a `console.log` and in FF or Chrome, go to the console tab and you should see your collection (array) printed out, if you have some data received. On the server side, have you checked that `from obj in defaultRateCharts select` produces result? Put that in a variable first and see if it indeed have a value. – von v. Apr 29 '13 at 14:07
I updated question again. sorry I understand it is new question. – DotnetSparrow Apr 29 '13 at 14:08
Are you sure that `results` is your `table`? and that it is not hidden or overwritten (z-index) by some element? Do an `alert($('#results').html())` after the loop and see if the rows are added - but they should be based on your code that I updated (answered to). – von v. Apr 29 '13 at 14:10

A circular reference was detected while serializing an object of type 'System.Reflection

2 Answers2

Linked