[Feat] Alias weglaten uit de active topics

[NMe]

Zie -NMe- in "Nieuwe versie van GoT is er!" en de post die ik quote. Het lijkt me nogal loos om in het geval van een alias altijd twee entries in de AT te hebben die beiden naar hetzelfde topic linken. Kunnen aliassen niet beter verborgen worden uit de AT?

[chem]

En wat als je een alias naar een sg-topic in med zet, en iemand sg als filter heeft?

[Outerspace]

Dan is die alias toch niet belangrijk, als het naar SG verwezen wordt. Of denk ik nu te logisch?

[moto-moi]

Je snapt het probleem niet.

Als ik alle fora's in m'n AT laat zien kun je eenvoudig de aliassen eruit mieteren
maar als je een forum blocked kan dat niet zo eenvoudig, aangezien je misschien het originele topic /of/ de alias blocked.
Volgens mij is het met sql wel op te lossen, maar het worden geen leuke leesbare queries gok ik

[NMe]

Ik weet niet hoe de queries eruit zien, dus don't shoot me, maar een group by doen op het topic-id waar je naar linkt lijkt me niet zo'n probleem. Alleen wordt het wat lastig om te bepalen welk topic/alias je laat zien. Als je een filter hebt op het forum waar het topic staat, dan moet je een alias zien, maar andersom moet je het originele topic zien. Toch wat lastiger dan ik dacht.

[Spider.007]

Misschien leuk voor later; maar voor nu iets te ingewikkeld

[ Voor 93% gewijzigd door Spider.007 op 31-07-2006 15:39 ]

[Spider.007]

FYI: Dit is al aan te passen door Tweakers.net; maar heeft wel de complicatie dat er eea verandert als je filtered-forums hebt.

$settings['data_overwrite']['list_activetopics']['show_aliases'] = FALSE;

[ Voor 25% gewijzigd door Spider.007 op 01-08-2006 09:56 ]

[NMe]

In LA stoort men zich er ook aan trouwens. Op zich wel terecht met de manier waarop ik in de Devschuur een topic onder de aandacht wil brengen, maar wat mij betreft is één keer in de AT staan voldoende.

RaZ in "Nieuwe versie van GoT is er!"

[iH8]

daar doelen jullie op toch? wilde er ook al een topic overmaken. ziet er verdomde raar uit zo en is in mijn ogen niet echt functioneel.

[RaZ]

Ik hou me hart vast als ze Harde Waren gaan voorzien van een feedback draadje, dat zijn er zelfs 6 ipv 5 als in devschuur

Of zelfs 9 voor Softe Goederen

[ Voor 21% gewijzigd door RaZ op 01-08-2006 18:41 ]

[the_stickie]

ingewikkelde query
Kijken of de url die je wil gebruiken al voorkomt, dat kan toch niet zo'n ingewikkelde query zijn?

[ Voor 12% gewijzigd door the_stickie op 01-08-2006 19:13 ]

[NMe]

The_Stickie schreef op dinsdag 01 augustus 2006 @ 19:12:
ingewikkelde query
Kijken of de url die je wil gebruiken al voorkomt, dat kan toch niet zo'n ingewikkelde query zijn?

Er komt echt wel meer bij kijken dan dat, zeker als je inderdaad ook nog eens rekening moet houden met forumfilters. Er staan geen URL's in de database.

[Hoedje]

Ik vind dit inderdaad ook wel wat ver gaan. Ik hoop dat het teruggebracht kan worden naar één draadje in de active topics, of anders maar helemaal filteren, zodat ze helemaal niet meer in de active topics komen te staan.

[the_stickie]

-NMe- schreef op dinsdag 01 augustus 2006 @ 19:14:
[...]

Er komt echt wel meer bij kijken dan dat, zeker als je inderdaad ook nog eens rekening moet houden met forumfilters. Er staan geen URL's in de database.

Daarom laat ik het devven over aan iemand anders

[Soultaker]

Ik kan me niet voorstellen dat het echt moeilijk is. Desnoods doe je het filteren ná het ophalen van de database (waardoor er misschien wat minder topics in de active topics lijst staan), maar naar de gebruiker toe is het gewoon zinnig om dubbele topics maar één keer te laten zien. (De vraag is dan natuurlijk welke je laat zien, maar in principe maakt dat weinig uit; of is het mogelijk dat een alias een andere titel heeft dan het topic dat 'ie aliast?)

Ik kan me ook voorstellen dat je een optie maakt om aliases helemaal niet te laten zien. De alias kun je dan zien als een link en die wordt zelf niet geüpdatet natuurlijk. Die situatie is vergelijkbaar met die waarin je in plaats van een alias een gesloten topic met een link erin hebt.

[ Voor 23% gewijzigd door Soultaker op 01-08-2006 19:31 ]

[NMe]

Soultaker schreef op dinsdag 01 augustus 2006 @ 19:30:
of is het mogelijk dat een alias een andere titel heeft dan het topic dat 'ie aliast?)

Dat is mogelijk ja, al zie ik niet echt een situatie waarin dat zinnig is.

[Eskimootje]

Als je gewoon zorgt dat de topic id's maar eenmaal mogen voorkomen in de query DISTINCT dan zal er altijd een topic zijn waar je rechten in hebt en maximaal 1. Lijkt me prima toch?

[crisp]

Spider.007 schreef op dinsdag 01 augustus 2006 @ 09:31:
FYI: Dit is al aan te passen door Tweakers.net; maar heeft wel de complicatie dat er eea verandert als je filtered-forums hebt.

$settings['data_overwrite']['list_activetopics']['show_aliases'] = FALSE;

Dat lijkt me voor nu wel een afdoende oplossing?

[NMe]

Eskimootje schreef op dinsdag 01 augustus 2006 @ 23:02:
Als je gewoon zorgt dat de topic id's maar eenmaal mogen voorkomen in de query DISTINCT dan zal er altijd een topic zijn waar je rechten in hebt en maximaal 1. Lijkt me prima toch?

Distinct gaat over unieke records, niet unieke velden.

crisp schreef op dinsdag 01 augustus 2006 @ 23:03:
[...]

Dat lijkt me voor nu wel een afdoende oplossing?

Lijkt mij op zich ook wel, al zou een permanente oplossing die de forumfilters in overweging neemt beter zijn natuurlijk.

[Gamo]

Ik sluit me aan bij het feit dat al die blauwe DEV mapjes weg mogen van mij... Overal staan zelfde aantal posts in, dus ik zie geen enkele meerwaarde.

Als jullie aant bespreken zijn hoe dit te doen, dan biedt ik eerlijk mijn apologies aan, want daar heb ik geen koek van gegeten

Geweldige update overigens

[ Voor 34% gewijzigd door Gamo op 02-08-2006 01:36 ]

[crisp]

Spider.007 schreef op dinsdag 01 augustus 2006 @ 09:31:
FYI: Dit is al aan te passen door Tweakers.net; maar heeft wel de complicatie dat er eea verandert als je filtered-forums hebt.

$settings['data_overwrite']['list_activetopics']['show_aliases'] = FALSE;

jammer dat React niets met die setting doet bij het aanroepen van get_activetopics()

[Spider.007]

Er moest nog een data>option worden toegevoegd; wat ik zojuist heb gedaan

Committed revision 5056.