Toon posts:

[Forum] Vinyl topic rendert URL in een LIST niet in de TS

Pagina: 1
Acties:
  • 343 views

  • franssie
  • Registratie: Februari 2000
  • Laatst online: 21:02

franssie

Save the albatross

Topicstarter
Mrs Redhawk in "Het Grote Vinyl Topic Deel 3"

In de topic start staan een aantal lijstjes met platen winkels (H3 Adressen retail winkels)

Bij de plaatsen waar er meerdere winkels per plaats zijn, zijn deze in een LIST geplaatst maar dat rendert niet.
(FF en IE).
Kopieer ik dat stuk uit de source en plak ik het in een reactie, dan werkt het wel. In notepad kom ik er niet uit en @Mrs Redhawk weet ook zo 123 wat er fout gaat. Niet noodzakelijk een bug maar een tot nu toe niet begrepen eigenschap door een typefout oid maar toch, waar te zoeken?

[Voor 250% gewijzigd door franssie op 24-11-2018 16:15]

🎸 Niets is zo permanent als een tijdelijke oplossing | Een goed probleem komt nooit alleen | Gibson guitar Fender Guitar God Damn Guitar


  • Hero of Time
  • Registratie: Oktober 2004
  • Laatst online: 20:49

Hero of Time

Moderator LNX

There is only one Legend

Het heeft even geduurd met testen, maar het komt omdat je er boven bullet items gebruikt zonder list. Dan gaat het om een of andere reden stuk. Zet de opsomming eens in een list en kijk of het dan wel werkt.

Ik kan het zo even snel niet in een reactie reproduceren.

Commandline FTW | Tweakt met mate


  • crisp
  • Registratie: Februari 2000
  • Laatst online: 21:16

crisp

Devver

Pixelated

Het lijkt inderdaad alsof de parser op enig moment probeert alles in een enkel list-item te renderen. Dat effect komt waarschijnlijk door het inconsistent gebruik van list-items binnen en buiten [list]-tags. Ik heb nu alles binnen [list]-tags geplaatst en dan gaat het in ieder geval goed. Je moet alleen nog even een paar linkjes naar platomania fixen ;)

Intentionally left blank


  • crisp
  • Registratie: Februari 2000
  • Laatst online: 21:16

crisp

Devver

Pixelated

Dit is overigens de kleinste testcase die ik kan maken waarbij het fout gaat:
code:
1
2
3
[*]Not in a list
[list][*]In a list[/list] 
[*]Not in a list

Ik zal eens kijken wat daar nu precies fout gaat in de parser :)

Intentionally left blank


  • crisp
  • Registratie: Februari 2000
  • Laatst online: 21:16

crisp

Devver

Pixelated

Ik heb het probleem in de parser weten te fixen. Komt met de volgende release mee :)

Intentionally left blank


  • franssie
  • Registratie: Februari 2000
  • Laatst online: 21:02

franssie

Save the albatross

Topicstarter
Dan even de tweaker vraag, waarom werkte het niet in de topic start, maar ging het wel goed in een reactie?
Snel opgepakt en opgelost voor iets wat ik rond half twee in de nacht plaatste _/-\o_

🎸 Niets is zo permanent als een tijdelijke oplossing | Een goed probleem komt nooit alleen | Gibson guitar Fender Guitar God Damn Guitar


  • crisp
  • Registratie: Februari 2000
  • Laatst online: 21:16

crisp

Devver

Pixelated

franssie schreef op zaterdag 24 november 2018 @ 13:25:
Dan even de tweaker vraag, waarom werkte het niet in de topic start, maar ging het wel goed in een reactie?
Snel opgepakt en opgelost voor iets wat ik rond half twee in de nacht plaatste _/-\o_
Het ging om de combinatie van delen van de post. Eea heeft te maken met de afwijkende nesting-rules wanneer een list-item binnen of buiten een [list]-tag wordt gebruikt.

Intentionally left blank


  • Hero of Time
  • Registratie: Oktober 2004
  • Laatst online: 20:49

Hero of Time

Moderator LNX

There is only one Legend

franssie schreef op zaterdag 24 november 2018 @ 13:25:
Dan even de tweaker vraag, waarom werkte het niet in de topic start, maar ging het wel goed in een reactie?
Snel opgepakt en opgelost voor iets wat ik rond half twee in de nacht plaatste _/-\o_
In je reactie deed je niet <bullet><list><bullet>, wat de trigger is voor de fout, zoals Crisp al liet zien hierboven. Doe je <bullet><list> dan gaat het nog, draai je het om, dan werkt het ook nog. Maar de specifieke combinatie dus niet.

Commandline FTW | Tweakt met mate


  • franssie
  • Registratie: Februari 2000
  • Laatst online: 21:02

franssie

Save the albatross

Topicstarter
Ik deed het overigens niet, dat MrsRedhawk.

