Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor constant names to follow Moodle frankenstyle coding standard #831

Open
Undreli-Studios opened this issue Nov 25, 2022 · 1 comment
Labels

Comments

@Undreli-Studios
Copy link

Good day team, there are a few places that could need a bit of a refactor to comply with Moodle coding style standard.

Description

In the file 'lib.php' which you can see here https://github.com/xAPI-vle/moodle-logstore_xapi/blob/master/lib.php, contains a few constants with names that do not comply with Moodle coding standard. In particular they lack the Frankenstyle with the plugin name that you can check here: https://docs.moodle.org/dev/Coding_style#Constants, for example:

define('XAPI_IMPORT_TYPE_HISTORIC', 1);
define('XAPI_IMPORT_TYPE_FAILED', 2);

This should be:

define('LOGSTORE_XAPI_IMPORT_TYPE_HISTORIC', 1);
define('LOGSTORE_XAPI_IMPORT_TYPE_FAILED', 2);

Version

Expected behaviour

  • The constants defined by this plugin should follow the Frankenstyle coding style from Moodle.

Actual behaviour

  • The constants do not follow this standard.

Thank you so much for you attention.

@davidpesce
Copy link
Collaborator

Thanks for the heads up on this! Feel free to submit a PR if you have some time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants