You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I rather disagree with this and I seem to recall the reason for this being opened originating from someone who simply had no understanding of how to actually work with PHP code and it's not our module's job to teach them that.
The existing examples are pretty well documented and not very complex. There's examples for using the database, calling on the API programmatically, and sending email templates which are all things that the majority of modules will want to do and having those examples is valuable. Someone who understands how to work with PHP (or even most programming languages) shouldn't really have issues following these example.
What the module doesn't do a good job of describing is what the DI is, how someone can see what the DI has available, as well as a general list of packages, classes, and tools that are available for them use. Those all need to be better, and getting that covered will also make the examples in the event examples a bit clearer for anyone who is a bit confused.
But at the end of the day, I don't think the event examples themselves are problematic at all and really we just need to do a better job of describing aspects like the DI which are fundamental to writing a module for FOSSBilling.
They are unnecessarily complex for a beginner.
We should make them simpler and demonstrate more common tools like the HTTP client.
The text was updated successfully, but these errors were encountered: