Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ticket/12956] Ensure the contact columns have been added before using t... #2858

Merged
merged 1 commit into from
Aug 10, 2014

Conversation

nickvergessen
Copy link
Contributor

@bantu
Copy link
Collaborator

bantu commented Aug 10, 2014

With this patch I get

PHP Fatal error:  SQL ERROR [ mysqli ]<br /><br />Unknown column 'field_show_on_pm' in 'field list' [1054]<br /><br />SQL<br /><br />INSERT INTO community_profile_fields  (field_name, field_type, field_ident, field_length, field_minlen, field_maxlen, field_novalue, field_default_value, field_validation, field_required, field_show_novalue, field_show_on_reg, field_show_on_pm, field_show_on_vt, field_show_profile, field_hide, field_no_view, field_active, field_is_contact, field_contact_desc, field_contact_url, field_order) VALUES ('phpbb_twitter', 'profilefields.type.string', 'phpbb_twitter', '20', '1', '15', '', '', '[\\w_]+', 0, 0, 0, 1, 1, 1, 0, 0, 1, 1, 'VIEW_TWITTER_PROFILE', 'http://twitter.com/%s', 5) in .../phpBB/phpbb/db/driver/driver.php on line 844

@bantu
Copy link
Collaborator

bantu commented Aug 10, 2014

This SQL error seems to be caused by me not dropping all tables correctly before restoring from backup.

@bantu bantu merged commit 2c7e81b into phpbb:develop-ascraeus Aug 10, 2014
bantu added a commit that referenced this pull request Aug 10, 2014
[ticket/12956] Ensure the contact columns have been added before using t...

* nickvergessen/ticket/12956:
  [ticket/12956] Ensure the contact columns have been added before using them
@nickvergessen nickvergessen deleted the ticket/12956 branch August 10, 2014 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants