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

Warning: Undefined array key "action" in /weberp/reportwriter/admin/ReportCreator.php on line 54 #247

Open
pakricard opened this issue Nov 20, 2024 · 22 comments
Labels
To be actioned Issue has been accepted for review

Comments

@pakricard
Copy link
Collaborator

Report creator shows a warning on v 4.15.2 and also latest GitHub code with PHP 8:

Warning: Undefined array key "action" in /weberp/reportwriter/admin/ReportCreator.php on line 54

@timschofield
Copy link
Owner

timschofield commented Nov 23, 2024 via email

@dalers dalers added the To be actioned Issue has been accepted for review label Nov 24, 2024
@pakricard
Copy link
Collaborator Author

I am not using it at all, probably in the past 10 years, as the reporting capabilities are very limited.
If we haven't received many bugs from users or other developers, most likely is because it isn't used.
Should we get rid of it for v5?

@dalers
Copy link
Collaborator

dalers commented Nov 25, 2024

What problem was ReportCreator intended to solve? Was it a good solution? If ReportCreator is removed, how can the same goal be achieved?

I'm 100% in favor of removing code that doesn't provide useful functionality or where the same goal could be accomplished better with a separate app, especially if the code doesn't work and is difficult to fix and maintain. I just don't know enough about ReportCreator to have an opinion.

@pakricard
Copy link
Collaborator Author

pakricard commented Nov 25, 2024

ReportCreator creates simple user-generated reports. By “simple” I mean "really simple".

To me, it was not a suitable solution and ended up creating custom PDF reports every time we needed something not available in webERP, as this ReportCreator has plenty of limitations.

I am not using it, just reported the warning message as I am cleaning and checking my weberp.

If we have not received errors or complaints in the past 10 years, to me, is a strong indication that virtually no one is using it.
But if we vote to keep it, it should be without warnings or errors as next PHP update most probably it will stop working,

@timschofield
Copy link
Owner

timschofield commented Nov 25, 2024 via email

@pakricard
Copy link
Collaborator Author

Agree.

@dalers
Copy link
Collaborator

dalers commented Nov 26, 2024

OK, I support @pakricard and @timschofield and agree ReportWriter should be removed. Does that mean the issue label needs to be updated? To what?

@andrewcouling
Copy link
Collaborator

May I suggest that we attempt gather some insight from users on the Report Creator before deciding to remove a feature that some might use routinely. This could be done with a simple Google Form (or similar) sent out to the mailing lists. I don't use the Report Creator myself , I'm not overly concerned about the outcome, but feel this should be an informed decision. Thoughts?

@dalers
Copy link
Collaborator

dalers commented Nov 26, 2024

More information is always good, and it's not like there's a deadline for a decision. :-) Are you volunteering Andrew? ;-)

I propose we put a pause on taking any action until at least 2 weeks after Andrew emails the form to the mailing list. Those For? All Against?

In the meantime I'm going to at least read the ReportWriter section in the manual so I am better informed...

@andrewcouling
Copy link
Collaborator

Are you volunteering Andrew? ;-)

Yes, I'm happy to. I can draft a survey, and share a link here for review before publishing.

@dalers
Copy link
Collaborator

dalers commented Nov 26, 2024

Fwiw, from the git log: On Dec 19, 2005 @PhilDaintree added "Dave's Form Maker" (likely Dave Premo), and it doesn't seem to have changed substantially since except @pakricard "Revised report writer with the ability for more fields" in 2010.

The rest of the commits look to be compatibility commits mostly by @timschofield @pakricard @PhilDaintree @TurboPT and largely ending ~2018, with a couple commits to the .inc files in the past couple months by Paul Thursby (aka TurboPT?), @aingelc12ell and @timschofield.

@timschofield
Copy link
Owner

timschofield commented Nov 26, 2024 via email

@andrewcouling
Copy link
Collaborator

Here's a link to a draft survey. All comments and suggestion are welcome.

Once we've settled on the survey design I intend to send out the link to these mailing list, plus I'll put a post in GH Discussions. Any suggestions on how to extend our reach beyond these?

@dalers
Copy link
Collaborator

dalers commented Dec 5, 2024 via email

@timschofield
Copy link
Owner

timschofield commented Dec 5, 2024 via email

@dalers
Copy link
Collaborator

dalers commented Dec 5, 2024

@timschofield are you asking if the email addresses submitted to the survey can be coorelated with the email addresses subscribed to the mailing lists? What would be the goal? (not being critical, just curious what your thoughts are)

If the survey appeals to the community in general, perhap we will get an 80% response of those who are actively using and/or following webERP.

@timschofield
Copy link
Owner

timschofield commented Dec 5, 2024 via email

@andrewcouling
Copy link
Collaborator

Can we track hits on the form, to differentiate with those who fill it in? Thanks Tim

I can get link analytics via a service such as bitly.com. Google Forms itself doesn't provide anything like this.

@andrewcouling
Copy link
Collaborator

if the email addresses submitted to the survey can be corelated with the email addresses subscribed to the mailing lists?

The Google Form is currently set to automatically collect the email address of the respondent, but they must be signed into a Google account. I figured that the majority of people have a google account these days. Alternatively you can request manual email address input (this might be off-putting), or form submissions can be anonymous.

@timschofield
Copy link
Owner

timschofield commented Dec 5, 2024 via email

@andrewcouling
Copy link
Collaborator

A couple thoughts imho....

Thanks @dalers.... its great to get your input. I have worked with research designers in the past, but its really not my strength.
I have actioned your suggestions, and have tried to make the wording more neutral.
Happy to take further rounds of comments/suggestions.

@andrewcouling
Copy link
Collaborator

It's not a must have, I'm just a collector of statistics and so when I see a chance to get some I jump at it :)

If we do more of this kind of thing in the future, we could consider using an email campaign platform such a Mailchimp which would provide comprehensive analytics capabilities including email delivery and read status.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
To be actioned Issue has been accepted for review
Projects
None yet
Development

No branches or pull requests

4 participants