Project

General

Profile

Actions
Copy link

Bug #18289

closed

Merge Feature: Missing function for merge.log

Added by Enis Nuredini 3 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Carsten Rose
Target version:
24.5.0
Start date:
04.04.2024
Due date:
% Done:

0%

Estimated time:
Discuss:
Prio Planung:
No
Vote:

Description

A given merge.log path in qfq configuration results to an php error. 

Call to undefined method IMATHUZH\Qfq\Core\Helper\Path::setAbsoluteMergeLogFile()

The functionality for path over system store is not completed.

Actions
Copy link
 #1

Updated by Enis Nuredini 3 months ago

  • Assignee set to Enis Nuredini
Actions
Copy link
 #2

Updated by Enis Nuredini 3 months ago

Es wurde folgende Variable verwendet im Report: {{mergeLog:Y::d}}
Jedoch existiert der Token 'd' nicht. An der Stelle wäre normalerweise ein Sanitize. Wird aber hier nicht benötigt.

Actions
Copy link
 #3

Updated by Enis Nuredini 3 months ago

  • Status changed from New to In Progress
Actions
Copy link
 #4

Updated by Enis Nuredini 3 months ago

Der Pfad muss als Variable im System Store existieren. Die anderen Log Files werden anderst gehandhabt (nur Serverseitig vewendet) und dort wird der aktuelle Pfad jederzeit über eine Path function aufgerufen.
Diese Möglichkeit haben wir im Report nicht. Hier wird der fertige Pfad aus der Variable benötigt.

Nun wurde neu bei setDefaults in Config.php folgendes definiert:

SYSTEM_MERGE_LOG_PATHFILENAME => '/var/www/html/fileadmin/protected/qfqProject/log/merge.log',

Ein relativer Pfad beginnend mit fileadmin hat nicht funktioniert. Hier scheint sich der AS _exec nicht fündig zu werden im System. Damit wäre die Beschreibung in der Config mit dem relativen Pfad in diesem Falle nicht gültig. Oder liegt hier das Problem an der Docker Installation?

Actions
Copy link
 #5

Updated by Enis Nuredini 3 months ago

  • Status changed from In Progress to Feedback
  • Assignee changed from Enis Nuredini to Carsten Rose
Actions
Copy link
 #6

Updated by Enis Nuredini 3 months ago

Änderungen vorgenommen. Beim Merge Log wird der Default automatisch genommen falls nichts vom User angegeben. Bei allen Log Files werden nun neu wenn Pfade angegeben der Absolute Pfad ermittelt und dieser dann in die Config eingetragen.

Actions
Copy link
 #7

Updated by Enis Nuredini 3 months ago

  • Status changed from Feedback to Ready to sync (develop)
Actions
Copy link
 #8

Updated by Enis Nuredini 3 months ago

  • Status changed from Ready to sync (develop) to Closed
Actions
Copy link
 #9

Updated by Carsten Rose about 2 months ago

  • Target version changed from 24.9.0 to 24.5.0
Actions
Copy link

Also available in: Atom PDF