Remove validation bypass
authoreileen <emcnaughton@wikimedia.org>
Mon, 25 May 2020 00:15:55 +0000 (12:15 +1200)
committereileen <emcnaughton@wikimedia.org>
Mon, 25 May 2020 00:20:35 +0000 (12:20 +1200)
commit7c9a45c39ac9b86a5d5132c786f5a500f3723213
treef001ef6230e27a33964d80ca04dcb725b0e9ec85
parent8e268efcf1363eb8b64ce7bd3b72c627c61d8d96
Remove validation bypass

I was trying to understand why executeQuery allows validation to be bypassed. That seems
intuitively like something that should be handled by the calling function. This is one of the few
places that calls it - but I can't see a reason for it to by pass validation....

My best guess is that a parameter could legitimately be NULL - and the group properly is the
only one that seems to be a candidate

I also note the case for passing i18nRewrite seems dubious as the table has no localisation
CRM/Utils/Cache/SqlGroup.php