OpenAPITools/openapi-generator

[REQ][php] declareStrictTypes option for all PHP generators

Aberta

#12.771 aberto em 5 de jul. de 2022

Ver no GitHubGuia do colaborador
 (2 comentários) (4 reações) (0 responsável)Java (7.516 forks)batch import
Client: PHPEnhancement: FeatureGood First IssueServer: PHP

Métricas do repositório

Stars
 (26.243 estrelas)
Métricas de merge de PR
 (Mesclagem média 9d 1h) (123 fundiu PRs em 30d)

Description

Is your feature request related to a problem? Please describe.

Usually I totally against codegen options which modifies one line of code, but this one looks really handy.

Describe the solution you'd like

Add declareStrictTypes switch option to AbstractPhpCodegen class, then:

openapi-generator-cli generate -g php --additional-properties=declareStrictTypes=true \
    -o out -i petstore.yaml

then in all mustache files:

<?php 

{{#declareStrictTypes}}declare(strict_types=1);{{/declareStrictTypes}}

We also need to decide whether this switch option should be on or off by default. I think it should be on, but it can cause breaking changes.

Describe alternatives you've considered

Close this issue as not necessary feature 😆

Additional context

cc @jebentier @dkarlovi @mandrean @jfastnacht @renepardon

Guia do colaborador

Direção de pesquisa
Revise a classe AbstractPhpCodegen na base de código Java para entender como as opções são adicionadas aos geradores de código. Examine os modelos mustache PHP existentes para identificar onde declare(strict types=1) deve ser inserido. Verifique os comentários da issue para qualquer discussão sobre o valor padrão e possíveis mudanças que quebram compatibilidade. Considere entrar em contato com os mantenedores para orientação sobre se a opção deve ser habilitada por padrão.
Pilha de tecnologia
javaphp
Domain
toolingbackend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaPHPGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80