Anzahl Top 10 erweiterbar

Kahmoon commented 10 years ago

Hi,

ich hab im Forum dazu schon einen Thread aufgemacht. Hier noch der Feature Request dazu. https://community.contao.org/de/showthread.php?47042-Top-10-auf-Top-20-erweitern

Es wäre super wenn man die Top 10 per Varibale in der localconfig.php oder Backendfeld erweitern könnte. In meinem Fall möchte ich in einem Blog (http://www.frankdaniels.de/blog.html) gerne mehr als nur 10 der häufigsten Tags verwenden. Ich denke das wäre es sehr cooles Feature :-)

Tags Version 3.1.1 stable 5

vg Frank

hschottm commented 10 years ago

Hi Frank, du hast deinen Pull Request leider auf den master branch gemacht, der sich mittlerweile ziemlich verändert hat. Du müsstest das entweder auf den contao/3.1 branch machen oder in der jetzigen Form mit deiner eigenen Erweiterung leben. Für Contao 3.2 habe ich die Tags jetzt so erweitert, dass die Anzahl der Top Tags in der Tag Cloud eingegeben werden kann.

Kahmoon commented 10 years ago

Hi,

oh sorry…war mein erstes Ticket auf Github. Kannst du das verschieben?

Problem ist das ich 3.2 noch nicht verwende. Auch nicht in absehbarer Zeit.

:: Gruß Frank Berger

Dipl. Multimedia Designer

Phon: +49 173 3512271

Skype: frberger

Mail: berger@mediastuff.de

Net: http://www.mediastuff.de http://www.mediastuff.de/

Von: Helmut Schottmüller [mailto:notifications@github.com] Gesendet: Mittwoch, 15. Januar 2014 11:27 An: hschottm/tags Cc: Kahmoon Betreff: Re: [tags] Anzahl Top 10 erweiterbar (#5)

Hi Frank, du hast deinen Pull Request leider auf den master branch gemacht, der sich mittlerweile ziemlich verändert hat. Du müsstest das entweder auf den contao/3.1 branch machen oder in der jetzigen Form mit deiner eigenen Erweiterung leben. Für Contao 3.2 habe ich die Tags jetzt so erweitert, dass die Anzahl der Top Tags in der Tag Cloud eingegeben werden kann.

— Reply to this email directly or view it on GitHub https://github.com/hschottm/tags/pull/5#issuecomment-32349682 . https://github.com/notifications/beacon/6390770__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNTMxNDM5OSwiZGF0YSI6eyJpZCI6MjM0MTU1MjN9fQ==--eadd48407e562d99f3eeef3bb8a90e0d46e5c461.gif

hschottm commented 10 years ago

Ehrlich gesagt weiß ich auch nicht, wie das geht, ich habe noch nie einen Pull Request erstellt :-) Ich schau mal, ob ich das für 3.1 ebenfalls noch einbaue.

Kahmoon commented 10 years ago

Servus,

dann willkommen im Club J

Mir reicht eigentlich auch die Anpassung in der PHP TagList.php PHP. Leider scheint heir noch irgendwo anders eine Begrenzung zu sein. Zahlen kleiner 10 gehen….größer nicht.

            public function getTopTenTagList()

            {

                           $list = $this->getTagList();

                           usort($list, array($this, "tag_asort"));

                           if (count($list) > 10) $list = array_reverse(array_slice($list, -15, 15));

                           return $list;

            }

:: Gruß Frank Berger

Dipl. Multimedia Designer

Phon: +49 173 3512271

Skype: frberger

Mail: berger@mediastuff.de

Net: http://www.mediastuff.de http://www.mediastuff.de/

Von: Helmut Schottmüller [mailto:notifications@github.com] Gesendet: Mittwoch, 15. Januar 2014 14:40 An: hschottm/tags Cc: Kahmoon Betreff: Re: [tags] Anzahl Top 10 erweiterbar (#5)

Ehrlich gesagt weiß ich auch nicht, wie das geht, ich habe noch nie einen Pull Request erstellt :-) Ich schau mal, ob ich das für 3.1 ebenfalls noch einbaue.

— Reply to this email directly or view it on GitHub https://github.com/hschottm/tags/pull/5#issuecomment-32361739 . https://github.com/notifications/beacon/6390770__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNTMyNTk3NywiZGF0YSI6eyJpZCI6MjM0MTU1MjN9fQ==--6d8e82b13eb884af44166782d2a2d62328f4d418.gif

hschottm commented 10 years ago

Machst du aber eigentlich richtig so. Du solltest nur if (count($list) > 15) $list = array_reverse(array_slice($list, -15, 15)); schreiben, weil es nur Sinn macht, die Liste zu reduzieren, wenn sie mindestens die gewünschte Größe hat. Aber ich werd mal sehen, dass ich das noch in die 3.1 einbaue, waren nicht so viele Änderungen.

Kahmoon commented 10 years ago

Oh ja..falscher code.

So hab ichs aktuell: if (count($list) > 10) $list = array_reverse(array_slice($list, -15, 15));

Und ich hab mit Sicherheit mehr als 10 Tags gesamt. Deshalb hat es mich gewundert das $list nur 10 Werte hat.

:: Gruß Frank Berger

Dipl. Multimedia Designer

Phon: +49 173 3512271

Skype: frberger

Mail: berger@mediastuff.de

Net: http://www.mediastuff.de http://www.mediastuff.de/

Von: Helmut Schottmüller [mailto:notifications@github.com] Gesendet: Mittwoch, 15. Januar 2014 14:55 An: hschottm/tags Cc: Kahmoon Betreff: Re: [tags] Anzahl Top 10 erweiterbar (#5)

Machst du aber eigentlich richtig so. Du solltest nur if (count($list) > 15) $list = array_reverse(array_slice($list, -15, 15)); schreiben, weil es nur Sinn macht, die Liste zu reduzieren, wenn sie mindestens die gewünschte Größe hat. Aber ich werd mal sehen, dass ich das noch in die 3.1 einbaue, waren nicht so viele Änderungen.

— Reply to this email directly or view it on GitHub https://github.com/hschottm/tags/pull/5#issuecomment-32362935 . https://github.com/notifications/beacon/6390770__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNTMyNjkyNSwiZGF0YSI6eyJpZCI6MjM0MTU1MjN9fQ==--acddb3c5283253c27c55f8fb3ce83217e5ac372f.gif

hschottm commented 10 years ago

Sollte aber gehen. $list ist ja nur die nach Größe sortierte komplette Tag-Liste... Bei mir hat das auch problemlos geklappt.

Kahmoon commented 10 years ago

Hmm muss ein Tag eine bestimmet Anzahl an Verwendungen haben? Es gibt sicher einige die nur einmalig verwendet wurden.

:: Gruß Frank Berger

Dipl. Multimedia Designer

Phon: +49 173 3512271

Skype: frberger

Mail: berger@mediastuff.de

Net: http://www.mediastuff.de http://www.mediastuff.de/

Von: Helmut Schottmüller [mailto:notifications@github.com] Gesendet: Mittwoch, 15. Januar 2014 15:59 An: hschottm/tags Cc: Kahmoon Betreff: Re: [tags] Anzahl Top 10 erweiterbar (#5)

Sollte aber gehen. $list ist ja nur die nach Größe sortierte komplette Tag-Liste... Bei mir hat das auch problemlos geklappt.

— Reply to this email directly or view it on GitHub https://github.com/hschottm/tags/pull/5#issuecomment-32368354 . https://github.com/notifications/beacon/6390770__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNTMzMDczNCwiZGF0YSI6eyJpZCI6MjM0MTU1MjN9fQ==--429b4293d48083701fefc80650add4237789c600.gif

hschottm commented 10 years ago

Nö, eigentlich nicht. Du kannst ja mal die aktuelle Tags-Version für Contao 3.1.x aus dem Repository updaten. In den Tag-Clouds kannst du jetzt die Anzahl der Top Tags angeben, ich hab das eben eingestellt. In meinen Tests hat das funktioniert. Ich hab nur leider grad immer nur mit Mini Tag Clouds getestet. Ich lege mir grad noch mal 'ne große an....

Kahmoon commented 10 years ago

Cool Danke. Leider zeigt es auch damit nur 10 Stück an L. Kein Plan warum.

www.frankdaniels.de/blog.html

:: Gruß Frank Berger

Dipl. Multimedia Designer

Phon: +49 173 3512271

Skype: frberger

Mail: berger@mediastuff.de

Net: http://www.mediastuff.de http://www.mediastuff.de/

Von: Helmut Schottmüller [mailto:notifications@github.com] Gesendet: Mittwoch, 15. Januar 2014 17:00 An: hschottm/tags Cc: Kahmoon Betreff: Re: [tags] Anzahl Top 10 erweiterbar (#5)

Nö, eigentlich nicht. Du kannst ja mal die aktuelle Tags-Version für Contao 3.1.x aus dem Repository updaten. In den Tag-Clouds kannst du jetzt die Anzahl der Top Tags angeben, ich hab das eben eingestellt. In meinen Tests hat das funktioniert. Ich hab nur leider grad immer nur mit Mini Tag Clouds getestet. Ich lege mir grad noch mal 'ne große an....

— Reply to this email directly or view it on GitHub https://github.com/hschottm/tags/pull/5#issuecomment-32374658 . https://github.com/notifications/beacon/6390770__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNTMzNDQxNSwiZGF0YSI6eyJpZCI6MjM0MTU1MjN9fQ==--98b9f71dc83e5466aeda85c773d7e0e73701b2f7.gif

hschottm commented 10 years ago

Was verwendst du denn? Tag Cloud oder eines von den Tag Cloud Spezialmodulen?

Kahmoon commented 10 years ago

Modul Tag Cloud. Alle Auszeichnungen sind deaktiviert ausser in News. Sehr seltsam…

:: Gruß Frank Berger

Dipl. Multimedia Designer

Phon: +49 173 3512271

Skype: frberger

Mail: berger@mediastuff.de

Net: http://www.mediastuff.de http://www.mediastuff.de/

Von: Helmut Schottmüller [mailto:notifications@github.com] Gesendet: Mittwoch, 15. Januar 2014 17:12 An: hschottm/tags Cc: Kahmoon Betreff: Re: [tags] Anzahl Top 10 erweiterbar (#5)

Was verwendst du denn? Tag Cloud oder eines von den Tag Cloud Spezialmodulen?

— Reply to this email directly or view it on GitHub https://github.com/hschottm/tags/pull/5#issuecomment-32375871 . https://github.com/notifications/beacon/6390770__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNTMzNTExNCwiZGF0YSI6eyJpZCI6MjM0MTU1MjN9fQ==--46137d4ad997fc90f48d31c912fe4b08052ab324.gif

Kahmoon commented 10 years ago

Mir fällt gerade auf das ich tags_news gar nicht installiert habe. Könnte es daran liegen?

:: Gruß Frank Berger

Dipl. Multimedia Designer

Phon: +49 173 3512271

Skype: frberger

Mail: berger@mediastuff.de

Net: http://www.mediastuff.de http://www.mediastuff.de/

Von: Helmut Schottmüller [mailto:notifications@github.com] Gesendet: Mittwoch, 15. Januar 2014 17:12 An: hschottm/tags Cc: Kahmoon Betreff: Re: [tags] Anzahl Top 10 erweiterbar (#5)

Was verwendst du denn? Tag Cloud oder eines von den Tag Cloud Spezialmodulen?

— Reply to this email directly or view it on GitHub https://github.com/hschottm/tags/pull/5#issuecomment-32375871 . https://github.com/notifications/beacon/6390770__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNTMzNTExNCwiZGF0YSI6eyJpZCI6MjM0MTU1MjN9fQ==--46137d4ad997fc90f48d31c912fe4b08052ab324.gif

hschottm commented 10 years ago

Sind das die Tags, die du in deinem Blog verwendest? Kann es vielleicht sein, dass du bei maximale Anzahl von Auszeichnungen einfach 10 reingeschrieben hast? Dann nimmt er natürlich nur maximal 10 Auszeichnungen... Die beliebten Auszeichnungen, die du da hast sind ja dann nur eine gefilterte Liste. Die Top 10 (jetzt ja Top) Auszeichnungen müssen weiter unten aktiviert werden. Die werden dann zusätzlich oberhalb der eigentlichen Auszeichnungsliste angezeigt. Wenn du eh nur eine Auswahl der xxx beliebtesten Tags anzeigen lassen willst, dann musst du doch nur bei maximale Anzahl von Auszeichnungen 15 oder 20 oder was auch immer angeben, dann wird die Tag Cloud darauf begrenzt :-) Dann hätte ich mir das ja auch sparen können. Na, macht ja nix.

Kahmoon commented 10 years ago

Oh mein Gott….die Anzahl war wirklich auf 10 gestellt. Das Feld hatte ich gar nicht mehr beachtet da ich dachte es betrifft die Top 10 nicht. Sorry! Es geht!! Danke!

:: Gruß Frank Berger

Dipl. Multimedia Designer

Phon: +49 173 3512271

Skype: frberger

Mail: berger@mediastuff.de

Net: http://www.mediastuff.de http://www.mediastuff.de/

Von: Helmut Schottmüller [mailto:notifications@github.com] Gesendet: Mittwoch, 15. Januar 2014 17:21 An: hschottm/tags Cc: Kahmoon Betreff: Re: [tags] Anzahl Top 10 erweiterbar (#5)

Sind das die Tags, die du in deinem Blog verwendest? Kann es vielleicht sein, dass du bei maximale Anzahl von Auszeichnungen einfach 10 reingeschrieben hast? Dann nimmt er natürlich nur maximal 10 Auszeichnungen... Die beliebten Auszeichnungen, die du da hast sind ja dann nur eine gefilterte Liste. Die Top 10 (jetzt ja Top) Auszeichnungen müssen weiter unten aktiviert werden. Die werden dann zusätzlich oberhalb der eigentlichen Auszeichnungsliste angezeigt. Wenn du eh nur eine Auswahl der xxx beliebtesten Tags anzeigen lassen willst, dann musst du doch nur bei maximale Anzahl von Auszeichnungen 15 oder 20 oder was auch immer angeben, dann wird die Tag Cloud darauf begrenzt :-) Dann hätte ich mir das ja auch sparen können. Na, macht ja nix.

— Reply to this email directly or view it on GitHub https://github.com/hschottm/tags/pull/5#issuecomment-32376870 . https://github.com/notifications/beacon/6390770__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNTMzNTY4NywiZGF0YSI6eyJpZCI6MjM0MTU1MjN9fQ==--5d20b19182f36939c42fec51435837953bc7602c.gif

Kahmoon commented 10 years ago

Nene die Anpassung war schon gut so für Leute die nur die Top 10 beeinflussen wollen J

Danke noch mal und schönen Abend.

:: Gruß Frank Berger

Dipl. Multimedia Designer

Phon: +49 173 3512271

Skype: frberger

Mail: berger@mediastuff.de

Net: http://www.mediastuff.de http://www.mediastuff.de/

Von: Helmut Schottmüller [mailto:notifications@github.com] Gesendet: Mittwoch, 15. Januar 2014 17:21 An: hschottm/tags Cc: Kahmoon Betreff: Re: [tags] Anzahl Top 10 erweiterbar (#5)

Sind das die Tags, die du in deinem Blog verwendest? Kann es vielleicht sein, dass du bei maximale Anzahl von Auszeichnungen einfach 10 reingeschrieben hast? Dann nimmt er natürlich nur maximal 10 Auszeichnungen... Die beliebten Auszeichnungen, die du da hast sind ja dann nur eine gefilterte Liste. Die Top 10 (jetzt ja Top) Auszeichnungen müssen weiter unten aktiviert werden. Die werden dann zusätzlich oberhalb der eigentlichen Auszeichnungsliste angezeigt. Wenn du eh nur eine Auswahl der xxx beliebtesten Tags anzeigen lassen willst, dann musst du doch nur bei maximale Anzahl von Auszeichnungen 15 oder 20 oder was auch immer angeben, dann wird die Tag Cloud darauf begrenzt :-) Dann hätte ich mir das ja auch sparen können. Na, macht ja nix.

— Reply to this email directly or view it on GitHub https://github.com/hschottm/tags/pull/5#issuecomment-32376870 . https://github.com/notifications/beacon/6390770__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNTMzNTY4NywiZGF0YSI6eyJpZCI6MjM0MTU1MjN9fQ==--5d20b19182f36939c42fec51435837953bc7602c.gif

hschottm commented 10 years ago

Dann ist das ja erledigt...

hschottm / tags

Anzahl Top 10 erweiterbar #5