-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
[4.1] Simplify statistic plugin #35983
Conversation
Remove "only once" data Recreate text.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also update the JavaScript code (media/plg_system_stats/js) of this plugin to reflect the "once" removal.
I have tested this item ✅ successfully on 669cf52 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35983. |
Good find, thx, is fixed. |
The english changes here are not very good and they are sending the wrong message. I agree about removing the send once but the rest is not correct. Also it is not clear what happens to sites that have previously said "once" when the plugin is updated. |
thanks for the feedback, could you elaborate on that? What would you change?
Nothing, as they sent it already and now the plugin is disabled. |
then the quality of the stats will not improve |
Not for the old one, but for the new one. It's all about sending regular data. "Once" is a "no" for regular data. I personaly would have prefered a opt-out solution and collecting more (anonymous) data, but this was just not possible. So the solution is to get more people clicking on "yes" by being more detailed. |
Honestly its a waste of time. This will have minimal impact on improving the statistics and their usefulness. More people will not be clicking on yes because the majority of people will never see it. The only way to do that would be to reset the plugin status when this new version is installed and ask EVERYONE the question. |
I like the idea to reset the current status of the plugin if it is set to "once". |
My2cent: 2 buttons, left no, right yes are a good solution. Eyes (in ltr countries) go from left to right and stay on the most right button. This is why all consent messages have the "yes" button right. So I think that changing the order of buttons will have an impact. Deciding between 2 alternatives is easy, 3 options make the user think. A normal user cannot know what's the difference between options always and once. What means "once"? |
I have tested this item ✅ successfully on 40ee732 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35983. |
PLG_SYSTEM_STATS_BTN_NEVER_SEND="Never" | ||
PLG_SYSTEM_STATS_BTN_SEND_ALWAYS="Always" | ||
PLG_SYSTEM_STATS_BTN_NEVER_SEND="No" | ||
PLG_SYSTEM_STATS_BTN_SEND_ALWAYS="Yes, I'll help Joomla!" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PLG_SYSTEM_STATS_BTN_SEND_ALWAYS="Yes, I'll help Joomla!" | |
PLG_SYSTEM_STATS_BTN_SEND_ALWAYS="Yes, I'll send statistics" |
I'm I the only one that would the yes button expect on the left side? Maybe I'm brain washed by windows but the primary button is always left (on windows) I think mac is diffrent? |
Its all irrelevant if we dont reset the plugin |
Thats another argumend raised by me on the discussion before so its not just you... |
Do we have the right to reset such thing? The decison to only send them once (while it does not make much sense) is a decision we have to respect right? Now its gone and the most accurate replacement is the new "no" option, IMO we should have disabled the plugin on the "no" and "only once" button click in the first place to be correct so here just reset the thing. If we want to I would say the only thing we should do would be to ask for a new decision on the "only once" users but keep the "no" decision intact. |
PLG_SYSTEM_STATS_MSG_JOOMLA_WANTS_TO_SEND_DATA="To better understand our install base and end user environments it is helpful if you send some site information back to a Joomla! controlled central server. No identifying data is captured at any point. You can change these settings later from Plugins → System - Joomla! Statistics." | ||
PLG_SYSTEM_STATS_MSG_WHAT_DATA_WILL_BE_SENT="Select here to see the information that will be sent." | ||
PLG_SYSTEM_STATS_MSG_JOOMLA_WANTS_TO_SEND_DATA="We want to increase the compatibility of our Joomla! software with our user's server settings. Therefore we need anonymous data from your site to better understand the install base and end user environments. All data is anonymised and only sent to a Joomla! controlled central server. You can change these settings later from Plugins → System - Joomla! Statistics." | ||
PLG_SYSTEM_STATS_MSG_WHAT_DATA_WILL_BE_SENT="No identifying data is captured at any point. Select here to review." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You know that atleast in germany the IP is interpreted as identifying data so just tell them that technically we collect the IP but do not use that.
And as noted above the "no" button should go and disable the plugin all together there is no point in running that plugin all the time when its not sending anything at all. |
I think the plugin should use the task scheduler... ok sorry a bit offtopic^^ Resetting "only once" should be ok, because this options doesn't exists anymore. The other options should be as they are now. On the other side you can argument that "only once" means for us "no" after this... don't care if it's only for new installation it's fine for me too. |
The IP of the server is not PII just the users internet connection IP |
@Shubhamverma2796 I can see a positive test from you counted for this pull request, but it seems you have not used the "Test this" button in the issue tracker to submit your test result, you have used the "Alter test" button, so we can see that only in the issue tracker but not on GitHub. If you have really tested this PR, then please use the "Test this" button in the issue tracker https://issues.joomla.org/tracker/joomla-cms/35983 to submit your test result. In the issue tracker and on GitHub you will see than a comment like this: The "alter test" button is meant to correct a previously submitted test result, e.g. in case if you have forgotten to select the appropriate result before submitting. Thanks in advance. |
I have tested this item ✅ successfully on 236b5eb This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35983. |
1 similar comment
I have tested this item ✅ successfully on 236b5eb This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35983. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35983. |
|
So I thought a lot over if we should reset the decisions the user made (at least for "once") or not. So I will go with the current version. |
Your choice of course but the plugin is still producing (or more accurately not producing) any usable statistics |
I don't disagree, but that was not the intention of this PR. Probably for 4.2 a complete rewrite could happen. @fancyFranci @roland-d |
A rewrite is fine but first we need a description of what it should be doing, otherwise there is no use doing a rewrite. We need to have a list of requirements that the plugin should fulfill. If we do that, are we actually using the data collected... |
I still disagree with the texts using suggestive language, its a bad sign to me that we think we have to use such suggestive language to gather questionable statistics.. :( |
Just to point out that while the joomla stats only show 3% on j4. |
Jup but that is not fixed by just changing the language and removing the once option. And as mentiond by Nic the serverside (of our statistics) have to be changed too like invalidate old reports etc. And we have to keep in mind that even when its widly used akeeba is not installed on all sites but maybe more specificly used on well maintained sites. |
I've not said remove the once option. Just that we should ask again (even if they said once in the past) for a major new release. I thought the serverside part of the stats had been updated sometime ago to invalidate any reports from more than x ago Of course akeeba stats will be skewed towards the higher end. But its a good indicator that the stats we have and supposedly use to determine various things are way off |
I'm not aware of any such change on the serverside but lucky me not everything goes via my desk. :-D By the commit logs we only had php and joomla version changes latly. |
I must have been thinking of this joomla/statistics-server#60 |
Possibly :-) |
joomla/joomla-cms#30522 + joomla/joomla-cms#32223 + joomla/joomla-cms#31675 + joomla/joomla-cms#35378 + joomla/joomla-cms#35612 + joomla/joomla-cms#35715 + joomla/joomla-cms#35610 + joomla/joomla-cms#35607 + joomla/joomla-cms#35788 + joomla/joomla-cms#35647 + joomla/joomla-cms#35143 + joomla/joomla-cms#36135 + joomla/joomla-cms#35998 + joomla/joomla-cms#36173 + joomla/joomla-cms#36212 + joomla/joomla-cms#36208 + joomla/joomla-cms#36206 + joomla/joomla-cms#36205 + joomla/joomla-cms#36203 + joomla/joomla-cms#36192 + joomla/joomla-cms#36191 + joomla/joomla-cms#36228 + joomla/joomla-cms#36211 + joomla/joomla-cms#36271 + joomla/joomla-cms#36270 + joomla/joomla-cms#36245 + joomla/joomla-cms#36294 + joomla/joomla-cms#36244 + joomla/joomla-cms#36242 + joomla/joomla-cms#36296 + joomla/joomla-cms#36190 + joomla/joomla-cms#36474 + joomla/joomla-cms#36297 + joomla/joomla-cms#36480 + joomla/joomla-cms#36479 + joomla/joomla-cms#36551 + joomla/joomla-cms#36366 + joomla/joomla-cms#36589 + joomla/joomla-cms#36583 + joomla/joomla-cms#36328 + joomla/joomla-cms#36515 + joomla/joomla-cms#36555 + joomla/joomla-cms#36653 + joomla/joomla-cms#36660 + joomla/joomla-cms#36657 + joomla/joomla-cms#36637 + joomla/joomla-cms#35983 + joomla/joomla-cms#36704 + joomla/joomla-cms#36708 + joomla/joomla-cms#36700 +
I agree with @brianteeman about reset for all, as the plugin will be changed. If the message is clear about why Joomla needs stats, it will have more effect, and some may say yes when they click no by the past. |
Based on the minutes from 19th October 2021 I recreated the statistic plugin:
Before
Now