[RC] Hide BBcode

[marcovo]

I'm sorry; my study does not go as well as planned. I imagined that during my last project I could also test the last things for this extension but sadly this is not the case. The deadline is July 1st so I really hope to finish this next week.
Surely there should not be any obstacles any more, since we indeed already arrived at 3.1.5 .
Arabic translation merged .

[infinitiv]

So does 3.1.5 adds necessary events?

[marcovo]

I guess 3.1.4 added the necessary event.

[FomenkoAndrey]

and?... any progress?

[Miri4ever]

I've added mine since 3.1.3 and still working great.

[infinitiv]

I guess 3.1.4 added the necessary event.

So it means that core edits won't be mandatory. I'm looking forward for the new version even harder.

And, bro...

and?... any progress?

The deadline is July 1st (...).

Be patient.

[marcovo]

I found some time this evening, I replaced the old temporary event by the official phpBB event. See also the latest commit on the github page.

Unfortunately I forgot that also 'draft_id', was added as temporary variable in the phpBB files, which has not been added yet to the official phpBB. Anyways, for now that one line is the only edit thats needed. Without it, your drafts will be censored if you load them back into your posting-page. The extra variable makes sure that your draft will be shown exactly as you saved it.

Also see the first post.

[Miri4ever]

Updated and saved my german translation for this ext:

Code: Select all

	'HIDEBB_HIDDEN_MESSAGE'	=>	'Diese Nachricht wurde versteckt',
	'HIDEBB_MESSAGE_UNHIDE'	=>	'Versteckte Nachricht: Aus',
	'HIDEBB_MESSAGE_HIDDEN'	=>	'Versteckte Nachricht: An',
	'HIDEBB_MESSAGE_HIDDEN_DESCRIPTION'	=>	'Sie müssen auf diesen Beitrag antworten um die versteckte Nachricht lesen zu können.',

[marcovo]

German language is now included, thanks .

[infinitiv]

Here goes basic Polish (pl) translation. Haven't been testing this ext in 3.1.5, so expect some tweaks, but as for now:

Code: Select all

	'HIDEBB_HIDDEN_MESSAGE'	=>	'Treść została ukryta',
	'HIDEBB_MESSAGE_UNHIDE'	=>	'Ukrywanie treści: wyłączone',
	'HIDEBB_MESSAGE_HIDDEN'	=>	'Ukrywanie treści: włączone',
	'HIDEBB_MESSAGE_HIDDEN_DESCRIPTION'	=>	'Aby zobaczyć ukrytą treść, musisz odpisać w tym temacie.',

UTF-8 is highly recommended.

[marcovo]

Thanks infinitiv, I will include it later. (EDIT 29-06 12:22 : Polish translation has been added)

For now, I've just made a pull request to phpBB to include the draft_id into the main code:
https://github.com/phpbb/phpbb/pull/3731
If this is merged, the extension may be 100% extension-ish when phpBB 3.1.6 comes out.

[marcovo]

I made a change to the code that allows you to use the extension without any core modifications. This fix does break however, if another extension modifies the $draft_id on posting.php, but I guess not many extensions will do so. Anyways, if the pull request is accepted into phpBB 3.1.6, this break will be fixed.

I will now be doing some final tests, after which I will look into submitting the extension into the customisation database. For now, the extension has been promoted to Beta stage .

Do note that today I've changed the extension-name from hide_bbcode to hideBBcode, in compliance with the coding guidelines.

[FomenkoAndrey]

set extension, I try it. works, thanks.
but there are a few comments:
1 - "You must be logged in to see the hidden text" - it is correct for the guests, but not right for users who are logged in, but had not yet replied to the topic.
2 - how to display the contents of bbcode when it is open, without the background and the inscription OPEN. if the user can view the content, then display it as a normal message.

[marcovo]

FomenkoAndrey:

1 - This extension is meant to just exactly do that: Hide the content from anyone who has not yet posted in the topic. If you are searching for an extension that hides the content from anyone who is not logged in, this extension is not what you're looking for.
2 - Modify ext/marcovo/hideBBcode/styles/all/template/hide_bbcode.html. To remove the box when the content is to be shown, replace the first two lines with this:

Code: Select all

<!-- BEGIN unhide_open --><!-- END unhide_open -->
<!-- BEGIN unhide_close --><!-- END unhide_close -->

[FomenkoAndrey]

This extension is meant to just exactly do that:

I understand well.

Now your extension says (Google translation from Russian):

"You must be logged in to see the hidden text"

you say:

Hide the content from anyone who has not yet posted in the topic

there is a difference?
or Russian translation of this block of text is not completely correct.

Modify

Thanks, what I need.