refactor: DRY flowview filters page layout wrapper

Abierto
#245 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

  • #246 de @somethingwithproof — cerrado sin fusionar

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
30/100
Tipo de issue
Refactorización
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
php
Área
backend

Línea de trabajo

Start by reading flowview_filters.php and tracing where action branches render top_header and bottom_footer, including embed handling. Add standalone regression tests for helper call ordering and embed behavior, plus source assertions that action paths use the shared helper. Run syntax checks on touched files; done means edit/default rendering use the helper and existing behavior is preserved.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

enhancement

Problem

flowview_filters.php repeats page wrapper layout handling for action rendering (top_header/bottom_footer) across action branches.

Proposed change

  • add a shared layout wrapper helper for flowview filters page rendering
  • route edit/default rendering through the shared helper
  • preserve behavior including embed handling

Tests

  • add standalone regression tests for helper call ordering and embed behavior
  • add source assertions that action paths use the shared helper
  • run syntax checks for touched files
Lenguaje dominante
PHP
Estrellas
21
Forks
13
Merge medio
1 d 15 h
PR fusionados (30 d)
13

Preparar el entorno

  • Sin Dockerfile ni archivo de Docker Compose
  • Tiene una plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de Cacti/plugin_flowview

Todos los issues de Cacti/plugin_flowview

Issues similares

Más issues de PHP

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.