Skip to content

Commit f71371d

Browse files
committed
Fix MakeResource Command to Properly Add Getters Functions
1 parent 7c906b1 commit f71371d

File tree

3 files changed

+10
-8
lines changed

3 files changed

+10
-8
lines changed

src/Commands/MakeResource.php

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -90,22 +90,20 @@ public function handle(): int
9090
$getterStub = file_get_contents($resourceStubsPath.'getter.default.stub');
9191
$foreignGetterStub = file_get_contents($resourceStubsPath.'getter.foreign.stub');
9292

93+
$getterFunctions = '';
9394
foreach ($columns as $_column) {
94-
$baseContent = substr_replace($baseContent,
95-
$this->writeGetter($getterStub, $_column->COLUMN_NAME, camel_case($_column->COLUMN_NAME)),
96-
-293, 0);
95+
$getterFunctions .= $this->writeGetter($getterStub, $_column->COLUMN_NAME, camel_case($_column->COLUMN_NAME));
9796
}
9897

98+
$foreignGetterFunctions = '';
9999
if ($detectForeignKeys) {
100100
foreach ($foreignKeys as $_foreignKey) {
101-
$baseContent = substr_replace($baseContent,
102-
$this->writeForeignGetter($foreignGetterStub, $_foreignKey->VARIABLE_NAME, $_foreignKey->ENTITY_DATA_TYPE),
103-
-20, 0);
101+
$foreignGetterFunctions .= $this->writeForeignGetter($foreignGetterStub, $_foreignKey->VARIABLE_NAME, $_foreignKey->ENTITY_DATA_TYPE);
104102
}
105103
}
106104

107-
$baseContent = str_replace(['{{ EntityName }}', '{{ EntityNamespace }}', '{{ EntityVariableName }}', '{{ ResourceName }}', '{{ ResourceNamespace }}'],
108-
[$entityName, $entityNamespace, $entityVariableName, $resourceName, $resourceNamespace],
105+
$baseContent = str_replace(['{{ GetterFunctions }}', '{{ ForeignGetterFunctions }}', '{{ EntityName }}', '{{ EntityNamespace }}', '{{ EntityVariableName }}', '{{ ResourceName }}', '{{ ResourceNamespace }}'],
106+
[substr($getterFunctions, 0, -1), substr($foreignGetterFunctions, 0, -1), $entityName, $entityNamespace, $entityVariableName, $resourceName, $resourceNamespace],
109107
$baseContent);
110108

111109
file_put_contents($filenameWithPath, $baseContent);

stubs/PHP7.4/repository.resource.class.stub

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ class {{ ResourceName }} extends Resource
1414
public function toArray(${{ EntityVariableName }}): array
1515
{
1616
return [
17+
{{ GetterFunctions }}
1718
];
1819
}
1920

@@ -24,6 +25,7 @@ class {{ ResourceName }} extends Resource
2425
public function toArrayWithForeignKeys(${{ EntityVariableName }}): array
2526
{
2627
return $this->toArray(${{ EntityVariableName }}) + [
28+
{{ ForeignGetterFunctions }}
2729
];
2830
}
2931
}

stubs/PHP8.0/repository.resource.class.stub

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ class {{ ResourceName }} extends Resource
1414
public function toArray(${{ EntityVariableName }}): array
1515
{
1616
return [
17+
{{ GetterFunctions }}
1718
];
1819
}
1920

@@ -24,6 +25,7 @@ class {{ ResourceName }} extends Resource
2425
public function toArrayWithForeignKeys(${{ EntityVariableName }}): array
2526
{
2627
return $this->toArray(${{ EntityVariableName }}) + [
28+
{{ ForeignGetterFunctions }}
2729
];
2830
}
2931
}

0 commit comments

Comments
 (0)