WBCE CMS Forum

WBCE CMS – Way Better Content Editing.

You are not logged in.

#1 15.07.2015 21:10:43

nibz
Member

WBCE template

Hi there all,

I thought i would also post something.
I installed WBCE and it's working good.

One note about the wbce template and short url, i think it would be nice to add the next to the template:

<?php if (file_exists(WB_PATH.'/.htaccess')) {?>
	[[SHORTURL]]
<?php } ?>

This checks if .htaccess exists and if it exists it adds the droplet to the page.
Because now when people 'activate' the .htaccess the urls on the page aren't 'clean', i know they are redirected but i like clean url's on the page also.

Offline

#2 15.07.2015 21:29:54

florian
Administrator

Re: WBCE template

+1
Thank you very much for this great idea. This is a very good solution!


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#3 21.07.2015 12:15:24

webbird
Administrator

Re: WBCE template

Did anyone add this to the repo at GitHub? Please note that I am not very active here and so I maybe miss such things...


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

#4 21.07.2015 12:40:25

webbird
Administrator

Re: WBCE template

Added this to index.php in template "wbce" -> https://github.com/WBCE/WebsiteBaker_CommunityEdition/blob/master/wb/templates/wbce/index.php

Untested!


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

#5 21.07.2015 14:17:58

florian
Administrator

Re: WBCE template

@webbird Thanks a lot. Neither I know how to work with Github yet (-.-), nor I have added any user there, so actually it's just up to you...


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#6 02.08.2015 17:56:11

norhei
Developer

Re: WBCE template

A .htaccess can be there for various reasons not just for routing.
Setting PHP version, setting file handlers , setting certain vars....

So i guess the only save way to activate routing is via a switch in the options as you never can say what this .htaccess file is for.

Offline

#7 01.11.2015 20:04:45

evaki
Guest

Re: WBCE template

missing </div> tag
<div class="center">

<div class="fg12 footer"><?php echo WEBSITE_FOOTER;?></div>
</div>
<div class="fg12 footer"><?php echo WEBSITE_FOOTER;?></div>
</div></div>

===========================
Message text not available. More: This "img" tag uses one or more entirely obsolete (in HTML5) attributes which must not be used in HTML5 documents. The HTML5 obsolete attributes for "img" include: "align", "border", "datafld", "datasrc", "hspace", "longdesc", "lowsrc", "name", and "vspace".

[== HTML5 ==]
	<a id="menu-button"><img src="http://localhost/templates/wbce/menu.png" alt="" width="32" height="32" border="0" /></a>

===========================

Sincerely Evaki

Last edited by evaki (02.11.2015 00:55:37)

#8 02.11.2015 11:51:31

florian
Administrator

Re: WBCE template

Oops. Fixed. Thank you for the hint.  ops


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#9 09.11.2015 21:47:45

evaki
Guest

Re: WBCE template

Thank you for the hint.

Trotzdem habe ich nur die Hälfte erzählt, z.B. die </br>-Tags zwar bei mir gemacht, aber das nicht mitgeteilt.
Das ist nicht so dolle hilfreich -aber ich bessere mich.
Bitte das Template durch den W3C-Validator laufen lassen, am besten alle beide
Bestens ist der Online CSE HTML Validator: http://www.onlinewebcheck.com/check.php

MfG. Evaki

Last edited by evaki (09.11.2015 23:01:34)

#10 10.11.2015 16:34:06

florian
Administrator

Re: WBCE template

Das WBCE-Template hat einen Bug, ein </div> ist in eine if-Abfrage gerutscht, wo es nicht hingehört. Anbei die korrigierte Version.

</br>-Tags

sagt mir gerade nichts. Es sind keine <br /> oder (falschen) </br>-Tags im Template. Kann das aus irgend einem Modul kommen?

--
The WBCE default template has a bug, a </div> tag was placed inside of an if/else-condition where it did not belong to. Please find the (hopefully) corrected version attached.


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#11 10.11.2015 16:50:45

evaki
Guest

Re: WBCE template

Irgendwo waren </br> statt <br> für den Zeilenumbruch eingesetzt, oder was zuviel, oder beides
Habs aber im Moment nicht parat.
Kannste ja mal https://validator.w3.org/nu/?doc=http%3 … %2Fwbce%2F
eingeben.
MfG. Evaki

Hab eben mal geschaut
Edit:  Du liegst richtig, es liegt am eingegebenen Text im expcounter

Und den  "</div> ist in eine if-Abfrage gerutscht" hatte ich auch noch nicht gesehen.

Deshalb meine Empfehlung die Templatetester zu nutzen -kost ja nix

Last edited by evaki (10.11.2015 16:56:09)

#12 10.11.2015 16:54:46

florian
Administrator

Re: WBCE template

;-)


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#13 14.11.2015 18:18:17

evaki
Guest

Re: WBCE template

Kannes sein, daß hier ein Fehler vorliegt?
\html\templates\wbce\index.php

[== HTML ==]
<?
// no direct file access
if(count(get_included_files())==1) header("Location: ../index.php",TRUE,301);
?><!DOCTYPE HTML>
<html lang="de-de" dir="ltr">
<head>

Solls nicht <?php
// no direct file access

