LogoManager 1.0.0.3 [27-11-2022] (16 Viewers)

[Sebastiii]

Hi

I don't know sql

 

[Ede]

I did a mistake, I just replaced the TvBusinessLayer.dll in TV server folder.
Now I replaced it additionaly in MP Folder and it works!
Thank you very much @Edalex and @Vasilich for your support!

 

[Edalex]

Anyway, @Sebastiii who could check if it is an actual bug?
Maybe these guys:
@HomeY @mm1352000
Thanks!
If case problem is important I think we need to explore other part of interactions with sql

 

[HomeY]

If case problem is important I think we need to explore other part of interactions with sql

SQL is a case insensitive language, but Ede's query uses >> gm.idGroup =1 instead of >> gm.idGroup =7

 

[Vasilich]

SQL is a case insensitive language

really depends on the implementation
https://dev.mysql.com/doc/refman/5.0/en/identifier-case-sensitivity.html
that is why it didn't work for Ede - he uses MySQL on Linux.
So we need only patch this string and that's it - we don't even need to port it to tve35 because the latter uses EF, so no inconsistency possible.

 

[HomeY]

really depend on the implementation

I seems the collation is leading in this for MySQL, if it's '_ci' (case-insensitive) or '_cs' (case-sensitive).

 

[mm1352000]

I have no idea how did you move TV Server database on linux but let's ask @Developers if they consider it as TVBusinessLayer bug
https://github.com/MediaPortal/Medi...atabase/TvBusinessLayer/BusinessLayer.cs#L484
I tried to build TVBusinessLayer with fixed query.

Yes, that is a bug.

 

[Edalex]

@sogge @fatt_fill and others!
Are you still interested in Argus support?
I think I made huge progress in it and LogoManager is working with Argus 2.2.2 (with repleaced dlls) and MP 1.7.0.

 

[sogge]

Thanks, for now I am using TVE but good to know.

 

[Vasilich]

@mm1352000 @Sebastiii should we open Jira for this?
mm, i hope that my explanation about just fixing it in tve3, because tve35 isn't affected, is correct?
I also have found another issue in same file (TVBusinessLayer), but that can affect TVE35 as well, so i will try to separate it.