@anonymous-piwik-user opened this Issue on January 16th 2011

The update to piwik 1.1.1 displayed this message:

--------------------8<--------------------
Warning messages:

Superuser login name "daniel-faber" is no longer a valid format. The login must be between 3 and 100 characters long and contain only letters, numbers, or the characters '_' or '-' or '.' or '@' Edit your config/config.ini.php to change it.
--------------------8<--------------------

This doesn't make sense since my user name 'daniel-faber' observes the mentioned rules.

My small analysis showed that the user name gets checked by Piwik::checkValidLoginString($userLogin), where it is matched against this regexp: /^[A-Za-z0-9_.-@]*$/

This is not what the warning message says because '.-@' is interpreted as the range from '.' to '@' and not as the three characters '.', '-', and '@'. To fix this, just replace the regexp with /^[A-Za-z0-9_.@-]*$/

I did not test it, but I think it will work.

@robocoder commented on January 16th 2011 Contributor

fixed in trunk; dupe of #1762

This Issue was closed on January 16th 2011
Powered by GitHub Issue Mirror