Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
"zerodahero/laravel-workflow": "^4.0|^5.0"
},
"require-dev": {
"larastan/larastan": "^2.9",
"laravel/pint": "^1.14",
"nunomaduro/collision": "^8.1.1||^7.10.0",
"orchestra/canvas": "^8.8||^9.1",
Expand All @@ -31,6 +30,7 @@
"pestphp/pest-plugin-arch": "^2.7",
"pestphp/pest-plugin-laravel": "^2.3",
"phpstan/extension-installer": "^1.3",
"phpstan/phpstan": "^1.0",
"phpstan/phpstan-deprecation-rules": "^1.1",
"phpstan/phpstan-phpunit": "^1.3",
"spatie/laravel-ray": "^1.35"
Expand Down
4 changes: 2 additions & 2 deletions phpstan.neon.dist
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@ parameters:
- src
- config
- database

tmpDir: build/phpstan
checkOctaneCompatibility: true
checkModelProperties: true

4 changes: 2 additions & 2 deletions src/DatabaseLoader.php
Original file line number Diff line number Diff line change
Expand Up @@ -57,14 +57,14 @@ public function getWorkflowStateTransitionTableName(): string

public function load(string $workflowName): array
{
//$repo = app()->make(WorkflowRepository::class);
// $repo = app()->make(WorkflowRepository::class);

return $this->repo->findByName($workflowName);
}

public function all(): array
{
//$repo = app()->make(WorkflowRepository::class);
// $repo = app()->make(WorkflowRepository::class);

return $this->repo->all();
}
Expand Down
14 changes: 13 additions & 1 deletion src/Models/WorkflowTransition.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ class WorkflowTransition extends Model
{
use HasFactory;

protected $table = 'workflow_transitions'; // Static fallback

protected $guarded = ['id'];

protected $casts = [
Expand All @@ -21,7 +23,17 @@ class WorkflowTransition extends Model

public function getTable(): string
{
return app(DatabaseLoader::class)->getWorkflowTransitionTableName();
// ใช้ static table name ในช่วงที่ analyze
if (app()->runningInConsole() &&
(! app()->bound(DatabaseLoader::class) || app()->environment('testing'))) {
return $this->table;
}

try {
return app(DatabaseLoader::class)->getWorkflowTransitionTableName();
} catch (\Exception $e) {
return $this->table;
}
}

public function workflow(): BelongsTo
Expand Down
46 changes: 33 additions & 13 deletions src/WorkflowLoaderServiceProvider.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,6 @@ class WorkflowLoaderServiceProvider extends PackageServiceProvider
{
public function configurePackage(Package $package): void
{
/*
* This class is a Package Service Provider
*
* More info: https://github.com/spatie/laravel-package-tools
*/
$package
->name('laravel-workflow-loader')
->hasConfigFile('workflow_loader')
Expand All @@ -30,16 +25,41 @@ public function configurePackage(Package $package): void

public function packageRegistered()
{
$this->app->when(\Soap\WorkflowLoader\DatabaseLoader::class)
->needs('$config')
->give($this->app->make('config')->get('workflow_loader.loaders.database'));
// Register DatabaseLoader with safe config retrieval
$this->app->singleton(DatabaseLoader::class, function ($app) {
$config = $app['config']->get('workflow_loader.loaders.database', []);

$this->app->when(\Soap\WorkflowLoader\WorkflowLoaderRegistry::class)
->needs('$loaders')
->give($this->app->make('config')->get('workflow_loader.loaders'));
// Provide default config if not found
if (empty($config) || ! isset($config['tableNames'])) {
$config = [
'tableNames' => [
'workflows' => 'workflows',
'workflow_states' => 'workflow_states',
'workflow_transitions' => 'workflow_transitions',
'workflow_state_transitions' => 'workflow_state_transitions',
],
];
}

$this->app->bind(\Soap\WorkflowLoader\Contracts\WorkflowDatabaseLoader::class, \Soap\WorkflowLoader\DatabaseLoader::class);
return new DatabaseLoader(
$config,
$app->make(Repositories\WorkflowRepository::class)
);
});

$this->app->singleton('workflowLoaderRegistry', \Soap\WorkflowLoader\WorkflowLoaderRegistry::class);
// Register WorkflowLoaderRegistry
$this->app->singleton(WorkflowLoaderRegistry::class, function ($app) {
$loaders = $app['config']->get('workflow_loader.loaders', []);

return new WorkflowLoaderRegistry($loaders);
});

// Bind contracts
$this->app->bind(
Contracts\WorkflowDatabaseLoader::class,
DatabaseLoader::class
);

$this->app->singleton('workflowLoaderRegistry', WorkflowLoaderRegistry::class);
}
}