Opened 3 years ago

Closed 3 years ago

#2083 closed defect (fixed)

Mediardir - Im Feld "Link" werden Node platzhalter falsch gerendert

Reported by: markusliechti Owned by: kramer
Priority: high Milestone: Contrexx 3.2
Component: other(specify) Version: 3.1.1
Severity: normal Keywords: mediadir, link
Cc:

Description (last modified by markusliechti)

Fehlerbericht von Unitb:

Ich habe ein kleines Problemchen mit dem Mediaverzeichnis bzw. mit dem FeldTyp? Link.

Wenn ich den Link rendere mittels a)

<!-- BEGIN mediadir_inputfield_184 -->

[[MEDIADIR_INPUTFIELD_VALUE]]

<!-- END mediadir_inputfield_184 -->

oder b)

<!-- BEGIN mediadir_inputfield_184 -->

<a href="[[MEDIADIR_INPUTFIELD_VALUE_HREF]]">[[MEDIADIR_INPUTFIELD_VALUE_NAME]]</a>

<!-- END mediadir_inputfield_184 -->

bekomme ich bei internen Links (z. B. [[NODE_HOME]] als FieldValue?) defekte Links:

bei a) <a href="http:///de/Willkommen" class="mediadirInputfieldLink" target="_blank">/de/Willkommen</a>

bei b) <a href="http:///de/Willkommen">/de/Willkommen</a>

Dass der Linkstext dann jetzt für uns bei internen Links nicht unbedingt optimal ist, ist Geschmackssache und läßt sich ja mittels eines separaten TextFields? ändern :) Aber der defekte Link ist schon ungünstig :)

Die Stelle ist /modules/mediadir/lib/inputfields/link.class.php#157ff und wäre gefixt mit z. B.:

if (

stristr('[[NODE') === false && (

substr($strValueHref, 0,7) != "http://"

substr($strValueHref, 0,8) != "https://"

)

) {

Günstiger fände ich es, das Inputfield schon selbst unterscheided zwischen externem und internem Link und dann jeweils einen Input für HRef und Text bietet (für Letzteres ggf. auch den Default, wie aktuell) ... :)

Change History (5)

comment:1 Changed 3 years ago by markusliechti

  • Description modified (diff)

comment:2 Changed 3 years ago by markusliechti

  • Description modified (diff)

comment:3 Changed 3 years ago by kramer

  • Owner changed from comvation to kramer
  • Status changed from new to assigned

comment:4 Changed 3 years ago by kramer

  • Milestone changed from unknown to Contrexx 3.2

comment:5 Changed 3 years ago by kramer

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.