lauten?

MfG. Evaki

#14 14.11.2015 18:38:45

cwsoft
Member

Re: WBCE template

Ab PHP 5.4 aufwärts sollte es keine grossen Probleme mehr bereiten, darunter hängts von den PHP Settings ab und ist eigentlich nicht zu empfehlen. Siehe Flussdiagramm auf SO: http://programmers.stackexchange.com/qu … tag-in-php

Gruss


Account inactive since 2018/11/17.

Offline

#15 14.11.2015 20:15:31

evaki
Guest

Re: WBCE template

Danke für die Bestätigung. Steht so in den Templates (v1.1beta).
WB-Classic/WBCE-Systemvoraussetzungen fangen halt mit PHP 5.3.6 an, und noch hab ich den ein oder anderen...  yikes  Dürfte sich aber in absehbarer Zeit endgültig erledigen.
MfG. Evaki

Last edited by evaki (14.11.2015 23:41:00)

#16 15.11.2015 17:15:04

florian
Administrator

Re: WBCE template

Wer hat denn das da reingeschrieben und wann? Ich wars nämlich nicht. yikes (verstehe die Befehle nicht mal.)
Short open tag wird zudem auch nicht überall unterstuetzt und sollte mmn nicht verwendet werden.


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#17 15.11.2015 17:29:19

evaki
Guest

Re: WBCE template

Bloß keine Panik, das gehört da schon rein -gugsu beta  smile
Wer seine Entwicklungsumgebung auf php5.4 aufwärts eingestellt hat, bekommt möglicherweise per def. den short open tag. Ist mir nur im Zusammenhang mit den Systemvoraussetzungen aufgefallen, da ich bei den Tests eben zwischen mehreren php-Versionen wähle bzw. wählen kann.
MfG. Evaki

Last edited by evaki (15.11.2015 17:31:13)

#18 15.11.2015 21:58:20

cwsoft
Member

Re: WBCE template

@Florian: Bei Fragen wie: wann, wo, wer, was, wie geändert hat, hilft Git blame. Norhei hat den shorttag Fehler an anderer Stelle schon mal diskutiert. Notfalls einfach mal nach shorttags in den PHP Dateien suchen. Geht mit Notepad++ oder Sublimetext innerhalb weniger Sekunden. Sitz grad nicht am Rechner, sonst hätte ich selbst schnell geschaut.

P.S.: Habe Norhei informiert. Denke ist beim update der count get_included_files Geschichte versehentlich mit reingerutscht.

Last edited by cwsoft (15.11.2015 22:07:18)


Account inactive since 2018/11/17.

Offline

#19 15.11.2015 23:06:56

norhei
Developer

Re: WBCE template

Das könnte ich gewesen sein ....

Was bedeuten die Befehle.
count(get_included_files())   halt Anzahl der includierten Dateien
Wenn die gleich 1 ist , ist keine andere Datei geladen , also nur diese , also direkter Dateizugriff.
Praktisch so ähnlich wie if (defined ("WB_PATH")).
Das funktioniert allerdings auch wenn wir mal die Konstantennamen alle umstellen und auch ausserhalb von WB(CE).
301 Redirect weil Google das lieber hat. Zuviele 302 auf einer Seite mag big G ned so doll.

Das Haupttemplate könnte(sollte?) eigentlich genauso geschützt werden wie alle anderen PHP Dateien. Das Short tag war ein Unfall.
Müsste man sich jetzt aber Gedanken machen ob das wirklich nötig ist weils die Templater doch ein wenig verwirren dürfte.
Persönlich würde ich keinen Einspruch erheben wenns wieder rausfliegt , aber wenn man mit der Sicherheit konsequent sein möchte, müsste man jedes Template File mit sowas füttern.   





Bin ich froh wenns php 5.4 wird...

Offline

#20 16.11.2015 09:01:19

florian
Administrator

Re: WBCE template

Was spricht denn gegen das bislang verwendete

[== PHP ==]
if (!defined('WB_PATH')) {
    die(header('Location: ../../index.php'));
}

Bin ich froh wenns php 5.4 wird...

Naja, da WBCE 1.1 ja ein Sicherheitsrelease ist, sollte das schon noch wie die 1.0 mit >=PHP 5.3.6 laufen.
Für 1.2 können wir ja dann Die Mindestversion auf 5.4  erhöhen.


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#21 16.11.2015 10:19:30

evaki
Guest

Re: WBCE template

Nichts spricht dagegen, vielmehr ist der Vorteil der neuen Variante zu sehen.
Mit v5.3.6 läuft auch dies, nur muß es dann eben <?php lauten. Das ist die einzige Bedingung.
Diese Kleinigkeit ist halt nur übersehen worden, wie andere Kleinigkeiten auch mal übersehen werden.
Wer nix macht, macht nur den Fehler, daß er nix macht.
MfG. Evaki

Last edited by evaki (16.11.2015 10:22:13)

#22 16.11.2015 10:21:25

cwsoft
Member

Re: WBCE template

Eigentlich spricht nichts dagegen. Allerdings sollte der Grund ein anderer als die Verwendung von Shorttags sein :-)


Account inactive since 2018/11/17.

Offline

Board footer

up