Yes, seriously - it may be going against "common" rules but these are phpBB rules.
composer.json
, ext.php
and licence.txt
services.yml
that start with @ or % should be surrounded by single quotes$language
object and not $user->lang
$language
instead of $user
is not applicable to this extension, as he also supports phpBB 3.1 by the looks of the composer, so that would require the $user
class.Not wishing to start an argument but the tag is for 3.2 and even if it does support 3.1 it will not be validated against 3.1
Now I just hope I can find a tool to convert variable/method names and everything else to snake_case...
An extension won't be denied for things like that at all.
Currently you have in your template assignments
'ACP_BOARD_NOTICES_MANAGER' => $this->user->lang('ACP_BOARD_NOTICES_MANAGER'),
and then in your template you have {ACP_BOARD_NOTICES_MANAGER}
. {{ lang('ACP_BOARD_NOTICES_MANAGER') }}
in the template.You can tell I started this extension a long time ago
Oh that's a relief! Thanks for that
I guess I just need to find out a reliable way of checking that the current page is the board index. I'm not sure how yet.
Done, but doesn't show up to activeting it.And copy all the files to the folder [root of your phpBB]/ext/fq/boardnotices/
Excellent! Exactly what I needed, thank you very muchmrgoldy wrote: ↑Thu Jan 24, 2019 6:20 pm For your index check: https://github.com/phpbb/phpbb/blob/mas ... x.php#L221