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

Minimalstatistiken #1440

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Minimalstatistiken #1440

wants to merge 4 commits into from

Conversation

hansmorb
Copy link
Contributor

@hansmorb hansmorb commented Dec 6, 2023

Verwendung:

Im Fließtext einbauen, keinen eigenen Shortcode Block. Dann [cb_statistics do=Funktion type=item|location|booking]

Es gibt die Funktion count, die zählt alle Vorkomnisse und dann noch die mächtigen Funktionen count_xxx sum_xxx und avg_xxx , das xxx ist dabei entweder der Rückgabewert einer Funktion des jeweiligen Models oder wenn es eine solche Funktion nicht gibt der Metawert. Damit sollten sich alle möglichen Statistiken zusammenstecken lassen. Aber natürlich lässt sich damit auch Schabernack treiben, z.B. könnte jemand den Shortcode [cb_statistics do=cancel type=booking] anlegen, dann würden alle Buchungen im System storniert werden. Die Implementierung hier ist dafür aber nicht sonderlich schwierig zu implementieren und neue Möglichkeiten für Statistiken können einfach durch neue Methoden in der Model Klasse eingerichtet werden.

Beispiel:

Aktuell haben wir [cb_statistics do=count type=item] Artikel und [cb_statistics do=count type=location] Standorte. Von allen Buchungen die wir haben sind [cb_statistics do=count_isConfirmed type=booking] bestätigte Buchungen. Die durschnittliche Postleitzahl ist [cb_statistics do=avg__cb_location_postcode type=location].

@hansmorb hansmorb linked an issue Dec 6, 2023 that may be closed by this pull request
@hansmorb hansmorb added enhancement New feature or request php Pull requests that update Php code labels Dec 6, 2023
@hansmorb hansmorb marked this pull request as draft December 6, 2023 00:45
Copy link

codecov bot commented Dec 6, 2023

Codecov Report

Attention: 45 lines in your changes are missing coverage. Please review.

Comparison is base (637cb31) 39.76% compared to head (6d8bbac) 39.57%.

Files Patch % Lines
src/View/Statistics.php 0.00% 44 Missing ⚠️
src/Plugin.php 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1440      +/-   ##
============================================
- Coverage     39.76%   39.57%   -0.19%     
- Complexity     2324     2341      +17     
============================================
  Files            91       92       +1     
  Lines          9602     9647      +45     
============================================
  Hits           3818     3818              
- Misses         5784     5829      +45     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@datengraben
Copy link
Contributor

Finde den Ansatz mit den Shortcodes sehr cool. Ich habe auch mal was begonnen, das geht aber direkt ins Dashboard und zeigt sowas an wie "Buchungen der letzten 7 Tage (versus der 7 Tage davor)". Ich versuche ob ich das mit deinem Code verbinden bzw. integrieren kann.

Der Shortcode hier stellt aber erstmal nur eine Zahl dar oder?
Interessant wären ja eher so nach Zeitraum aggregierte Auswertungen.

Mit dem von dir gewählten Interface in den Shortcodes baust du prinzipiell ja entweder SQL oder BI-Tools nach. Vielleicht findet man in dem Bereich ja interessante Implementierungs-Ideen oder Lösungen. Da müssen wir das Rad nicht neu erfinden. Denn beim Zusatz __xxx bei do:

[...] entweder der Rückgabewert einer Funktion des jeweiligen Models oder wenn es eine solche Funktion nicht gibt der Metawert.

... könnte ich mir vorstellen das es schnell unübersichtlich werden kann.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Minimalstatistiken
2 participants