Hi,

Just a small note: could you add brackets around the database tables in the module code?
Eg. 'INSERT INTO role_weights' would become 'INSERT INTO {role_weights}'.

This way the module works for people who have set up their database with a table prefix.

Kind regards

CommentFileSizeAuthor
#3 rw_dbprefixes.patch1.4 KBpfaocle

Comments

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented
Version: master » 4.7.x-1.x-dev

Spot on. I'm strapped for time atm, but is a simple tweak. I'm not looking at 5.0 just yet, so I'll get the 4.7 version sorted as soon as I can. Cheers.

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented
Assigned: Unassigned » pfaocle
Status: Active » Reviewed & tested by the community

Patch for 4.7 release attached. Getting my head around the new revision system etc now, so I'll commit this as soon as I can.

(Note: needs fixing in HEAD branch too before sorting a 5.0 release)

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented
StatusFileSize
new rw_dbprefixes.patch1.4 KB

Ug.

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

This patch wasn't committed - instead nedjo's patch has been backported from HEAD into the 4.7 release which included a fix to this. Cheers!

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)