Skip to content

[ticket/14069] Corrected sql_fetchfield() in style_update_p1 migration#3811

Merged
bantu merged 1 commit intophpbb:3.1.xfrom
marcovo:ticket/14069
Aug 10, 2015
Merged

[ticket/14069] Corrected sql_fetchfield() in style_update_p1 migration#3811
bantu merged 1 commit intophpbb:3.1.xfrom
marcovo:ticket/14069

Conversation

@marcovo
Copy link
Copy Markdown
Contributor

@marcovo marcovo commented Aug 5, 2015

Changed the sql_fetchfield() call to comply to the function definition.

PHPBB3-14069

Changed the sql_fetchfield() call to comply to the function definition.

PHPBB3-14069
@bantu
Copy link
Copy Markdown
Collaborator

bantu commented Aug 10, 2015

Patch looks good. Good catch. However, depending on what this migration does, we have to ensure it is correctly applied in later updates. @nickvergessen @Nicofuma

@Nicofuma
Copy link
Copy Markdown
Member

I don't think we should do anything if this migration has already been run.

@bantu
Copy link
Copy Markdown
Collaborator

bantu commented Aug 10, 2015

@Nicofuma In this case, this should be good to go.

bantu added a commit to bantu/phpbb that referenced this pull request Aug 10, 2015
[ticket/14069] Corrected sql_fetchfield() in style_update_p1 migration

* marcovo/ticket/14069:
  [ticket/14069] Corrected sql_fetchfield() in style_update_p1 migration
@bantu bantu merged commit 32f5dc3 into phpbb:3.1.x Aug 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants