[Approved] Casing in SQL queries (1 Viewer)

Oxan

Retired Team Member
  • Premium Supporter
  • August 29, 2009
    1,730
    1,124
    Home Country
    Netherlands Netherlands
    Hi,

    The TvBusinessLayer.BuildCommandTextMiniGuide() method uses some broken queries with different casing of table names in the where and from part of the query, resulting in broken behaviour on platforms that have case sensitive table names (Linux).

    See attached patch for the two errors that were a problem in my case. Please think about this when writing new queries.
     

    Attachments

    • tvbusinesslayer-casing.patch
      1.7 KB

    Oxan

    Retired Team Member
  • Premium Supporter
  • August 29, 2009
    1,730
    1,124
    Home Country
    Netherlands Netherlands
    I'd really like it if someone could commit this patch to SVN so that I can start using the offical builds again :)
     

    mm1352000

    Retired Team Member
  • Premium Supporter
  • September 1, 2008
    21,577
    8,224
    Home Country
    New Zealand New Zealand
    It looks perfectly fine to me. I'll raise a mantis and try and get some attention from some of the other devs...

    mm
     

    mm1352000

    Retired Team Member
  • Premium Supporter
  • September 1, 2008
    21,577
    8,224
    Home Country
    New Zealand New Zealand
    Hi again Oxan

    I just committed your patch in SVN 28078. Cheers for the contribution, and do let us know if you encounter any more of these issues!

    :D
     

    Users who are viewing this thread

    Top Bottom