Fails with prefix for database tables

magpie5212 - September 16, 2009 - 10:11
Project:Views Mail
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:active
Description

If the installation is using a prefix for the database then the module fails to get the default newsletter series.

Line 128 is missing lots of {} and uses the un-prefixed name. The atteached file has the fixes on line 128. I have not checked the rest of the code and am about to start testing.

AttachmentSize
views_mail.module.txt83 KB

#1

Summit - November 4, 2009 - 14:50

Hi,
Did you find other things? Looking into this module also.
greetings,
Martijn

#2

magpie5212 - November 6, 2009 - 14:18

Seems to be working OK after my fix.

#3

SomebodySysop - November 27, 2009 - 19:47

According to your fix, is this what line 128 should have?

<?php
  $res
= db_query("SELECT term_data.tid,term_data.name FROM {term_data} INNER JOIN {vocabulary} ON term_data.vid = vocabulary.vid WHERE vocabulary.module = 'simplenews' ORDER BY term_data.name");
?>

 
 

Drupal is a registered trademark of Dries Buytaert.