Skip to content

Conversation

michalcichon
Copy link
Contributor

SMF2 importer script accepts the database prefix param (-f) but it is not properly applied for some of the tables. To fix it, the hardcoded smf_ prefix is replaced by {prefix} as for other tables in the importer script.

Affected tables:

  • personal_messages
  • pm_recipients

@CLAassistant
Copy link

CLAassistant commented Aug 20, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the migrations-tooling PR which includes changes to migrations tooling label Aug 20, 2025
Copy link
Member

@gschlager gschlager left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@gschlager gschlager merged commit a078ec3 into discourse:main Aug 20, 2025
17 checks passed
yuriyaran pushed a commit that referenced this pull request Aug 21, 2025
SMF2 importer script accepts the database prefix param (-f) but it is
not properly applied for some of the tables. To fix it, the hardcoded
`smf_` prefix is replaced by `{prefix}` as for other tables in the
importer script.

Affected tables:
- personal_messages 
- pm_recipients
@michalcichon michalcichon deleted the fix/smf-importer-personal-msg-prefix branch August 22, 2025 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
migrations-tooling PR which includes changes to migrations tooling
Development

Successfully merging this pull request may close these issues.

3 participants