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/12961] Add link in anti-spam ACP page which links directly to captchas in titania #2875

Merged
merged 9 commits into from
Sep 18, 2014

Conversation

PayBas
Copy link
Contributor

@PayBas PayBas commented Aug 13, 2014

Since many of the CAPTCHAs that will be shipping with 3.1 are broken, we should add a (big) link, which goes directly to the anti-spam category/tag in the extensions database on phpbb.com.

Additionally, we might want to include a link to the knowledge-base. Or this post: https://www.phpbb.com/community/viewtopic.php?f=46&t=2122696

https://tracker.phpbb.com/browse/PHPBB3-12961

@phpbb-user phpbb-user added the WIP label Aug 13, 2014
@@ -298,6 +298,8 @@
// Visual Confirmation Settings
$lang = array_merge($lang, array(
'ACP_VC_SETTINGS_EXPLAIN' => 'Here you can select and configure plugins, which are designed to block automated form submissions by spambots. These plugins typically work by challenging the user with a <em>CAPTCHA</em>, a test which is designed to be difficult for computers to solve.',
'ACP_VC_EXT_GET_MORE' => 'For additional (and possibly better) anti-spam plugins, visit the',
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be For additional (and possibly better) anti-spam plugins, visit the %sphpBB.com anti-spam extensions database%s',
Instead.
Then sprintf the link into it via php and put it into the template.
The way you do it now will not work for a lot of translations

@@ -61,6 +61,9 @@ function main($id, $mode)

$this->tpl_name = 'acp_captcha';
$this->page_title = 'ACP_VC_SETTINGS';
$template->assign_vars(array(L_ACP_VC_EXT_GET_MORE => sprintf($user->lang['ACP_VC_EXT_GET_MORE'],
'<a href="https://www.phpbb.com/go/anti-spam-ext" target="_blank" style="font-weight: bold;">', '</a>')
Copy link
Member

Choose a reason for hiding this comment

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

I don't think target="_blank" is valid or has been for the last few years.

Copy link

Choose a reason for hiding this comment

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

@phpbb-user phpbb-user removed the WIP label Aug 21, 2014
@PayBas
Copy link
Contributor Author

PayBas commented Aug 21, 2014

Website team will need to create redirects for:
https://www.phpbb.com/go/anti-spam-ext >>> to the titania extension category for anti-spam extensions
https://www.phpbb.com/go/anti-spam >>> to the knowledge base article about spam prevention

@@ -298,6 +298,7 @@
// Visual Confirmation Settings
$lang = array_merge($lang, array(
'ACP_VC_SETTINGS_EXPLAIN' => 'Here you can select and configure plugins, which are designed to block automated form submissions by spambots. These plugins typically work by challenging the user with a <em>CAPTCHA</em>, a test which is designed to be difficult for computers to solve.',
'ACP_VC_EXT_GET_MORE' => 'For additional (and possibly better) anti-spam plugins, visit the <a href="https://www.phpbb.com/go/anti-spam-ext" target="_blank" style="font-weight: bold;">phpBB.com anti-spam extensions database</a>. For more information on preventing spam on your board, visit the <a href="https://www.phpbb.com/go/anti-spam" target="_blank" style="font-weight: bold;">phpBB.com knowledge base</a>.',
Copy link
Contributor

Choose a reason for hiding this comment

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

I still don't think target="_blank" is a good idea. We never do that for any .com links
People are clever enough to open links in a new tab nowerdays.

@@ -298,6 +298,7 @@
// Visual Confirmation Settings
$lang = array_merge($lang, array(
'ACP_VC_SETTINGS_EXPLAIN' => 'Here you can select and configure plugins, which are designed to block automated form submissions by spambots. These plugins typically work by challenging the user with a <em>CAPTCHA</em>, a test which is designed to be difficult for computers to solve.',
'ACP_VC_EXT_GET_MORE' => 'For additional (and possibly better) anti-spam plugins, visit the <a href="https://www.phpbb.com/go/anti-spam-ext"><strong>phpBB.com anti-spam extensions database</strong></a>. For more information on preventing spam on your board, visit the <a href="https://www.phpbb.com/go/anti-spam"><strong>.com knowledge base</strong></a>.',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be "phpBB.com knowledge base" ?

@bantu bantu added the WIP label Aug 23, 2014
Don't know how that happened. Weird

PHPBB3-12961
@PayBas
Copy link
Contributor Author

PayBas commented Aug 23, 2014

Don't know how I missed that. Weird. Fixed

@Nicofuma Nicofuma removed the WIP label Aug 23, 2014
@@ -298,6 +298,7 @@
// Visual Confirmation Settings
$lang = array_merge($lang, array(
'ACP_VC_SETTINGS_EXPLAIN' => 'Here you can select and configure plugins, which are designed to block automated form submissions by spambots. These plugins typically work by challenging the user with a <em>CAPTCHA</em>, a test which is designed to be difficult for computers to solve.',
'ACP_VC_EXT_GET_MORE' => 'For additional (and possibly better) anti-spam plugins, visit the <a href="https://www.phpbb.com/go/anti-spam-ext"><strong>phpBB.com anti-spam extensions database</strong></a>. For more information on preventing spam on your board, visit the <a href="https://www.phpbb.com/go/anti-spam"><strong>phpBB.com knowledge base</strong></a>.',
Copy link
Collaborator

Choose a reason for hiding this comment

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

"phpBB.com anti-spam extensions database" should be "phpBB.com extensions database" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

But we want to indicate that we are going to a specific page within the ext database (where all the anti-spam ext are located).

If you think the text of the link shouldn't be that detailed, that's fine with me.

@nickvergessen
Copy link
Contributor

@michaelc

Website team will need to create redirects for:
https://www.phpbb.com/go/anti-spam-ext >>> to the titania extension category for anti-spam extensions
https://www.phpbb.com/go/anti-spam >>> to the knowledge base article about spam prevention

Can you do that, so we can merge it?

nickvergessen added a commit to nickvergessen/phpbb that referenced this pull request Sep 18, 2014
[ticket/12961] Add link in anti-spam ACP page which links directly to captchas in titania

* PayBas/ticket/12961:
  [ticket/12961] Shorten ext-db link text
  [ticket/12961] Fix missing phpBB.com
  [ticket/12961] Remove target="_blank" and switch to <strong>
  [ticket/12961] Add additional link to the KB
  [ticket/12961] Move link to language file
  [ticket/12961] Refactor lang entry
  [ticket/12961] Change url to anti-spam-ext
  [ticket/12961] Change url to https://www.phpbb.com/go/anti-spam/
  [ticket/12961] Add link in anti-spam ACP page to titania
@nickvergessen nickvergessen merged commit ab9cdca into phpbb:develop-ascraeus Sep 18, 2014
nickvergessen added a commit that referenced this pull request Sep 18, 2014
[ticket/12961] Add link in anti-spam ACP page which links directly to captchas in titania

* PayBas/ticket/12961:
  [ticket/12961] Shorten ext-db link text
  [ticket/12961] Fix missing phpBB.com
  [ticket/12961] Remove target="_blank" and switch to <strong>
  [ticket/12961] Add additional link to the KB
  [ticket/12961] Move link to language file
  [ticket/12961] Refactor lang entry
  [ticket/12961] Change url to anti-spam-ext
  [ticket/12961] Change url to https://www.phpbb.com/go/anti-spam/
  [ticket/12961] Add link in anti-spam ACP page to titania
@PayBas PayBas deleted the ticket/12961 branch September 18, 2014 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants