fix(php-nextgen): always return if not void

[simPod]

We're missing return statements if return type is not void, Since in that case the method returns either {{{returnType}}} or Error object:

    public function deleteZone(
        string $serverId,
        string $zoneId,
        string $contentType = self::contentTypes['deleteZone'][0]
    ): \My\Project\OpenApi\DnsInt\Model\Error
    {
        $this->deleteZoneWithHttpInfo($serverId, $zoneId, $contentType);
    }

for

paths:
  "/error":
    get:
      summary: Will always generate an error
      operationId: error
      responses: &commonErrors
        "400":
          description: The supplied request was not valid
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Error"
        "404":
          description: Requested item was not found
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Error"
        "422":
          description: The input to the operation was not valid
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Error"
        "500":
          description: Internal server error
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Error"
  "/servers/{server_id}/zones/{zone_id}":
    delete:
      summary: Deletes this zone, all attached metadata and rrsets.
      operationId: deleteZone
      tags:
        - zones
      parameters:
        - name: server_id
          in: path
          required: true
          description: The id of the server to retrieve
          schema:
            type: string
        - name: zone_id
          in: path
          required: true
          description: The id of the zone to retrieve
          schema:
            type: string
      responses:
        "204":
          description: "Returns 204 No Content on success."
        <<: *commonErrors

PR checklist

• Read the contribution guidelines.
• Pull Request title clearly describes the work in the pull request and Pull Request description provides details about how to validate the work. Missing information here may result in delayed response from the community.
• Run the following to build the project and update samples:

  ./mvnw clean package || exit
  ./bin/generate-samples.sh ./bin/configs/*.yaml || exit
  ./bin/utils/export_docs_generators.sh || exit
  

• File the PR against the correct branch: master (upcoming 7.x.0 minor release - breaking changes with fallbacks), 8.0.x (breaking changes without fallbacks)
• If your PR is targeting a particular programming language, @mention the technical committee members, so they are more likely to review the pull request.

[wing328]

thanks for the pr

what about adding a test endpoint to modules/openapi-generator/src/test/resources/3_0/php-nextgen/petstore-with-fake-endpoints-models-for-testing.yaml?

[simPod]

Thanks for the file pointer. I have adjusted the definition.