You are not logged in.

[Easy-WI] FastDownload

  • 1
  • 2

Terrorkarotte

Super Moderator

Posts: 708

  • Send private message

21

Monday, September 2nd 2013, 9:33pm

Hatte einen Zahlendreher bei der Zeilenangabe drinne. Sorry, dass ich Verwirrung gestiftet habe und danke @Lacrimosa99 fürs entdecken.
Webbasierender Config Ersteller: www.ulrich-block.de für CS 1.6, CSS, DODS und TF2.

Ebenso wird werden verschiedene Debian Gameserverkernel zum Download angeboten.

MadMakz

Super Moderator

Posts: 1,878

Location: ~#

wcf.user.option.userOption53: Ja

  • Send private message

22

Monday, September 2nd 2013, 10:03pm

Will dafür kein eigenes Topic aufmachen aber Icons für Protectioncheck-Status fehlen. Ob da jemand was basteln möchte falls die wegen GPL Problemen entfernt wurden? :)

PS: Hat GitHub keinen Bugtracker mehr oder ist der nicht Aktiv in dem Projekt?

Lacrimosa99

Intermediate

Posts: 275

Location: Leipzig

Occupation: IT-Systemintegrator

wcf.user.option.userOption53: Ja

  • Send private message

23

Monday, September 2nd 2013, 10:06pm

Muss nochmal leider was schreiben... hab meinen Beitrag editiert aufgrund dessen, dass das Forum die Backslashes "gefressen" hatte :evil2: .. funktionierender Code steht nun in meinem vorherigen Post ;)



Bara

Professional

Posts: 834

wcf.user.option.userOption53: Ja

  • Send private message

24

Monday, September 2nd 2013, 10:29pm

Quoted from "MadMakz"

Will dafür kein eigenes Topic aufmachen aber Icons für Protectioncheck-Status fehlen. Ob da jemand was basteln möchte falls die wegen GPL Problemen entfernt wurden? :)

PS: Hat GitHub keinen Bugtracker mehr oder ist der nicht Aktiv in dem Projekt?

https://github.com/easy-wi/developer/iss…=1&state=closed
Issue dürfte dir hier helfen ;)

MadMakz

Super Moderator

Posts: 1,878

Location: ~#

wcf.user.option.userOption53: Ja

  • Send private message

25

Monday, September 2nd 2013, 10:50pm

GitHub ist nicht immer so ganz übersichtlich^^

Wie wäre es noch mit einem codeing standart?
Z.B.:

Source code

 
1
2
3
4

        if (isset($serverip)) {
            $template_file="userpanel_gserver_fdl_es.tpl";
        } else {
            $template_file='userpanel_404.tpl';


Einmal single quotes, einmal doubble-quotes. Gibt es mehrere sachen von.

Wenn mehr Leute ihre Hand drann setzen könnt das noch dreckig enden. Ich habe in meinem zeug zwar auch meisitens einen misch masch aber bei so einem großen und komplexen gebilde wäre das evtl angebracht. Ich helfe gerne dabei das global umzusetzen.

This post has been edited 1 times, last edit by "MadMakz" (Sep 2nd 2013, 11:07pm)

Terrorkarotte

Super Moderator

Posts: 708

  • Send private message

26

Tuesday, September 3rd 2013, 7:45am

Hi Madmakz,

die double quotes sind in aller Regel alter code > 2 Jahre alt. Ich bemühe mich sowas anzupassen, wenn ich Codestellen wegen Bugs, oder neuen Funktionen aufräumen muss. Oft war es aber leider so, dass man entscheiden musste, ob man ein Release raus haut, weil alle quengeln, oder aber sich viel Zeit fürs Refactoring nimmt.

Die Betroffene Code Stelle würde ich derzeit so schreiben:

PHP Source code

 
1

$template_file=(isset($serverip)) ? 'userpanel_gserver_fdl_es.tpl' : 'userpanel_404.tpl';


Gerade mal mit notepad++ gesucht:

Source code

 
1

Search ".tpl"" (119 hits in 43 files)


Ich habe gerade mal den aktuellen Entwicklungsstand von mir committed. Offen sind noch 2 Pull Requests, die ich mir noch näher anschauen muss, da sie Core Funktionen betreffen.

Wenn du dich der Sache annehmen willst, bin ich dankbar. Achte dann aber bitte darauf, dass man keine Variablen innerhalb von Single Quotes verwenden kann.
Webbasierender Config Ersteller: www.ulrich-block.de für CS 1.6, CSS, DODS und TF2.

Ebenso wird werden verschiedene Debian Gameserverkernel zum Download angeboten.

This post has been edited 1 times, last edit by "Terrorkarotte" (Sep 3rd 2013, 8:20am)

  • 1
  • 2

Similar threads