Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installer: Download-Anzahl, sortieren nach Download-Anzahl #5696

Merged
merged 5 commits into from
Jul 1, 2023
Merged

Installer: Download-Anzahl, sortieren nach Download-Anzahl #5696

merged 5 commits into from
Jul 1, 2023

Conversation

eaCe
Copy link
Contributor

@eaCe eaCe commented Apr 23, 2023

Erweitert den Installer um die Anzahl der gesamten Downloads eines Addons und der jeweiligen Version. Zusätzlich kann nach Downloads sortiert werden.

Ein Hinweis, dass die gesamten Downloads keine Aussage über die Qualität oder so etwas in der Art sind, habe ich nicht hinzugefügt, ich wüsste aktuell nicht wohin dieser sollte.

grafik

grafik

@eaCe
Copy link
Contributor Author

eaCe commented Apr 23, 2023

Oh, ich muss mich hier wohl noch um ein paar psalm Probleme kümmern

Copy link
Member

@gharlan gharlan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich denke, meist wenn man nach Veröffentlichung oder Downloads sortiert, möchte man jeweils absteigend sortieren. Daher würde ich bei erstem Klick entsprechend jeweils eher absteigend sortieren.

phpstan/psalm kann ich mir sonst auch am Ende anschauen (evtl nehme ich manchen Stellen auch einfach in die Baseline erstmal auf)

<th>' . $package->i18n('shortdescription') . '</th>
<th class="rex-table-action">' . $package->i18n('header_function') . '</th>
</tr>
</thead>
<tbody>';

foreach ($addons as $key => $addon) {
$downloads = array_sum(array_column($addon['files'], 'counter')) ?: 0;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Die Gesamtzahl müsste auch direkt in der API verfügbar sein, auf Addon-Ebene.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Die habe ich total übersehen, danke für den Hinweis. Das passe ich an. Bei Psalm habe ich ein paar Stellen gefunden, an denen ich die Kommentare verändern muss, aber dann muss ich die Baseline vermutlich neu erstellen.

@eaCe
Copy link
Contributor Author

eaCe commented May 17, 2023

Ich weiß nur nicht genau was ich mit psalm machen soll.
EDIT: Sortierung ist angepasst

@gharlan gharlan added this to the REDAXO 5.16 milestone Jul 1, 2023
@gharlan gharlan changed the title Download-Anzahl, sortieren nach Download-Anzahl Installer: Download-Anzahl, sortieren nach Download-Anzahl Jul 1, 2023
@gharlan gharlan enabled auto-merge (squash) July 1, 2023 10:54
@gharlan gharlan merged commit 9fd018e into redaxo:main Jul 1, 2023
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants