Problem with double type column about monitor-table-change-with-sqltabledependency HOT 14 CLOSED

I was looking at code, and I asume the main cause of this problem is in RecordChangedEventArgs, in GetValueObject(PropertyInfo propertyInfo, string value), becuase it uses the CultureInfo for the conversion (return Double.Parse(value, this.Culture);, but I don't know where change this configuration only for SqlTableDependency.

from monitor-table-change-with-sqltabledependency.

Hi, there is a property

      /// <summary>
    /// Gets or sets the culture information five letters iso code.
    /// </summary>
    /// <value>
    /// The culture information five letters iso code.
    /// </value>
    public string CultureInfoFiveLettersIsoCode { get; set; } = "en-US";

can you try to use it ?

tell me if is ok.

from monitor-table-change-with-sqltabledependency.

Hi, sorry for respond until now,

I think that this doesn't resolve my problem, My database has a double column with at least 10 decimals, and there isn´t a culture info with this configuration.

from monitor-table-change-with-sqltabledependency.

Hi, thanks for reopen this issue. What do you think of create an inexistent culture info (maybe an "xx-xx") and in the switch of the GetValueObject function replace return Double.Parse(value, this.Culture) with a function like this:

private ConvertDoubleValueObject(string value, string cultureInfo)
{
if (cultureInfo == "xx-xx")
return Double.Parse(value);
return Double.Parse(value, this.CultureInfo);
}

from monitor-table-change-with-sqltabledependency.

Sound a good idea.
Just one thing: can you download source code, apply your fix and run all integration tests?
If all if fine, i integrate your fix

from monitor-table-change-with-sqltabledependency.

Hi.
I change the property
public string CultureInfoFiveLettersIsoCode { get; set; } = "en-US";

to
public CultureInfo CultureInfo{ get; set; };

Now, setting the .NumberFormat of the specified culture info, you should be able to set number of decimals.

from monitor-table-change-with-sqltabledependency.

Hi Christian,

I was testing this upgrade, and i observe the same behavior,

            var mapper = new ModelToTableMapper<EventoGIL>();
            var updateOfModel = new UpdateOfModel<EventoGIL>();
            var cuInfo = new System.Globalization.CultureInfo("es-mx");

            Dependency = new SqlTableDependency<EventoGIL>(Conexion, Tabla, null, null, ColumnasMapeadas, null, DmlTriggerType.All);
            cuInfo.NumberFormat.NumberDecimalDigits = 10;
            cuInfo.NumberFormat.CurrencyDecimalDigits = 10;
            cuInfo.NumberFormat.PercentDecimalDigits = 10;
            Dependency.CultureInfo = cuInfo;

In my Database, I have this records:

The column FechaHora is my problem, because when I recieved the changed record, I receive only this:

So, I think the problem is not in
protected virtual object GetValueObject(PropertyInfo propertyInfo, string value){}

I think the problem could be generated from the MessageBag received.

I was checking your CultureInfoTest, but it only test for DateTimeFormat, but it doesn't test for single or double type columns.

from monitor-table-change-with-sqltabledependency.

I confirm that is not only a problem with the cultureInfo. I changed GetValueObject(PropertyInfo propertyInfo, string value) function, removing cultureInfo for single, double and decimal column, and the problem continue:

case TypeCode.Single:
return float.Parse(value);
//return float.Parse(value, base.CultureInfo);

            case TypeCode.Double:
                return double.Parse(value);
                //return double.Parse(value, base.CultureInfo);

            case TypeCode.Decimal:
                return decimal.Parse(value);
                //return decimal.Parse(value, base.CultureInfo);

I recieve this value:

Do you have any idea where is the problem?

from monitor-table-change-with-sqltabledependency.

working on

from monitor-table-change-with-sqltabledependency.

I have checked. As you can see from this video, all works fine for me.

Can you send me the script for creating your table and a zip file containing a solution that reproduce this error?

Regards

from monitor-table-change-with-sqltabledependency.

Hi Christian,

I saw the video, and check that you use a decimal column type. Could you use a float column type instead a decimal, I think the problem is related with the floating point variables.. You don't need to create a video.

And with your request, I need to create other project and DB, because it's strictly confidential because is for an utility company, and information it's a national secutiry issue. So let me check.

And again, thanks for all the time you spend (I know, it's time invested), but thanks.

from monitor-table-change-with-sqltabledependency.

It should be fixed.

Assuming your table is:

Your C# model property mapping FLOAT table column, must be a decimal:

I get:

Fix available with version 7.4.0.

from monitor-table-change-with-sqltabledependency.

Thank you for all your help Christian,

I'll be testing this temporal fix in a while, but a think it should work. Thanks a lot.
I'm surprised that nobody would have presented this problem before (sorry for my english, it's not my natural language).

from monitor-table-change-with-sqltabledependency.

As soon as integration test finish, i will be publich version 7.4.0

from monitor-table-change-with-sqltabledependency.