Skip to content

Commit 0eb7bb2

Browse files
Merge pull request #392 from appwrite/fix-php-sdk-delete-return
fix: php delete methods to return array
2 parents 111fb88 + 8e04d94 commit 0eb7bb2

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

templates/php/src/Services/Service.php.twig

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,10 @@ class {{ service.name | caseUcfirst }} extends Service
2020
* @param {{ parameter.type | typeName }}${{ parameter.name | caseCamel | escapeKeyword }}
2121
{% endfor %}
2222
* @throws {{spec.title | caseUcfirst}}Exception
23-
* @return {% if (method.type == 'location') or (method.method | caseUpper == 'DELETE') %}string{% else %}array{% endif %}
23+
* @return {% if (method.type == 'location') %}string{% else %}array{% endif %}
24+
2425
*/
25-
public function {{ method.name | caseCamel }}({% for parameter in method.parameters.all %}{{ parameter.type | typeName }}${{ parameter.name | caseCamel | escapeKeyword }}{% if not parameter.required %} = null{% endif %}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, callable $onProgress = null{% endif %}): {% if (method.type == 'location') or (method.method | caseUpper == 'DELETE') %}string{% else %}array{% endif %}
26+
public function {{ method.name | caseCamel }}({% for parameter in method.parameters.all %}{{ parameter.type | typeName }}${{ parameter.name | caseCamel | escapeKeyword }}{% if not parameter.required %} = null{% endif %}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, callable $onProgress = null{% endif %}): {% if (method.type == 'location') %}string{% else %}array{% endif %}
2627
2728
{
2829
{% for parameter in method.parameters.all %}

0 commit comments

Comments
 (0)