Showing 479 of 479 total issues
The method entity_name is not named in camelCase. Open
Open
public function entity_name()
{
return $this->hasOne(UniverseName::class, 'entity_id', 'entity_id');
}- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
The method login_history is not named in camelCase. Open
Open
public function login_history()
{
return $this->hasMany(UserLoginHistory::class);
}- Read upRead up
- Exclude checks
CamelCaseMethodName
Since: 0.2
It is considered best practice to use the camelCase notation to name methods.
Example
class ClassName {
public function get_name() {
}
}Source
TODO found Open
Open
// TODO : add scope to MailHeader- Exclude checks
TODO found Open
Open
// TODO: add sorting- Exclude checks
TODO found Open
Open
// TODO : surround with global superuser- Exclude checks
TODO found Open
Open
// TODO : switch to ajax calls- Exclude checks
TODO found Open
Open
* TODO: Make locale aware.- Exclude checks
TODO found Open
Open
{{-- TODO : use an ajax call to load the modal content and generate the modal only once instead for all row --}}- Exclude checks
TODO found Open
Open
// TODO: Match the attempt to write to the sec log- Exclude checks
TODO found Open
Open
* TODO: This is definitely a candidate for a better refactor!- Exclude checks
TODO found Open
Open
// TODO: Another silent failure?- Exclude checks
TODO found Open
Open
// TODO: find a way to handle this using recursive loop and determine common patterns- Exclude checks
TODO found Open
Open
// TODO: Use [].sort instead?- Exclude checks
TODO found Open
Open
// TODO: remove 'name' arg after deprecation is removed- Exclude checks
TODO found Open
Open
// TODO: Use this.as('ms')?- Exclude checks
TODO found Open
Open
{{-- TODO: Get rid of this copy pasta by using a partial or something. --}}- Exclude checks
TODO found Open
Open
// TODO: Find a better way to register and load all the locales in Node- Exclude checks
TODO found Open
Open
// TODO: Move this to another part of the creation flow to prevent circular deps- Exclude checks
TODO found Open
Open
// TODO: We need to take the current isoWeekYear, but that depends on- Exclude checks