Cannot expand relation

[CyberiaResurrection]

@CyberiaResurrection, i try on a new clean instance of laravel and now it has passed this issue, but i have another 2 new issues.

1. Route login not found. I have in .env APP_DISABLE_AUTH=true and i run artisan config:clear and artisan config:cache and the problem still persist. As a temporary solution, we defined the following 3 routes :

$controllerMethod = '\AlgoWeb\PODataLaravel\Controllers\ODataController@index';
Route::get('odata2.svc/$metadata', ['uses' => $controllerMethod]);
Route::any('odata2.svc/{section}', ['uses' => $controllerMethod])->where(['section' => '.*']);
Route::any('odata2.svc', ['uses' => $controllerMethod]);
Now it's working

2. When I try to use the $expand parameter based on a relationship defined in the model, I receive the following error:

Symfony\Component\Debug\Exception\FatalThrowableError
Argument 2 passed to POData\Providers\Metadata\ResourceAssociationSet::keyNameFromTypeAndProperty() must be an instance of POData\Providers\Metadata\ResourceProperty, null given, called in /var/www/testing/vendor/algo-web/podata/src/POData/UriProcessor/QueryProcessor/ExpandProjectionParser/ExpandProjectionParser.php on line 193

Thank you very much and I really appreciate the effort.

Originally posted by @CalamarMihai in #236 (comment)

[CyberiaResurrection]

Since I cut a release overnight, and that seems to be a POData issue,@CalamarMihai, can you try setting your POData dependency to 0.3.7 on your testbed to see if that solves your second issue?

[CyberiaResurrection]

@CalamarMihai, after seeing if explicitly including POData 0.3.7 on your testbed solves your issue or not (since I expect you're using dev-master on POData indirectly), you'll need to give me more information so I can reproduce your second issue, solve it (and, ultimately, add it as a regression test).

• What models are you trying to use?

  • Could you supply cut-down versions of the model files you're using, and their associated migrations, so I can add them to our regression tests, such as (for example) here:
  • https://github.com/Algo-Web/POData-Laravel/blob/master/tests/Regression/Bgoak/Tests/Functional/ExpandTest.php

• What relation(s) are you trying to expand?

[CyberiaResurrection]

@CalamarMihai, all packaged tests are currently passing on a clean install of POData-Laravel itself as at current master HEAD ( 7727b6e ). This is just trying to constrain where the problem lurks.

@c-harris , given the timing of the report, it looks like something fired up the hyperdrive and departed to east hyperspace among PRs 226 and/or 229 on POData. Given how big and gnarly 229 ended up, I'm suspecting that in lieu of further feedback.

In light of that, what do you propose to audit and what do you propose to look for?

[CalamarMihai]

First, I apologize for the delay.
Right now I have done a new test using the versions:

• algo-web/podata 0.3.7
• algo-web/podata-laravel dev-master 7727b6e
• algo-web/o-data-metadata dev-master 4a7e2ef
• Laravel Framework 6.18.3

The first issue is ok now, after reinstall, i had no issue with the authentication error.

I tried the common types of relationships supported by Eloquent (belongsTo, hasOne, hasMany, etc..)

Regardless of the value of parameter $expand, even if the relationship exists or not, I receive the same error. If you think it helps, I can make a copy of the project and database and put it on file sharing.

Sorry for my bad english and thanks for your help.

Models.zip

[CyberiaResurrection]

Thanks for your patience, your help and the extra information, @CalamarMihai . I've updated the issue title and will try to investigate in the next few days, as I'm a little busy at the moment (unless @c-harris wants to take a swing).

[CalamarMihai]

Any news about this issue?