* Begin reworking permission management
* Tests
* This.. tested functionality that didn't exist in the app
* Fixes
* Added permission checks for the system module
* Implemented extension settings permission
The "extension" settings page will now check for permission before allowing a staff member to even visit the settings page for a client.
FOSSBilling will also check for permission before saving a new config so the API can't be used to bypass this limitation.
I also removed a test as it was checking a function which only ever had one possible return value, no point in updating it to work with this PR
* This should fix the test
* I got a little mixed up with that, but this should work
* Fixed the tests
* Implemented give all & revoke all buttons
* Implemented more, enhancements, & bugfix
* Rename function
* Fix tests
* Optimize, address comment
* Now let's make the tests happy again
* Deduplicate code, cleanup a bit
* Bugfix
* Added selectbox support to the front-end
* Quick bugfix
* Fix issue PHPStan detected
* Bug fix
* Improved page layout & styling
* Implemented more checks
* Fix the tests
* Update error.html.twig
* Cleanup
* Fix the tests
* Updated the design to be more space efficient
* Don't list module access when always given
* Updated the exception type
* setMethod -> onlyMethods
* I donno how that got goofed up
* Fix the test, email module should always be accessible
* Removed the broken test
It's.. not really checking anything anyways. Just that the `extension` module is in a hardcoded array.
* Update rector config to include tests
* Run Rector against the tests
* Delete test-results
* Update .gitignore
* Correctly set the PHP min to 8.1, split the configs
* Corrected PHPDoc
* Deleted tests we can't really get to pass
PHPunit is hard-coded to fail when a class has a return type of `never`, which is the correct return type of our redirect function (and rightfully so).
No way to disable the exception as far as we can tell
* Update PHPUnit to version 10
Update composer and CI version of PHPUnit to 10, and migrate XML config to updated schema.
* Update tests for PHPUnit 10
Update tests to replace depreciated functions in PHPUnit 10. Also, remove risky tests which don't assert anything.
* Merge Kb and Support modules
Merge the Kb and Support modules as they contain related functionality.
* General improvements to Support module UI
Update Support module copy and navigation to be more consistent, and other general improvements.
* This was missed.. and now it's not 😉
---------
Co-authored-by: Belle Aerni <belleaerni@gmail.com>
* Began implementing Sentry.io support
* Added to the installer
* Added the ID to the bug report template and UI
* Hook it into our error handler, added the ability to filter by error code
* Attempting to add the DSN via GH actions
* Spelling and making PHPstan happy
* Skip exceptions for various incomplete configurations
Also brings in some of the category suggestions made by @Anuril in #195
* Update tests to account for changes
* Added & renamed exception classes
* Fix tests
* Change a bunch of items to be InformationExceptions
* Use a lighter HTTP client, fixed API error reporting
* I guess that's not needed now?
Arighty then
* Improvements, additional info added
* Update SentryHelper.php
* Improved error logging, added instance ID to error page
Also made it so that errors are correctly sent to Sentry rather than exclusively exceptions
* Make the tests happy
* Added the link, improved wording
Also removed the "performance" part of the reporting as it needs to be added to the front-end and I'm not currently implementing that
* Began implementing UI alerts and info
* Added the ability to toggle, fixed tests, use path constants
* Added some useful info & links to the page
* These should be InformationException
* Fix the automated Sentry DSN entry
* Fix the install number text
* Minor cleanup, converted more to InformationException
* Replaced some direct config references with constants
We have the BB_DEBUG constant to indicate if we have debug mode enabled or not, so we shouldn't be directly referencing it in the config file unless we actually need to do so.
Using the constant means there's less locations to update the reference if the config structure is changed
* Bugfix
* Consistency & fixed a PHPDoc
* Corrections
* We actually do want exceptions thrown here
* Change how we get the instance ID
* Stability improvements
Having support for translatable error pages was proving to be problematic, so I had to remove it.
I also had to update the exception class to handle cases where the trans function wasn't setup yet (needed for the /run-patcher page if an error occurs)
* Missed this change on git
* Also fix the variable names...
* Re-added new deps after rebase
* This should also accept "Error" types
* Added more info to the bug report and removed a dug line
* This wasn't supposed to get renamed
* Updated some missed references to Box_Exception
* Caught a typo
* New ExtensionDirectory class
* Tests
* Tests and error handling
* Stuff
* Literally the same outcome anyway. Will write this in the future.
* Removed tests for a function that does nothing
* "stable" -> "release"
* Fix misnamed exception errors
* Update class name to ExtensionManager
---------
Co-authored-by: Adam Daley <admdly@users.noreply.github.com>
* Refactor Box/Tools to remove unnecessary functions
Refactor Box/Tools to remove functions which essentially duplicate PHP functionality, or provide a thin wrapper around such functions.
* Move config migrate/update to relevant class
Move config update functionality from Box/Tools to Box/Update.
* Move Gravatar function to class where used
Moved Gravatar function to Twig filter where it is used, rather than in a custom Box/Tool function.
* Replace usage of array_get
* Fixes
* And a few more
* Remove remaining references
* Update Service.php
* Few more fixes
* Fix.. I think?
* Fix Client / Invoice updates
* Fix stats
* Fix issues noticed during code review
* More fixes
* And some more fixes :)
* Replace guzzlehttp/guzzle with symfony/http-client
Replace guzzlehttp/guzzle with symfony/http-client to standardise around one globally available HTTP client.
* Replace Box\Curl library and native cURL with symfony/http-client
Replace the Box\Curl library and native cURL with symfony/http-client to standardise around one well supported HTTP client, reduce unnecessary code, and improve compatibility.
* Remove obsolete functionality from Spamchecker module
Remove obsolete functions from Spamchecker module which depended upon cURL.
* Fix typo in Email module tests
Fix missing semicolon in tests for Email module causing PHPUnit to fail.
* Fix module test errors due to RedBean
Fix module tests causing error due to RedBean when being mocked in PHPUnit.
* Fix module test failures
Update module tests to fix failures, and remove obsoleted tests causing failure.
* Move RedBeanPHP to composer
* Remove missed references to rb.php
* This *should* fix the tests
* Update DummyBean.php
* Remove payment adapter tests
They.. honestly didn't test much and are proving to cause issues when moving rb.php to composer
