misfits-be / l5.x-database-layering Goto Github PK

Needed to improve code quality.

For now the selector in the where statement is hardcoded. This is a bad practice.
This should by coded with a variable. That defaults to =

Voor de worden literale strings gebruikt bij het aanmaken van Repositories. Maar aangezien dit tegen de code standard is van laravel. Zou dit moeten veranderen naar de ::class notatie.

NOTE:

// Voorbeeld van ::class notatie
return \model_path\model_name::class;

The tests are in complete.

De ->lists() functie word niet meer ondersteund in Laravel versies hoger dan 5.3

De lists() is hernoemd naar ->pluck().

Voor nu kunnen gebruikers alleen maar een alle columns opvangen van de ->findOrFail(); repository methode. Dit zou moeten veranderen zodat zijn ook kunnen definiëren welke columns zij nodig hebben.

Voorbeeld van use case: Een edit formulier waar niet alle velden aangepast kunnen worden.

Voor nu onze Ci triggerdt errors op PHP HHVM. Maar aangezien we dit niet gebruiken. Mag die in onze ci staan als allow failure

Deze is nodig voor de refactoring van sommige systemen. In sommige systemen is direct direct in de vendor folder geinjecteerd. Vandaar de hoge prior

There is an undefined variable $id on ->update() and updateRich() method.
Rename the $id variable to $primaryKey

Critical priority given because this blocks travisCi tests on platforms.

Nu hebben we wel halve docblocks. Maar dit zou moeten vervolledigd worden.

Is handig omdat de gebruiker de providers niet meer hoeft te registreren aangezien dit automatisch gebeurd.

Op de documentatie wesbite staan in de table of contents de gebruikte packages. Hier zou een link achter de hyperlink moeten komen.

https://github.com/CPSB/ActivismeBE-database-layering/blob/94a23f1a13040b02df88e9be13b76146610fb469/resources/stubs/repository.stub#L15

IPV van 1 blank line worden er 2 gegenereerd. Dit zou moeten opgelost worden.

need to be done in the readme.

Voor sommige functies is het nodig om een pure Eloquent entiteit te hebben. Men kan deze implementeren. Door een ->baseModel(); functie aan te maken. In de aangemaakte repository van de applicatie.

Maar aangezien dit 'nutteloze lijnen code' zijn. Is het beter om een hoofd entiteit te creëren in de basis repository. Zodat er een reductie van codebase ontstaat.

Implementatie idee:

/**
 * Returns clean entity of model
 *
 * @return \Illuminate\Database\Eloquent\Model
 */
public function entity()
{
    return $this->newModel;
}

https://github.com/CPSB/ActivismeBE-database-layering/blob/175afdca5937bf6c4cccad581d7bd36f99470593/src/ActivismeBE/Repositories/Eloquent/Repository.php#L226

Op deze lijn zou gewoon een * geplaatst moeten worden.

Voor nu in de repository functions worde de identifier variables $id genoemd.

Dit word word als een slecht voorbeeld genoemd omdat variablen. Meestal minimaal 3 karakters moeten hebben. Vandaar dat alle $id variablen hernoemd moeten worden naar $identifier of $primaryKey

REF: https://github.com/bosnadev/repository/pull/89/files

Sommige onder de gebruikers gebruiken een ->fill(); method. Maar deze is niet gedefineerd in de Eloquent repository.

Lower the repository template to php 5.6

Voor nu is er geen ndocumentatie hoe de gebruiker deze library kan installeren in zijn/haar project.

Dit zou moeten moeten gedocumenteerd worden.

REF: https://github.com/mishbah/bosnadev_repository/commit/d58bf2fcdfcf60364f724e2b09068c66a27ab89f

Laravel 5.5 is vrij gegeven voor gebruik.

Onze package moet compatibel gemaakt worden voor deze versie.

Er zijn foutieve namespaces gedefineerd dit zou moete opgelost worden door de juiste namespaces

https://github.com/CPSB/ActivismeBE-database-layering/blob/a5dc38a5049854107655ac4b9982e069f908e27f/src/ActivismeBE/Repositories/Eloquent/Repository.php#L294

Wij hebben een paar bugs gefixt. Dus zou er nu een nieuwe versie getagd moeten worden.

Na een nieuwe versie kunnen we ook verder et de ontwikkeling aan de CSAM.