Matthias Nagel
Matthias Nagel
I can confirm that #1299 fixed it, see here https://github.com/nagmat84/larastan-example/runs/7161581815?check_suite_focus=true. I will now use what I have learned (i.e. that you must not rename the template parameter!) and apply it...
Unfortunately, the problem is back again in https://github.com/nunomaduro/larastan/pull/1285 after I have added a test class for custom relations.
Addendum: After adding some `var_dump` I can know confirm that the variable `$rgt` happens to equal the title of album A (each album has an attribute title) and `$lft` equals...
I believe I found an solution and it is indeed a bug. `QueryBuilder::moveNode($key, $position)`, line 544 assumes that list($lft, $rgt) = $this->model->newNestedSetQuery()->getPlainNodeData($key, true); only returns an array with exactly two...
> :warning: We have a computer scientist onboard. 👨🏻🔬 Why the warning sign? Are you afraid of theoretical computer scientists? :astonished: (BTW: You guessed correctly, I am indeed a theoretical...
> While it makes sense to have the declaring class also in the generic types, I do not really see a use case for that. First of all it fixes...
I updated the change log. Where and how do I document user facing changes in order to complete task 3 out of 3?
I wanted to add a test which provides an real-world example that uses `TDeclaringModel`. But I was unable to figure out how. Is this somewhere documented? I expected that it...
I updated all PHPDoc comments to contain FQCN as requested by [this comment](https://github.com/nunomaduro/larastan/pull/1285#issuecomment-1159783587). I also added a test case to illustrate why `TDeclaringModel` is required. However, in doing so I...
I believe I fixed the problem even though I have not fully understood what I did. In order to solve the "one-must-not-rename template-parameter-names" I stepped through the source code, watched...