On Dec 15, 3:50 pm, Jeremy Evans <[email protected]> wrote: > The fact that that syntax (a__b__c___d) works is just an accident due > to a bug in Dataset#from (calling :to_sym on a string instead of > creating an SQL::Identifier with it). > > Sequel only supports the following when splitting symbols: > > :a__b => a.b > :a___c => a AS c > :a__b___c => a.b AS c > > You should use this instead: > > DB1[:dbo__table.qualify(:database).as(:t1)] > > I'm going to commit a change that fixes the bug and will break the > syntax you are using. I agree that your version is more friendly to > your use case, but it's inconsistent with the rest of Sequel's > behavior, and will lead users to believe that they can use that syntax > in other places, which won't work.
Fixed: https://github.com/jeremyevans/sequel/commit/0202abc1a7564e3a26f2402c7f6c7251d08be918 Jeremy -- You received this message because you are subscribed to the Google Groups "sequel-talk" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/sequel-talk?hl=en.
