WBCE CMS Forum

WBCE CMS – Way Better Content Editing.

You are not logged in.

#26 03.02.2016 20:50:40

evaki
Guest

Re: Download Gallery 2

download_gallery_v2.65_DEV

Isch hab färtisch - wird also für die weitere Entwicklung: (CSRF) Token / Design  abgegeben.

Bin dann mal weg  smile 

In den nächsten Tagen schaue ich dann mal in Stefeks Patch.

MfG. Evaki

Last edited by evaki (04.02.2016 09:40:17)

#27 04.02.2016 12:41:22

hgs
Member

Re: Download Gallery 2

Beim Update von 2.64 kommt diese Fehkermeldung

[== Undefiniert ==]
Creating new table for download gallery groups

There was an unknown exception: Call to a member function fetchRow() on null in line (62) of (/modules/download_gallery/upgrade.php) 

LG Harald
"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

Offline

#28 04.02.2016 13:30:23

evaki
Guest

Re: Download Gallery 2

jep, seit eh und je (die letzte von mir gesehene 2.61) drin
vorübergehende Lösung >Erweiterungen>Erweitert: reload
(in der patch-version fehlt die Datei übrigens)
Wenn man statt update z.B. install wählt, bekommt man auch eine Fehlermeldung, wenn auch nur eine Notice.

Das Teil hat halt lange niemand mehr angefaßt, einfach installiert und "geht".
Solange es seinen Zweck erfüllte, war es bisher keine Meldung wert.
PHP und MySql stellen mittlerweile höhere Anforderungen.
Schauen wir mal. Ist kein Beinbruch, es steht ja noch einiges an. roll

Ich gucke mir die Sache unabhängig vom Fortgang der Entwicklung trotzdem auch nochmal an.

Edit: Mit der beigefügen upgrade.php sollte der Fehler beseitigt sein.

Edit: Für die Dev hab' ich aber noch einen: Wird statt upgrade install gewählt zeigt's anschließend diverse Notice: "Undefined variable xxx" in download_gallery\view.php

MfG. Evaki

Last edited by evaki (05.02.2016 01:26:41)

#29 04.02.2016 15:04:44

hgs
Member

Re: Download Gallery 2

Nach update kommt jetzt beim upgrade:

[== Undefiniert ==]
Creating new table for download gallery groups

Adding new fields to the settings table
Database Field gfooter exists update not needed
Database Field gloop exists update not needed
Database Field gheader exists update not needed
Database Field search_filter exists update not needed
Database Field search_layout exists update not needed
Database Field suse_captcha exists update not needed
Database Field pushmode exists update not needed


Add default settings to database for downloadgallery section_id= 18
Database data header added successfully - There was an unknown exception: Call to undefined function mysql_error() in line (174) of (/modules/download_gallery/upgrade.php)

LG Harald
"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

Offline

#30 04.02.2016 15:35:16

evaki
Guest

Re: Download Gallery 2

Tja, ich teste im Moment nicht unter php7. Da geht sowas unter.
Gut, daß das dann von Dir kommt!

Zeile 174 auskommentieren !
// echo '<span class="bad">'.mysql_error().'</span><br />';

Dann sollte es gehen. (hoffe ich). Alles andere ist soweit ok.

Auskommentieren statt löschen deshalb, weil hier später möglicherweise wieder drauf -wenn auch anders- zurückgegriffen wird.

MfG. Evaki

Hab' nochmal ne neue drangehängt

Last edited by evaki (05.02.2016 01:24:38)

#31 04.02.2016 17:01:03

webbird
Administrator

Re: Download Gallery 2

Schön, ich übernehm dann mal. "Download Gallery 3"?


Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

#32 04.02.2016 17:01:56

webbird
Administrator

Re: Download Gallery 2

Is eigentlich noch was von der Wunschliste offen?


Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

#33 04.02.2016 17:10:56

evaki
Guest

Re: Download Gallery 2