Maar wat ik niet snapte (of ik begrijp jullie antwoorden verkeerd) is wat @Hero of Time ook opmerkte dat het niet reproduceerbaar leek in een reactie. Maar ik merk dat als ik de code nu in deze TS stop het wel werkt. Er leek dus een inconsequentie te zitten tussen het parsen van code in het eerste bericht en een reply.
Dus het is al opgelost of er zat al een rafetje in de code voor de sectie van fysieke winkels (ik had niet de hele TS getest).

Hoe dan ook het lijkt te werken, top. Mede namens de Vinyl fans <3

🎸 Niets is zo permanent als een tijdelijke oplossing | Een goed probleem komt nooit alleen | Gibson guitar Fender Guitar God Damn Guitar


  • Hero of Time
  • Registratie: Oktober 2004
  • Laatst online: 20:49

Hero of Time

Moderator LNX

There is only one Legend

Om er een voorbeeld voor te maken:
• Dit is een bullet buiten een list tag.
  • [*]En dit is dus met een list. [*][url=https://tweakers.net]Tweakers homepage link[/url]
• En om de bug te triggeren nog een bullet buiten de list.

De link in de list zou nu dus niet moeten werken. Zou een van de bullets voor of na de list er niet staan, dan gaat het wel goed.

Commandline FTW | Tweakt met mate


  • franssie
  • Registratie: Februari 2000
  • Laatst online: 21:02

franssie

Save the albatross

Topicstarter
Dat deel was me al duidelijk maar dank!

🎸 Niets is zo permanent als een tijdelijke oplossing | Een goed probleem komt nooit alleen | Gibson guitar Fender Guitar God Damn Guitar


  • Hero of Time
  • Registratie: Oktober 2004
  • Laatst online: 20:49

Hero of Time

Moderator LNX

There is only one Legend

franssie schreef op zondag 25 november 2018 @ 15:47:
Dat deel was me al duidelijk maar dank!
Nee, je snapte niet waarom het in een reactie wel gewoon werkte. En dat heb ik juist aangetoond dat het daar ook gewoon gebeurt. ;)

Toen ik zei het niet zo snel te kunnen reproduceren was omdat ik niet de volledige omstandigheden om de bug te triggeren uitvoerde. Ik had een bullet voor een list en meer niet. Ik had nog een bullet erna moeten doen, maar tijdens het testen lette ik meer op wat er voor stond dan erna. :)

Commandline FTW | Tweakt met mate


  • franssie
  • Registratie: Februari 2000
  • Laatst online: 21:02

franssie

Save the albatross

Topicstarter
Ah zo, dus in mijn test had ik een te klein stukje code gebruikt, duidelijk dank!

🎸 Niets is zo permanent als een tijdelijke oplossing | Een goed probleem komt nooit alleen | Gibson guitar Fender Guitar God Damn Guitar


  • crisp
  • Registratie: Februari 2000
  • Laatst online: 21:16

crisp

Devver

Pixelated

Zojuist is ook de fix voor de onderliggende issue online gekomen. Probleem zou dus niet meer voor mogen komen :)

Intentionally left blank

Pagina: 1


Tweakers maakt gebruik van cookies

Tweakers plaatst functionele en analytische cookies voor het functioneren van de website en het verbeteren van de website-ervaring. Deze cookies zijn noodzakelijk. Om op Tweakers relevantere advertenties te tonen en om ingesloten content van derden te tonen (bijvoorbeeld video's), vragen we je toestemming. Via ingesloten content kunnen derde partijen diensten leveren en verbeteren, bezoekersstatistieken bijhouden, gepersonaliseerde content tonen, gerichte advertenties tonen en gebruikersprofielen opbouwen. Hiervoor worden apparaatgegevens, IP-adres, geolocatie en surfgedrag vastgelegd.

Meer informatie vind je in ons cookiebeleid.

Sluiten

Toestemming beheren

Hieronder kun je per doeleinde of partij toestemming geven of intrekken. Meer informatie vind je in ons cookiebeleid.

Functioneel en analytisch

Deze cookies zijn noodzakelijk voor het functioneren van de website en het verbeteren van de website-ervaring. Klik op het informatie-icoon voor meer informatie. Meer details

janee

    Relevantere advertenties

    Dit beperkt het aantal keer dat dezelfde advertentie getoond wordt (frequency capping) en maakt het mogelijk om binnen Tweakers contextuele advertenties te tonen op basis van pagina's die je hebt bezocht. Meer details

    Tweakers genereert een willekeurige unieke code als identifier. Deze data wordt niet gedeeld met adverteerders of andere derde partijen en je kunt niet buiten Tweakers gevolgd worden. Indien je bent ingelogd, wordt deze identifier gekoppeld aan je account. Indien je niet bent ingelogd, wordt deze identifier gekoppeld aan je sessie die maximaal 4 maanden actief blijft. Je kunt deze toestemming te allen tijde intrekken.

    Ingesloten content van derden

    Deze cookies kunnen door derde partijen geplaatst worden via ingesloten content. Klik op het informatie-icoon voor meer informatie over de verwerkingsdoeleinden. Meer details

    janee