Avoid using HTML tags in translation strings (bp-forums/deprecated/1.6.php)

[ramiy]

this patch fixes several things:

1. I Found one string that didn't used i18n function - <code>bb-config.php</code> file location:

2. Moving out the <code> tags out of the translation strings.

3. Replacing the bb-config.php string with %s placeholders, like we do it in WP core.

4. Add <code> tags around all the bb-config.php strings. Unifying the strings. Some had, some didn't. Now all the file names wrapped with code tags.

[DJPaul]

Lots of good stuff, but I don't like the new string '%s file location:'. I think it ought to have a translator comment explaining the value that the token will be replaced with.

Note to committers: whoever puts this in once the patch is revised, please look at the escaping on the changed lines. There are a few more improvements we can make to this legacy code.

[ramiy]

@DJPaul, This is how we do it in WordPress core. When we have <code> (usually to emphasis file names, classes of functions), we use %s placeholder, and move the HTML tags outside.

[ramiy]

No escaping is needed. The placeholders are simple strings - not translation strings.

[boonebgorges]

In 10680:

Remove markup from translatable strings in legacy Forums component.

Props ramiy.
Fixes #6907.