Giter Club home page Giter Club logo

Comments (7)

riseoflex88 avatar riseoflex88 commented on August 17, 2024

It feels like this is still just as much a general php question as it is a Latitude question, I'll keep hunting for an answer myself but if anyone can put me out of my misery it'd be greatly appreciated.

from latitude.

riseoflex88 avatar riseoflex88 commented on August 17, 2024

Yay, got it...

Code fragment removed by @shadowhand because it is was vulnerable to SQL Injection.

Criteria is the key, make it in raw sql, then use criteria to interpret that into the Latitude format

Thanks again @shadowhand for a great package!

from latitude.

shadowhand avatar shadowhand commented on August 17, 2024

This seems like a situation where IN would be appropriate:

$query->andWhere(field('pd.destination_id')->in($destinations));

from latitude.

shadowhand avatar shadowhand commented on August 17, 2024

@riseoflex88 definitely do not use the code you posted, see my edit above.

from latitude.

riseoflex88 avatar riseoflex88 commented on August 17, 2024

The code you suggested is where I started but didn't work for me, I think I must have overlooked the error's I was experiencing so tried a different route.

$query->andWhere(field('pd.destination_id')->in($destinations));

I'm not sure this is building the correct output when providing an array. If I provide an array as above, I get an "Array to string conversion" error and the SQL produced looks like:

WHERE pd.destination_id IN (Array)

However, if I convert to a string...

$query->andWhere(field('pd.destination_id')->in(implode(', ', $destinations)));

SQL Output:

WHERE pd.destination_id IN ("a, b, c")

It'll (obviously) wrap the string in quotes so the query will look for a destination id of "a, b, c"

I've tried looking through the latitude code for a suggestion on what needs to change, but I can't find one,

from latitude.

shadowhand avatar shadowhand commented on August 17, 2024

Ah, my mistake. I forgot to unpack:

$query->andWhere(field('pd.destination_id')->in(/* unpack change */ ...$destinations));

Let me know if that works.

from latitude.

riseoflex88 avatar riseoflex88 commented on August 17, 2024

Ah, perfect. Sorry that's me not understanding the use of ... to even make a guess at adding that in there

from latitude.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.