Obrigatoriedade do campo complemento no cadastro de Empresa
Mauro Soares
Nós, do IFS Sul de Minas campus Inconfidentes, observamos que a
obrigatoriedade de preenchimento do campo complemento no cadastro de
empresas, em algumas situações, acaba por gerar uma entrada de dados
desnecessária. Assim, convencionamos a hipótese de manter o campo,
porém, sem sua obrigatoriedade de preenchimento e fizemos algumas
adições no código do arquivo common.php que fica no diretório /app/
sagu . Segue abaixo trecho modificado, onde, as linhas comentadas são
as que foram acrescentadas pela eliminação da obrigatoriedade de
preenchimento do campo complemento.
Pedimos aos interessados que testem essa modificação, para que
possamos eliminar possíveis bugs relacionados a esta alteração.
Agradecemos desde já a oportunidade de poder colaborar e também de
obter qualquer colaboração, pois, é de grande valia a todos os
envolvidos. Obrigado.
function CheckFormParameters($list,$href=""){
$n = count($list);
$void = false; //variável atua como uma flag, eliminando a
obrigatoriedade do campo complemento
//print_r($list);
for ( $i=0; $i<$n; $i++ )
{
$name = $list[$i];
if ( !$name )
continue;
if(array_key_exists($name, $GLOBALS['_GET']))
{
$value = trim($GLOBALS['_GET']["$name"]);
}
else
{
if (array_key_exists($name, $GLOBALS['_POST']))
{
$value = trim($GLOBALS['_POST']["$name"]);
}
}
if($name == "complemento") //identificando o campo complemento
e atribuindo a flag
{
$void = true;
}
if (!is_numeric($value) AND empty($value) AND $void == false) //
porfim eliminando sua obrigatoriedade adicionando 'AND $void == false'
{
$msg = "Campo obrigatório [<b><i>$name</i></b>] não
informado!";
FatalExit("Erro de Digitação!",$msg,$href);
}
}
}
Wanderson Reis
On 27 ago, 16:41, Mauro Soares <maus...@gmail.com> wrote:
> Olá,
>
> Nós, do IFS Sul de Minas campus Inconfidentes, observamos que a
> obrigatoriedade de preenchimento do campo complemento no cadastro de
> empresas, em algumas situações, acaba por gerar uma entrada de dados
> desnecessária. Assim, convencionamos a hipótese de manter o campo,
> porém, sem sua obrigatoriedade de preenchimento e fizemos algumas
> adições no código do arquivo common.php que fica no diretório /app/
> sagu . Segue abaixo trecho modificado, onde, as linhas comentadas são
> as que foram acrescentadas pela eliminação da obrigatoriedade de
> preenchimento do campo complemento.
Realmente a obrigatoriedade deste campo não se faz tão útil em grande
parte dos casos. Vou anotar a possibilidade de excluir esta
obrigatoriedade em uma futura revisão.
Apenas gostaria de recomendar uma maneira mais adequada de resolver o
problema, existe um arquivo que processa este formulário é faz uma
chamada a função que você alterou (CheckFormParameter). Na chamada da
função é passado os campos que precisam ser verificados, bastaria
editar este arquivo específico (que não sei dizer qual de cabeça) e
retirar o campo complemento dos parâmetros passados para a função.
A solução que você implementou poderá afetar negativamente outras
partes / formulários do sistema que utilizem um campo com nome
"complemento".
Sobre aquele outro problema, editar a empresa, que conversamos pelo
telefone, eu já verifiquei e corrigi o erro. O problema era causado
por uma função desatualizada no banco de dados.
Uma nova revisão já esta disponível para download com a correção.
[ ]'s
Wanderson S. Reis
Mauro Soares
Segundo o que você bem explanou no post anterior, o que eu havia
pensado como uma resolução se tornaria uma futura falha de segurança.
Conforme, novamente, sua explicação consegui encontrar os arquivos e
os locais onde são passados os parâmetros para função, portanto,
problema sanado. Em face disto para uma melhor interação e facilidade
de ambos os lados, reitero meu pedido a ter permissão de commit no
svn, até porque já tenho uma solução simples e testada para as issues
29 e 30, sem quaisquer modificações radicais no que diz respeito ao
sistema.
Desde já agradeço o espaço cedido e aguardo prontamente seu parecer.
Att.
Mauro A. S. Rodrigues
IFSULDEMINAS - Inconfidentes/MG
Wanderson Santiago dos Reis
> Wanderson,
>
> Segundo o que você bem explanou no post anterior, o que eu havia
> pensado como uma resolução se tornaria uma futura falha de segurança.
> Conforme, novamente, sua explicação consegui encontrar os arquivos e
> os locais onde são passados os parâmetros para função, portanto,
> problema sanado. Em face disto para uma melhor interação e facilidade
> de ambos os lados, reitero meu pedido a ter permissão de commit no
> svn, até porque já tenho uma solução simples e testada para as issues
> 29 e 30, sem quaisquer modificações radicais no que diz respeito ao
> sistema.
>
Mauro,
Ficamos muitos satisfeitos com a participação do IFSULDEMINAS no
SA-EPROTEC, e agrademos muito a colaboração.
Esperamos em breve termos um modelo mais adequado para recolher as
contribuições dos usuários do programa. Mas no momento como ainda
dependemos do SA-EPROTEC e utilizamos em produção a mesma versão que
está no repositório não pretendemos compartilhar por agora o acesso de
commit ao repositório, pois alterações enviadas passam a fazer parte
do sistema imediatamente e sem nenhuma revisão. Com o tempo e com o
IFSULDEMINAS usando o sistema já em produção quem sabe podemos firmar
uma parceria oficial entre as instituições para o desenvolvimento em
conjunto?
De qualquer forma iremos incluir as suas modificações com muito prazer
em nosso repositório, mas precisamos de um patch para que possamos
testar antes de aplicá-lo e enviá-lo ao repositório. Para isso envie
como anexo ao issue que você abriu um patch da modificação gerada com
a ferramenta diff.
No Linux você pode gerar um patch com o seguinte comando:
$ diff -upr <caminho/arquivo/diretorio alterado>
<caminho/arquivo/diretorio original> > caminho/arquivo_patch.diff
--
Atenciosamente,
Wanderson Santiago dos Reis
Gerente de Tecnologia da Informação
Instituto Federal Minas Gerais - Campus Bambuí
"A simplicidade é o último grau de sofisticação"
Leonardo da Vinci
Mauro Soares
Já criei as patches e as anexei na issue 30, e estão separadas em dois
comentários, o primeiro com relação às issues e o segundo é da remoção
da obrigatoriedade do campo complemento.
Grato.
Wanderson Santiago dos Reis
>
> Já criei as patches e as anexei na issue 30, e estão separadas em dois
> comentários, o primeiro com relação às issues e o segundo é da remoção
> da obrigatoriedade do campo complemento.
>
Mauro,
Vou aplicar e testar os patches e estando tudo OK envio as alterações
para o repositório.
Wanderson Santiago dos Reis
>> Wanderson,
>>
>> Já criei as patches e as anexei na issue 30, e estão separadas em dois
>> comentários, o primeiro com relação às issues e o segundo é da remoção
>> da obrigatoriedade do campo complemento.
>>
>
> Mauro,
>
> Vou aplicar e testar os patches e estando tudo OK envio as alterações
> para o repositório.
Mauro,
Não vou conseguir fazer as atualizações no tempo que eu gostaria, pois
estamos numa fase muito corrida aqui em Bambuí (planejamento 2011).
Mas não se preocupe que todos os códigos serão revisados e incluídos
no repositório.