| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | |
|
| | |
|
|\ \
| |/
|/|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This brings another tool in to help us keep the code in order - Psalm.
(I thought the name was fitting! :)
This will do fairly simple static analysis of the code, and report
problems and suggest fixes. It can help fix some issues itself, but
please double check that it does the right thing.
More info: https://psalm.dev
This merge also brings in fixes that was suggested by Paslm. Mostly this
is typa annotations for functions, but also some bugfixes discovered by
the tool.
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
|/ |
|
|
|
|
| |
Modified returnuser function to not allow a user to assign multiple slots to themselves
|
|
|
|
| |
CSS for edit form in giglog
|
|
|
|
| |
Added test to create duplicate concert with varied cases in string
|
|\
| |
| |
| | |
https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Restructure the method a bit, drop the `c`prefix on variables, rename
the variable to hold the created concert, and use object notation to
pass the attributes to the constructor.
Also rename the method `get` to `find`, the only call site was the
`create` method. Drop the unnecessary method `check_duplicate`. Just use
`find` instead.
|
| | |
|
|\|
| |
| |
| | |
https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges
|
| |
| |
| |
| |
| |
| |
| | |
Since running the bash script on windows was a bit troublesome, make the
`composer test` command a first class citizen. Still don't know how to
run a specific test case using the composer command, but at least it
should be easy to run the full test suite.
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | | |
We probably need some better error handling here. There's a myriad of
reasons why this call could fail, and we might need to communicate the
failure reason somewhere.
|
| | | |
|
| |/
|/|
| |
| | |
before merging latest changes
|
|/
|
|
| |
modified assign/unassing/returnuser functions to use dynamic column name instead of 4 different codes for each assignment type
|
| |
|
| |
|
|
|
|
|
| |
Also moved dropping of band foreign key from concerts table into the db
version 5 changeset.
|
|\ |
|
| |
| |
| |
| | |
Modified so that band is no longer in use
|
| | |
|
| | |
|
|\| |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
Particularly useful for running a specific test case:
For example:
./run-tests --filter Band::testCreateBandWithName
|
|/
|
|
|
|
| |
The expected attributes did not have names corresponding with the
table columns, which meant that creating a band directly from a returned
table row did not produce the expected result.
|
|\ |
|
| | |
|
| | |
|
| |\
| | |
| | |
| | | |
https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges
|
| |\ \
| | | |
| | | |
| | | | |
https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges
|
| | | |
| | | |
| | | |
| | | | |
Set up a test env before running the test cases.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Sidenote: UK is not included in the country list. Did the brexit
everything?
|
| |_|/
|/| |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This makes it easier to run the test script directly from the editor it
your editor supports composer. Otherwise run it from the command line:
composer run test
Or like before:
./run-tests
|
| | |
| | |
| | |
| | |
| | |
| | | |
Not sure if it's a good idea to have `create` return an existing band.
Will have to look at callsites to see if it should be renamed back or if
the callsite should be changed.
|
| | | |
|
|\| |
| |/
|/| |
|
| |
| |
| |
| | |
Refactored band
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
After much reading I finally found the magic incantations, so now we can
run tests with real database access. This means we no longer need the
primitive $wpdb_stub.
The setup as now _requires_ wp-env, or an environment set up
sufficiently similar. Running in wp-env is the easiest, so aim for that.
I've added a `run-tests` script that will invoke the magic incantation
without having to remember it every time.
To set up for testing:
1. make sure you have composer[1] installed.
2. run `composer install`
3. make sure you have wp-env[2] installed
4. start the wordpress env: `wp-env start`
5. run the tests: `./run-tests`
Let the thousand tests bloom!
[1]: https://github.com/wp-phpunit/wp-phpunit
[2]: https://www.npmjs.com/package/@wordpress/env
|
| | |
|
| | |
|