search

learnweb / moodle-mod_moodleoverflow

:speech_balloon: Non-linear discussion module for Q&A forums
GNU General Public License v3.0
12 stars 22 forks source link

fix: Patched SQL dialect issue LIMIT vs TOP in favor of Microsoft SQL Server #101

Open mrkskwsnck opened 2 years ago

mrkskwsnck commented 2 years ago

I stumbled over the LIMIT keyword which is not known for Microsoft SQL Server. For the latter SELECT TOP needs to be used.

image

The PR at hand here refactors the code to handle both dialects, while keeping behavior.