Probably missing attribute classes? about condorcet

[julien-boudry]

Hi @toddy15

Actually, I never need to use the corresponding attribute class (static or object). I must declare them with use keywords, but then, I use reflection based on the class name, to read the value of the attribute; so PHP never needs to load the corresponding class. It's used only for documentation generation.

PHP 8 allows it. If there is a PSR for attributes class, properties for documentation, probably I will use it instead. But there is not yet, so I have build a custom system.

[julien-boudry]

To be more precise : The use it only for using then alias. If I don't make the use, then I must do it each time :

#[CondorcetPHP\Condorcet\CondorcetDocAttributes\Description("Get the actual Ranking of this Vote.")] #[CondorcetPHP\Condorcet\CondorcetDocAttributes\FunctionReturn("Multidimenssionnal array populated by Candidate object.")] #[CondorcetPHP\Condorcet\CondorcetDocAttributes\Related("Vote::setRanking")] public function getRanking () : array { return $this->_ranking; }

[toddy15]

Yes, I see, it does not make sense to use that long form for the attributes. However, it seems to me that creating minimal classes for attributes is recommended by PHP, see https://www.php.net/manual/en/language.attributes.classes.php for details.

I've just created PR #39 with new classes, please take a look and decide if that works for you.

[julien-boudry]

#39 is merged