Ticket #629 (closed defeito: fixed)

Opened 11 years ago

Last modified 11 years ago

Não é possível alterar o endereço de um contato compartilhado

Reported by: wmerlotto Owned by: david.ianakiara
Priority: média Milestone: Expresso 2.0
Component: ContactCenter Version: trunk
Severity: Keywords: contactcenter compartilhamento
Cc: diogenesduarte, niltonneto, david.ianakiara WorkGroup:

Description

Após compartilhar o catálogo de endereços com outra pessoa, fui alterar as informações de um registro deste catálogo. A primeira e segunda aba (Pessoal e E-mails e Telefones) funcionaram perfeitamente, mas a última (Endereços) não. Qualquer informação adicionada ali não é gravada.

Change History

comment:1 Changed 11 years ago by david.ianakiara

  • Owner changed from alguem to david.ianakiara

comment:2 Changed 11 years ago by david.ianakiara

Durante a resolução do ticket #629 (Não é possível alterar o endereço de um contato compartilhado) eu observei no banco de dados que a tabela "phpgw_cc_typeof_ct_addrs" estava vazia, esta tabela é responsável de preencher o combo "tipo de endereço" do contato. Será necessário adicionar na instalação do expresso o seguinte comando:

insert into phpgw_cc_typeof_ct_addrs values (1,'Residencial'); insert into phpgw_cc_typeof_ct_addrs values (2,'Comercial');

esta é uma das correções a serem feitas para resolver o ticket #629, gostaria de saber como posso fazer isso.

comment:3 Changed 11 years ago by rommelcysne

David, o comando abaixo já existe no arquivo postgres.dump:

COPY phpgw_cc_typeof_ct_addrs (id_typeof_contact_address, contact_address_type_name) FROM stdin; 1 Comercial 2 Residencial \.

Outros comandos similares no mesmo arquivo são executados, mas este, por algum motivo, não é. Criei um arquivo só com essas linhas e executei o psql com ele, funcionou.

Pois é, o comando que resolve o problema já tá lá...

comment:4 follow-up: ↓ 5 Changed 11 years ago by david.ianakiara

Bom, então vejo este erro no dump como um problema na instação do expresso, não seria melhor criar outro ticket para isso? para a correção do ticket #629 além da inserção no banco de dados,foi necessário uma alteração no arquivo 'class.bo_people_catalog.inc.php', posso comitar a alteração que fiz em 'bo_people_catalog' e fechar este ticket?

comment:5 in reply to: ↑ 4 Changed 11 years ago by wmerlotto

Replying to david.ianakiara:

Bom, então vejo este erro no dump como um problema na instação do expresso, não seria melhor criar outro ticket para isso?

Mesmo estando relacionado ao processo de instalação, as correções pertinentes a este problema devem ser reportadas neste ticket. Então, pode fazer o commit da alteração do arquivo class.bo_people_catalog.inc.php e ainda alterar o arquivo de dump utilizado na instalação.

comment:6 Changed 11 years ago by eduardoalex

  • Status changed from new to closed
  • Resolution set to fixed

Testes foram feitos com o script de instalação e o Expresso foi instalado perfeitamente na criação e preenchimento da tabela phpgw_cc_typeof_ct_addrs.

Resolvido na revisão [1392]

comment:7 Changed 11 years ago by alexandrecorreia

  • Status changed from closed to reopened
  • Resolution fixed deleted

Problemas ao adicionar contatos, utilizando a Adição Rápida de Contatos, depois da revisão acima.

<p><b>Database error:</b> Invalid SQL: INSERT INTO phpgw_cc_addresses ( id_city,id_state,id_country,address1,postal_code,address_is_default,id_address) VALUES ( NULL,NULL,NULL,NULL,NULL,'1','1')<br>
<b>pgsql Error</b>: -1 (ERROR:  null value in column "id_country" violates not-null constraint)<br>
<br><b>File:</b> /home/alexandre/workspace/expresso/contactcenter/inc/class.so_main.inc.php<br><b>Line:</b> 492<br><b>Function:</b> so_main::commit_new / so_main::commit / bo_people_catalog::add_single_entry / bo_people_catalog::quick_add / ui_data::quick_add / ui_data::data_manager / ExecMethod
<p><b>Session halted.</b>

comment:8 Changed 11 years ago by eduardoalex

  • Status changed from reopened to closed
  • Resolution set to fixed

Resolvido na revisão [1399]

comment:9 Changed 11 years ago by alexandrecorreia

  • Status changed from closed to reopened
  • Resolution fixed deleted

Eduardo

Este ticket(#629), relata um problema referente a falta de registros no banco, conforme comentado acima. Com estes registros no banco o problema já é resolvido. Gostaria de saber referente as revisões [1392] e [1399] se são realmente necessárias ?

comment:10 Changed 11 years ago by niltonneto

Eduardo, poderia confirmar se isso procede? Realmente testei aqui comentando o código da revisão [1392] e funcionou normalmente.....

comment:11 follow-up: ↓ 12 Changed 11 years ago by eduardoalex

  • Cc david.ianakiara added

Além do problema na inserção dos registros no banco, que realmente não foi identificado por nós, haviam problemas na criação do endereço pois o código não contava com esses atributos. A revisão [1392] corrige o mesmo problema na Adição rápida de contatos...

comment:12 in reply to: ↑ 11 ; follow-up: ↓ 13 Changed 11 years ago by niltonneto

Replying to eduardoalex:

Além do problema na inserção dos registros no banco, que realmente não foi identificado por nós, haviam problemas na criação do endereço pois o código não contava com esses atributos. A revisão [1392] corrige o mesmo problema na Adição rápida de contatos...

Cara, acabei de testar e uma VM com a instalação Debian Lenny, e comentei as alterações das revisões [1392] e [1399]. Tanto a adição rápida quando a edição de contatos pessoais e compartilhados (emails,telefones e endereços com/res) funcionaram perfeitamente. Portanto, não conseguimos enxergar o porquê dessas revisões...... Alguém mais poderia testar por favor? Precisamos fechar esse ticket logo!!!!

comment:13 in reply to: ↑ 12 Changed 11 years ago by niltonneto

Replying to niltonneto:

Replying to eduardoalex:

Além do problema na inserção dos registros no banco, que realmente não foi identificado por nós, haviam problemas na criação do endereço pois o código não contava com esses atributos. A revisão [1392] corrige o mesmo problema na Adição rápida de contatos...

Cara, acabei de testar e uma VM com a instalação Debian Lenny, e comentei as alterações das revisões [1392] e [1399]. Tanto a adição rápida quando a edição de contatos pessoais e compartilhados (emails,telefones e endereços com/res) funcionaram perfeitamente. Portanto, não conseguimos enxergar o porquê dessas revisões...... Alguém mais poderia testar por favor? Precisamos fechar esse ticket logo!!!!

Eduardo, alguma novidade sobre esse ticket?

comment:14 Changed 11 years ago by david.ianakiara

Realmente, eu testei aqui e funcionou sim, houve um erro de minha parte adicionando a parte de endereços na função quick_add que é referente apenas à adicição de contatos rápidos e importação de contatos.

comment:15 Changed 11 years ago by alexandrecorreia

  • Status changed from reopened to closed
  • Resolution set to fixed

Revertido conforme revisão [1408]

Note: See TracTickets for help on using tickets.