Project

General

Profile

Bug #7250

Forum > New thread > Additional view frame is useless

Added by Michela Mosquera almost 5 years ago. Updated almost 5 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
anibal copitan
Category:
Forum
Target version:
1.9.10
Start date:
27/08/2014
Due date:
% Done:

100%

Estimated time:
Spent time:
6.26 h
Complexity:
Normal
SCRUM pts - complexity:
?

Description

Pasos para reproducir:

1. Crear un curso
2. Ir a la herramienta de Foro
3. Crear un foro
4. Dentro de un foro crear un Nuevo Tema

Files

1.png (51.7 KB) 1.png Michela Mosquera, 27/08/2014 19:36
responder_tema_link.png (8.94 KB) responder_tema_link.png anibal copitan, 18/09/2014 01:06
responder_tema.png (58.4 KB) responder_tema.png anibal copitan, 18/09/2014 01:09

Associated revisions

Revision 1eaac65b (diff)
Added by Anibal Copitan almost 5 years ago

Minor PRS - refs #7250

Revision adc2c1ae
Added by Yannick Warnier almost 5 years ago

Merge pull request #361 from enlacee/7250

Remove iframe from new thread creation page - refs #7250

History

#1

Updated by Michela Mosquera almost 5 years ago

  • Target version set to 1.9.10
#2

Updated by anibal copitan almost 5 years ago

Segun la imagen (marcado en rojo).
Este cuadro es un iframe que apunta a : http://chamilo19.me/main/forum/iframe_thread.php?forum=5&thread=2

Pero el tema es que aparece en Crear un nuevo tema (foro).

VISTA 01
Y revisando me di cuenta que es correcto que aparesca en :
HOME>cursoX/CategoriaForo/Categorías de foro/foro/foroTema/
(responder tema)

VISTA 02
Aqui se muetsra el iframe en la parte inferior del formulario. listando todas las respuestas.
El archivo de configuracion para mostrar el iframe : main/forum/forumconfig.inc.php:38

$forum_setting['show_thread_iframe_on_reply'] = true;

Nota:
tablas relacionadas:
c_forum_post

#3

Updated by anibal copitan almost 5 years ago

  • File deleted (responder_tema.png)
#5

Updated by anibal copitan almost 5 years ago

  • Assignee set to anibal copitan
  • % Done changed from 0 to 50
#6

Updated by anibal copitan almost 5 years ago

  • Status changed from New to Needs testing
  • Assignee deleted (anibal copitan)
  • % Done changed from 50 to 70

Actualizado, corrección realizada(para no mostrar iframe al momento de crear nuevo tema).

PR = [[https://github.com/chamilo/chamilo-lms/pull/361]]

#7

Updated by Angel Quiroz almost 5 years ago

Test:

En el código la variable $my_thread debería renombrarse a $myThread

Pero los cambios en el commit funcionan para no mostrar el iframe al crear un tema dentro de un foro

#8

Updated by anibal copitan almost 5 years ago

Se realizo la normalizacion para esta variable.
segun PSR.

verificar y aproboar para aprobar el PR.

#9

Updated by Yannick Warnier almost 5 years ago

  • Subject changed from Foro < Nuevo Tema < Vista adicional de visualización errado to Forum > New thread > Additional view frame is useless
  • Assignee set to Yannick Warnier
#10

Updated by Yannick Warnier almost 5 years ago

  • Status changed from Needs testing to Bug resolved
  • Assignee changed from Yannick Warnier to anibal copitan
  • % Done changed from 70 to 100

Verificado, aprobado y cerrado

Also available in: Atom PDF