Skip to content

[ticket/12931] Fix general error on user registration#2864

Merged
marc1706 merged 1 commit intophpbb:develop-ascraeusfrom
rxu:ticket/12931
Aug 10, 2014
Merged

[ticket/12931] Fix general error on user registration#2864
marc1706 merged 1 commit intophpbb:develop-ascraeusfrom
rxu:ticket/12931

Conversation

@rxu
Copy link
Copy Markdown
Contributor

@rxu rxu commented Aug 10, 2014

Fix "Illegal use of $_SERVER" general error on user registration
when $config['server_name'] is not set.

PHPBB3-12931.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should assign $config['server_name'] to $domain if check evaluates to true.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

?: without a space
and remove the brackets

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nvm, see what nickvergessen said.

Fix "Illegal use of $_SERVER" general error on user registration
when $config['server_name'] is not set.

PHPBB3-12931
@rxu
Copy link
Copy Markdown
Contributor Author

rxu commented Aug 10, 2014

Updated #2864..

marc1706 added a commit to marc1706/phpbb that referenced this pull request Aug 10, 2014
[ticket/12931] Fix general error on user registration
@marc1706 marc1706 merged commit 838d4dc into phpbb:develop-ascraeus Aug 10, 2014
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.

3 participants