search

telkins / laravel-dag-manager

A SQL-based Directed Acyclic Graph (DAG) solution for Laravel.
MIT License
32 stars 9 forks source link

add getTable to dagEdge Model #23

Closed dealense7 closed 1 year ago

dealense7 commented 1 year ago

If dag table is created with different name from config, dagEdge model will try to get records by className (dag_edges). We can add getTable to fix this bug, as it is described in [ Laravel API].(https://laravel.com/api/9.x/Illuminate/Database/Eloquent/Model.html#method_getTable)

telkins commented 1 year ago

If dag table is created with different name from config, dagEdge model will try to get records by className (dag_edges). We can add getTable to fix this bug, as it is described in [ Laravel API].(https://laravel.com/api/9.x/Illuminate/Database/Eloquent/Model.html#method_getTable)

Hello and thanks for your PR submission. I'm not sure why, but I did not see any notification for this. Probably missed it. 😅

Anyway, I will take a closer look and try to get back to you soon. 🤓