Frau webbird, übernehmen Sie  cool
(CSRF) Token / Design
und der da von janz oben:
Edit: Für die Dev hab' ich aber noch einen: Wird statt upgrade install gewählt zeigt's anschließend diverse Notice: "Undefined variable xxx" in download_gallery\view.php

und die Korrektur von upgrade /upgrade265NEU_2.zip)

MfG. Evaki

Last edited by evaki (04.02.2016 17:12:11)

#34 04.02.2016 17:13:31

webbird
Administrator

Re: Download Gallery 2

Argh, das geht ja gut los...

[== PHP ==]
if (isset($_POST["SF"]) && isset($sort[$_POST["SF"]])) {    }             

Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

#35 04.02.2016 17:17:46

evaki
Guest

Re: Download Gallery 2

Jo, das ist die Sortierung, die vordem XSS-Warnung ausgelöst hatte.

(CSRF) Token wird soweit ich sehen konnte, für <form blubs bla gebraucht.
Aber das wirst Du ja viel besser wissen
MfG. Evaki

p.s. Hab' zwar nicht viel gemacht, aber viel gelernt

Last edited by evaki (04.02.2016 17:19:09)

#36 04.02.2016 17:29:18

webbird
Administrator

Re: Download Gallery 2

Oh Hiiiiiiiiiiiiiiilfe...

[== PHP ==]
if (isset($_POST["R1"]) && isset($sort[$_POST["sort1"]])) {     

} elseif (isset($_POST["R2"]) && isset($sort[$_POST["sort2"]])) {    

} elseif (isset($_POST["R3"]) && isset($sort[$_POST["sort3"]])) {   

} elseif (isset($_POST["R4"]) && isset($sort[$_POST["sort4"]])) {   

} elseif (isset($_POST["R5"]) && isset($sort[$_POST["sort5"]])) {   
  
}

Wenn ich ja wüßte, wofür das EIGENTLICH mal war, würde ich die Funktion ja einbauen...


Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

#37 04.02.2016 17:43:55

webbird
Administrator

Re: Download Gallery 2

Hat jemand was dagegen, wenn ich die ganzen HTML-Ausgaben in Twig-Templates auslagere?


Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

#38 04.02.2016 17:55:15

evaki
Guest

Re: Download Gallery 2

Wenn's so klug gedacht ist wie die Gedanken zum Token, wird es wohl richtig sein. Vielleicht auch nochmal Florian fragen oder auch 'nen Tag vergehen lassen.........
MfG. Evaki

#39 04.02.2016 18:14:54

webbird
Administrator

Re: Download Gallery 2

Hach, da begegnen mir doch einige lustige Codestellen wieder, die ich schon längst vergessen hatte...

$MESSAGE['ADMIN_INSUFFICIENT_PRIVELLIGES']


Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

#40 04.02.2016 18:19:44

evaki
Guest

Re: Download Gallery 2

Das hast Du nun davon  smile  smile

Ein Déjà-vu gefällig?

In Lep. gab's den Token in modify_extensions.php (gucksu $token = '';)
So wie dort eingebaut wurde verhinderte er in WBCE und WB die Option Filetype hinzufügen.

MfG. Evaki

Last edited by evaki (04.02.2016 18:25:39)

#41 04.02.2016 18:34:43

webbird
Administrator

Re: Download Gallery 2

Hör mir auf mit Token, davon krieg ich Pickel...


Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

#42 04.02.2016 18:36:04

webbird
Administrator

Re: Download Gallery 2


Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

#43 04.02.2016 18:37:07

hgs
Member

Re: Download Gallery 2

Auch das Update hilft nicht wirklich weiter, die Fehlermeldung wandert dann zu Zeile 181
Da warte ich doch bis Bianca fertig hat. wink
Wobei die Funktion im BE und FE ohne Probleme läuft.


LG Harald
"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

Offline

#44 04.02.2016 18:40:43

