Sql query error on GetJobs

[muhammetustaoglu]

In the query, one field in the join statement is "id" instead of "Id". This cause application error

[erwin-faceit]

With what type of database? I’ve not seen this error before.

[muhammetustaoglu]

We try this on MS Sql Server.

[muhammetustaoglu]

SqlTagsServiceStorage.cs page line 104 and 205. Can you fix the lines and create new version please? We would like to use this nuget package. Besides, hangfire version is also a little bit old. May be you can update it either.

[erwin-faceit]

We use this on MSSql in production for a few years now. What is the database collation used? If that contains CS, can you change that to CI?

[muhammetustaoglu]

Our database collation is "Turkish_CI_AS", so im sorry but it is already case insensitive

[erwin-faceit]

I’ll try to create a reproduction of this issue, but am on holiday this week.

[erwin-faceit]

I’m a bit surprised though:

The way to determine if a database or database object is to check its "COLLATION" property and look for "CI" or "CS" in the result. The CI indicates that the server is case insensitive.

[muhammetustaoglu]

It's ok. I have no rush. I'll create a PR. Review it when you have time. Btw, nice feature. Congrats!

[muhammetustaoglu]

Since i'm not authorized, i cant create PR. :(

[muhammetustaoglu]

Hello,
Did you have chance to debug the issue?

[erwin-faceit]

Sorry for the late reaction to this, caught up in regular work. I've just pushed an update of the components.

About updating the Hangfire version: usually packages reference the lowest possible version to maintain compatibility with older projects. You're free to use a higher version though, it should give no problems whatsoever.