Conversation
some are Copy from Backup
TillBrede
left a comment
TillBrede
left a comment
There was a problem hiding this comment.
Habe das Modul auf der 2 getestet. Das Zippen und Aufteilen auf mehrere Dateien scheint zu laufe. Bei mir lokal konnte ich den Branch nicht auschecken (Problem mit composer?). Das hochladen über FTP/SFTP/FTPS konnte ich nicht komplett testen (SB_UIGoDeeper, TargetDir). Der Login und das auslesen der Verzeichnisse läuft aber.
CSVZipExport/form.json
Outdated
| "confirm": "Go into this Directory?" | ||
| }, | ||
| { | ||
| "onClick": "SB_UIGoDeeper($id,$CurrentDir.'/'.$SelectTargetDirectory['SelectedDirectory'],$Host, $Port, $Username, $Password, $ConnectionType);", |
CSVZipExport/form.json
Outdated
| }, | ||
| { | ||
| "type": "Label", | ||
| "caption": "Single File per Variable will zip in a archive", |
| if (!$showError) { | ||
| $this->SetStatus(201); | ||
| } else { | ||
| echo $this->Translate('Username/Password is invalid'); |
There was a problem hiding this comment.
Dieser Fehler scheint nur bei SFTP korrekt angezeigt werden. Bei den anderen gibt es nur unschöne Fehlermeldungen.
Der Status 201 "A Variable is not selected" kann da irreführend sein
There was a problem hiding this comment.
Was meinst du mit unschönen Fehlermeldungen?
There was a problem hiding this comment.
Die Fehlermeldung im Bild. Bei SFTP wird einfach 'Username/Password is invalid' angezeigt. Bei dem Rest wird nichts abgefangen.
CSVZipExport/module.php
Outdated
| //*---------------*// | ||
|
|
||
| /** Handle the export via Button */ | ||
| public function UserExport(int $ArchiveVariable, int $AggregationStage, string $AggregationStart, string $AggregationEnd) |
There was a problem hiding this comment.
ArchiveVariable wird wohl nur für die Fehlermeldung genutzt
| */ | ||
| private function fetchAllValues(string $filePath, int $variable, int $level, int $start, int $end, int $limit) | ||
| { | ||
| $separator = $this->ReadPropertyString('DecimalSeparator'); |
There was a problem hiding this comment.
Es wird nicht geprüft ob die Variablen existieren. Nicht existierende Variablen führen zu unschönen Fehlermeldungen
| $this->UpdateMailInterval(); | ||
| $this->UpdateCyclicInterval(); |
There was a problem hiding this comment.
Der Status der Instanz scheint nicht richtig zurückgesetzt werden. Hier wäre ein guter Ort dafür.
CSVZipExport/module.php
Outdated
| $this->UpdateFormField('ZipFile', 'enabled', $exportOption == 'multi'); | ||
| $this->UpdateFormField('MultiInfo', 'visible', $exportOption == 'multi'); | ||
| } | ||
| /*UI Funktion to change the Port base on the connection type */ |
TillBrede
left a comment
TillBrede
left a comment
There was a problem hiding this comment.
Der Login Fehler wird noch nicht schön abgefangen.
Funktional konnte ich nicht testen, da das Submodul fehlt / nicht richtig geladen wird
| if (!$showError) { | ||
| $this->SetStatus(201); | ||
| } else { | ||
| echo $this->Translate('Username/Password is invalid'); |
There was a problem hiding this comment.
Die Fehlermeldung im Bild. Bei SFTP wird einfach 'Username/Password is invalid' angezeigt. Bei dem Rest wird nichts abgefangen.
CSVZipExport/module.php
Outdated
| $this->UpdateFormField('InformationLabel', 'caption', $this->Translate('Connection is valid')); | ||
| $this->UpdateFormField('Progress', 'visible', false); |
There was a problem hiding this comment.
Die Elemente InformationLabel sowie Progress existieren nicht
try to fix the composer autoload
2 participants
close https://github.com/symcon/modules/issues/281