evaki
Guest

Re: Download Gallery 2

@hgs
ja, habe es soeben erst gesehen. update hat zig zeilen mit mysql_error()
Das machen wir dann später.
Hauptsache daß die Tabellen aktualisiert wurden.
Soll heißen, trotz der Fehlermeldungen arbeitet das Modul richtig!

Beim nächsten Modul werd' ich dann doch php7 einsetzen
MfG. Evaki

Last edited by evaki (04.02.2016 18:45:27)

#45 04.02.2016 18:47:58

webbird
Administrator

Re: Download Gallery 2

Bisherige Änderungen (Dateien view.php und add.php):

* removed: "gallery created in xx seconds" HTML comment (view.php)
* removed: CSS file loading (WB < 2.7)
* removed: empty if-statements (view.php)
* removed: DB error messages in HTML comments (view.php)
* Backend components check user permissions
* DB Statements: mask column names with ``
* DB Statements: mysql strict compliance

Darüber hinaus:

* view.php kommt jetzt auch mit fehlenden Settings klar
* längliches if()..elseif()... durch switch() ersetzt

In der view.php gibt es noch mehr zu optimieren, vor allem wenn der HTML-Kram noch ausgelagert wird.


Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

#46 04.02.2016 18:48:04

evaki
Guest

Re: Download Gallery 2

@webbird
Wobei das aktuelle Modul, eben das Original, auf allen WB+Gabeln lief und nach meinem Verständnis auch weiterlaufen sollte. Wenn dann mit WB2.8.4 -so wie Steff dies schon anmerkte- die Systeme eine Schere darstellen, oder auch der Patch noch in Bearbeitung geht, kann man sich immer noch für'n Fork entscheiden.

Meine 3 Cents, 2 Lire, 1 Öre oder so
MfG. Evaki


webbird wrote:

* längliches if()..elseif()... durch switch() ersetzt

die if isset Konstruktionen wurden als Schutz gegen XSS eingebracht.
Ob das mit switch dann noch gewährleistet ist, müßte -wenn's nicht genau so gesichert ist- erneut überprüft werden.

Last edited by evaki (04.02.2016 19:00:15)

#47 04.02.2016 18:52:44

florian
Administrator

Re: Download Gallery 2

HTML-Kram gern nach Twig auslagern. Ich hab ja gesagt, dass die zig Eingabefelder im BE sowieso unübersichtlich sind.
Von mir aus braucht die DLC 3.0 auch nicht mehr mit WB classic kompatibel zu sein, zumal es dort ja auch einen nur für WB-Entwicklungsstrang des Moduls gibt.
http://forum.wbce.org/viewtopic.php?pid=3186#p3186

Ihr macht das schon.


Nicht ärgern. Nur wundern.

Offline

#48 04.02.2016 19:02:18

evaki
Guest

Re: Download Gallery 2

Bei einer  DLC 3.0 wärs mir auch egal
Hier haben wir aber noch die DLG 2 vor uns liegen

Ich könnte ja auch das tote Pferd DLG2 hin zum Sauerbraten reiten, und Ihr bastelt ein DLG3 im Git  big_smile
MfG. Evaki

Last edited by evaki (04.02.2016 19:07:00)

#49 04.02.2016 19:06:45

webbird
Administrator

Re: Download Gallery 2

Ohne Gewähr, bei mir funktioniert's....


Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

#50 04.02.2016 19:08:06

webbird
Administrator

Re: Download Gallery 2

evaki wrote:
webbird wrote:

* längliches if()..elseif()... durch switch() ersetzt

die if isset Konstruktionen wurden als Schutz gegen XSS eingebracht.
Ob das mit switch dann noch gewährleistet ist, müßte -wenn's nicht genau so gesichert ist- erneut überprüft werden.

Nee, da sind andere gemeint, die isset()s sind noch da.


Ich habe eine Amazon-Wishlist. wink Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

Offline

Board footer

up