The row keeping track of event repeat IDs within the sequences table is not properly appending the database prefix as set in one's settings.php file.

My settings.php file has this for the prefixes:

$db_prefix = array(
  'default'   => 'corp_',
  'users'     => 'shared_',
  'sequences' => 'shared_',
  'sessions' => 'shared_',
);

Yet, my sequences table contains:

INSERT INTO `shared_sequences` (`name`, `id`) VALUES ('shared_users_uid', 6),
('corp_menu_mid', 72),
('corp_vocabulary_vid', 4),
('corp_node_nid', 24),
('corp_node_revisions_vid', 24),
('event_repeat', 2),
('corp_term_data_tid', 9),
('corp_view_view_vid', 1),
('corp_aggregator_feed_fid', 1),
('corp_aggregator_item_iid', 4);

Shouldn't 'event_repeat' be prepended by my set prefix like the rest? I'm worried this will cause a conflict if I set up the Event Repeat on one of my other sites in my multi-site, shared database situation.

CommentFileSizeAuthor
#2 eventrepeat_sequence.patch856 bytesseanbfuller

Comments

seanbfuller’s picture

Version: 4.7.x-1.x-dev » 4.7.x-2.x-dev
seanbfuller’s picture

Status: Active » Needs review
StatusFileSize
new856 bytes

Good catch. Sorry for taking so long to get to this. This patch just wraps the sequence name in curly brackets so it should get a proper prefix.

Up for review.

francisu’s picture

Looks good to me.

seanbfuller’s picture

Status: Needs review » Fixed

Committed to both 4.7.x-2.x and HEAD

Anonymous’s picture

Status: Fixed » Closed (fixed)