Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Module\Rpost: Just a little bit of doc. | Harald Eilertsen | 2024-06-13 | 1 | -0/+6 |
| | |||||
* | Module\Rpost: Redirect to submitted post on success. | Harald Eilertsen | 2024-06-13 | 1 | -10/+1 |
| | | | | | | This eliminates a open redirect issue where it was possible to craft a link that when clicked would take the victim to an external site controlled by an attacker. | ||||
* | Fix warnings exposed by tests. | Harald Eilertsen | 2024-06-13 | 4 | -8/+11 |
| | | | | Mainly missing variables for templates, and channel entries. | ||||
* | Module\Rpost: Add return type and visibilty for `get`. | Harald Eilertsen | 2024-06-13 | 1 | -1/+1 |
| | |||||
* | tests: More tests for Module\Rpost. | Harald Eilertsen | 2024-06-13 | 1 | -14/+44 |
| | | | | Also refactor the tests a bit to avoid duplicatng code. | ||||
* | Module\Rpost: Refactor handling of attachments. | Harald Eilertsen | 2024-06-13 | 1 | -68/+79 |
| | | | | Move to private function for now. | ||||
* | Module\Rpost: Reuse value of local_chanel. | Harald Eilertsen | 2024-06-13 | 1 | -2/+4 |
| | | | | We don't need to call it twice (actually trice in the original code). | ||||
* | Module\Rpost: Refactor redirect or login logic. | Harald Eilertsen | 2024-06-13 | 1 | -26/+37 |
| | |||||
* | tests: Module\Rpost shows login form if not authenticated. | Harald Eilertsen | 2024-06-13 | 1 | -0/+13 |
| | |||||
* | tests: Add comment to Tests\Unit\Module\RpostTest. | Harald Eilertsen | 2024-06-13 | 1 | -0/+4 |
| | |||||
* | tests: Set query string in Module\TestCase::get method. | Harald Eilertsen | 2024-06-13 | 1 | -0/+1 |
| | |||||
* | tests: Configure system.baseurl for tests. | Harald Eilertsen | 2024-06-13 | 2 | -7/+5 |
| | |||||
* | Module\Rpost: Remove obsolete local variable $o. | Harald Eilertsen | 2024-06-13 | 1 | -10/+1 |
| | |||||
* | Module\Rpost: Remove unused local variables. | Harald Eilertsen | 2024-06-13 | 1 | -4/+1 |
| | |||||
* | Module\Rpost: Add basic test and fix session access. | Harald Eilertsen | 2024-06-13 | 2 | -1/+32 |
| | | | | | | | | | | | | | Just a basic test to ensure that the module `get()` method behaves somewhat reasonable when no query params are given. Had to make a small change to the Rpost module itself. Since the `$_SESSION` superglobal may not always be set (and is not in the test), use `isset` instead of `array_key_exists` to check if we have saved query params in the session. In general, isset is safer than array_key_exists if there's a chance that the array itself may not exist. | ||||
* | docu and remove superfluous break statement after return statement | Mario | 2024-06-13 | 1 | -3/+1 |
| | |||||
* | Merge branch 'add-config-for-phpcs' into 'dev' | Mario | 2024-06-13 | 1 | -0/+78 |
|\ | | | | | | | | | Add config file and rules for PHP Code Sniffer. See merge request hubzilla/core!2135 | ||||
| * | Add config file and rules for PHP Code Sniffer. | Harald Eilertsen | 2024-06-12 | 1 | -0/+78 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The rules are based on the "Generic" ruleset included by PHP Code Sniffer, with a significant portion of the rules disabled. This is a tradeoff between getting some useful feedback, and not being overloaded by noise. I've tried to encode a coe style that resembles the existing code as much as possible, but have included some sniffs that requires code changes to satisfy the style. This is meant as a starting point, and we can disable or enable more sniffs as we see fit. PHPCS also has ready rule sets for other common coding standards we may want to gravitate towards, e.g. PSR-12. Others are available from the community. The best way to run PHPCS is to integrate it with your editor, so that it will display diacnostics inline when saving or modifying the code. It can also be run from the command line like this: ./vendor/bin/phpcs -n [<path-to-file-to-check>] If no file is specified it will try to check the entire project. The `-n` means don't bother with warnings (I recommend that to begin with. Enable the warnings when the errors are taken care of.) | ||||
* | | Merge branch 'fix-test-warnings' into 'dev' | Mario | 2024-06-13 | 6 | -14/+21 |
|\ \ | | | | | | | | | | | | | Fix test warnings See merge request hubzilla/core!2134 | ||||
| * | | Module\Setup: Don't access static variable as non static. | Harald Eilertsen | 2024-06-12 | 1 | -3/+3 |
| | | | |||||
| * | | Module\Help: Only variables can be passed by reference. | Harald Eilertsen | 2024-06-12 | 1 | -1/+2 |
| | | | | | | | | | | | | | | | | | | Introduce an intermediate variable when extracting the file type from the file name. Otherwise we would try to pass a returned value as a reference. | ||||
| * | | Module\Rbmark: Pass all fields to input field templates. | Harald Eilertsen | 2024-06-12 | 1 | -3/+3 |
| | | | |||||
| * | | boot/login: Pass all expected args to sub templates. | Harald Eilertsen | 2024-06-12 | 1 | -3/+3 |
| | | | |||||
| * | | Module\Rbmark: Specify all fields in the template. | Harald Eilertsen | 2024-06-12 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | The `field_select` sub template wants five elements in the `field` array. | ||||
| * | | includes/menu: Fix timestamp handling in menu_create. | Harald Eilertsen | 2024-06-12 | 1 | -3/+4 |
| | | | | | | | | | | | | | | | Referencing undefined array keys are not allowed anymore, so we need to check whether they exist first. | ||||
| * | | Module\Setup: Pass all required params for the template. | Harald Eilertsen | 2024-06-12 | 2 | -1/+5 |
| |/ | | | | | | | | | | | Non-existing keys in the array passed to the template causes a warning. Also make optional parts of the template actually optional by skipping them if the value is empty. | ||||
* | | remove not needed namespace | Mario | 2024-06-11 | 1 | -2/+2 |
| | | |||||
* | | remove not required includes and include security.php in boot.php | Mario | 2024-06-11 | 2 | -8/+2 |
| | | |||||
* | | cleanup unused code | Mario | 2024-06-11 | 1 | -11/+0 |
| | | |||||
* | | pass the force argument to the xchan_photo daemon | Mario | 2024-06-11 | 2 | -4/+7 |
| | | |||||
* | | duplicate array key | Mario | 2024-06-10 | 1 | -1/+0 |
| | | |||||
* | | too many args | Mario | 2024-06-10 | 1 | -2/+2 |
| | | |||||
* | | fix wrong variable | Mario | 2024-06-10 | 1 | -1/+1 |
| | | |||||
* | | remove superfluous backslash | Mario | 2024-06-10 | 2 | -4/+4 |
| | | |||||
* | | Ãfix undefined variable | Mario | 2024-06-10 | 1 | -1/+1 |
| | | |||||
* | | remove duplicate array key | Mario | 2024-06-10 | 1 | -1/+0 |
| | | |||||
* | | docu | Mario | 2024-06-10 | 1 | -2/+2 |
| | | |||||
* | | docu | Mario | 2024-06-10 | 1 | -2/+2 |
| | | |||||
* | | fix updated not supported in wrapper function | Mario | 2024-06-10 | 2 | -3/+3 |
| | | |||||
* | | docu and declare return type for t() | Mario | 2024-06-10 | 1 | -2/+2 |
| | | |||||
* | | remove superfluous param, fix wrong var and declare types for unparse_url() | Mario | 2024-06-10 | 2 | -5/+5 |
| | | |||||
* | | remove unused variable and superfluous backslash | Mario | 2024-06-10 | 1 | -3/+1 |
| | | |||||
* | | update changelog | Mario | 2024-06-07 | 1 | -0/+2 |
| | | |||||
* | | update changelog | Mario | 2024-06-07 | 1 | -0/+9 |
| | | |||||
* | | move button class to the right dom | Mario | 2024-06-07 | 2 | -4/+4 |
| | | |||||
* | | adjusting to the wrong direction when exporting ical and do not use Z ↵ | Mario | 2024-06-05 | 1 | -2/+2 |
| | | | | | | | | because timezone is attached | ||||
* | | bump version | Mario | 2024-06-05 | 1 | -1/+1 |
| | | |||||
* | | Merge branch 'correct-type-annotation-for-config-get' into 'dev' | Mario | 2024-06-05 | 1 | -1/+1 |
|\ \ | | | | | | | | | | | | | Correct type annotation in comment for Config::Get. See merge request hubzilla/core!2127 | ||||
| * | | Correct type annotation in comment for Config::Get. | Harald Eilertsen | 2024-06-05 | 1 | -1/+1 |
| |/ | |||||
* | | Merge branch 'disable-mfa-for-dav-and-cdav' into 'dev' | Mario | 2024-06-05 | 4 | -2/+119 |
|\ \ | | | | | | | | | | | | | Skip checking MFA status for WebDAV and CardDAV requests. See merge request hubzilla/core!2131 |