I let php-cs-fixer format the code.. seems that they worked on it so the
most ugly line breaks are gone.
And hopefully we Travis won't get angry any more.
:-)
Fix coding guidelines incoherences. Code should match PSR1/2 now, and
php-cs is set to check it on each commit.
Also fixed the Git hook to take into account only added, modified,
copied and renamed files (preventing errors when trying to check deleted
files).
Closes#1260.
...due to changes on Transifex and the re-pluralization.
Some small changes to harmonize the texts and make them more
clear/userfriendly.
Fixes#1301
:-)
Wrong gettext function calls
There were some issues when gettext is not installed on the system, due
to direct function calls to gettext and not to the encapsulated ones.
Closes#1211.
Aurora.JS was included, but we were not including the codecs JS files.
Including them fix the impossibility to play OGG vorbis audio files when
HTML5 player is disabled and only Flash player and Aurora.JS are
enabled.
Closes#1250.
Generate `sceret_key` on Ampache installation
Generate a `secret_key` on Ampache installation and put it in the
generated config. This removes the need to do it manually, and the
secret key is ensured to be crypto safe.
Also move the `web_path` configuration setting under a "Various"
fieldset, as it had nothing to do with database settings.
Also added a comment about a missing `break` statement, which was on
purpose, but one could think it was a mistake at first sight.
Closes#1248.
This commit introduces a `nT_` function to handle pluralization in
localizations, and avoid the need to call ngettext, which was resulting
in errors if PHP Gettext module was not available (and this was not
enforced by Ampache).
Any occurrence of `ngettext` has been replaced by this `nT_` call.
Also removed the useless keywords in the `gather-messages.sh` script,
and added the newly introduced `nT_` keyword.
Closes issue #123.
Since ever, I hated it that the Update notification is within the
headerbox. Now it's on the top of the content div like the "updated
config" notification.
I also kicked out one permission check so the
"autoupdate" notification and/or the "new config" notification is only
shown if the permission is 100... Like before but in another way.
^^
Before, there was a `<span>...</span>`, no matter what user was
logged in. Now there is a div with `id="update_notify"` what is also
only shown, if the admin is logged in.
I also cleaned the depending CSS
rule in `/reborn/templates/default.css`
