summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Add Psalm as dev dependencyHarald Eilertsen2021-05-082-2/+2129
|
* removed older icons. Added new ones created by meAndreaChirulescu2021-04-2510-16/+43
| | | | Modified returnuser function to not allow a user to assign multiple slots to themselves
* added venue test to check duplicate with all letter casesAndreaChirulescu2021-04-243-16/+63
| | | | CSS for edit form in giglog
* Added an uppercase in sql for venue and concert duplicate checkAndreaChirulescu2021-04-233-2/+32
| | | | Added test to create duplicate concert with varied cases in string
* Merge branch 'dev' of ↵AndreaChirulescu2021-04-231-40/+27
|\ | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges
| * Refactor Concert::createHarald Eilertsen2021-04-231-40/+27
| | | | | | | | | | | | | | | | | | | | 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.
* | Added run tests for windows usageAndreaChirulescu2021-04-231-0/+2
| |
* | Merge branch 'dev' of ↵AndreaChirulescu2021-04-234-12/+66
|\| | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges
| * Fix composer test command.Harald Eilertsen2021-04-221-1/+1
| | | | | | | | | | | | | | 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.
| * Merge remote-tracking branch 'andrea/andreaschanges' into devHarald Eilertsen2021-04-223-2181/+186
| |\
| * | Concert::create return NULL if concert already exist.Harald Eilertsen2021-04-223-12/+17
| | | | | | | | | | | | | | | | | | 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.
| * | Add some tests for Concert class.Harald Eilertsen2021-04-221-0/+49
| | |
* | | no idea what these changes are but got stuck with them, so just commiting ↵AndreaChirulescu2021-04-231-1652/+10
| |/ |/| | | | | before merging latest changes
* | changed oackage.json to use wp-env 4.0.0AndreaChirulescu2021-04-223-2181/+186
|/ | | | modified assign/unassing/returnuser functions to use dynamic column name instead of 4 different codes for each assignment type
* Fix wrong property name in Concert::cname()Harald Eilertsen2021-04-221-1/+1
|
* Activate db version 5Harald Eilertsen2021-04-221-1/+1
|
* Fix typo in register_db_table.Harald Eilertsen2021-04-221-11/+11
| | | | | Also moved dropping of band foreign key from concerts table into the db version 5 changeset.
* Merge remote-tracking branch 'andrea/andreaschanges' into devHarald Eilertsen2021-04-227-128/+73
|\
| * Added column for concert title in databaseAndreaChirulescu2021-04-217-138/+73
| | | | | | | | Modified so that band is no longer in use
| * added check that GiglogAdmin_Band::get_band returns somethingAndreaChirulescu2021-04-211-3/+13
| |
* | Typo in comment for Concert constructor.Harald Eilertsen2021-04-211-2/+2
| |
* | Merge remote-tracking branch 'andrea/andreaschanges' into devHarald Eilertsen2021-04-211-1/+4
|\|
| * Added an else to return something when concert is empty on return user functionAndreaChirulescu2021-04-211-0/+1
| |
| * Added check if emty concert when fetching assigned userAndreaChirulescu2021-04-211-1/+3
| |
* | run-tests: Pass any additional args to phpunit.Harald Eilertsen2021-04-211-1/+1
| | | | | | | | | | | | | | Particularly useful for running a specific test case: For example: ./run-tests --filter Band::testCreateBandWithName
* | Fix Bands class constructor.Harald Eilertsen2021-04-212-5/+12
|/ | | | | | 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.
* Merge remote-tracking branch 'andrea/andreaschanges' into devHarald Eilertsen2021-04-213-3/+62
|\
| * Added form fields and functionality for admin to edit assigned usersAndreaChirulescu2021-04-192-2/+56
| |
| * Added NN country for defaulting bands to an unknown countryAndreaChirulescu2021-04-181-1/+6
| |
| * Merge branch 'dev' of ↵AndreaChirulescu2021-04-183-27/+63
| |\ | | | | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges
| * \ Merge branch 'dev' of ↵AndreaChirulescu2021-04-158-2136/+114
| |\ \ | | | | | | | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges
* | | | Refactor BandTest.Harald Eilertsen2021-04-181-13/+22
| | | | | | | | | | | | | | | | Set up a test env before running the test cases.
* | | | Add a few more tests for GoglogAdmin_Band class.Harald Eilertsen2021-04-161-0/+28
| | | | | | | | | | | | | | | | | | | | Sidenote: UK is not included in the country list. Did the brexit everything?
* | | | Add test script to composer.jsonHarald Eilertsen2021-04-161-0/+3
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Refactor Band::create into Band::find and ::createHarald Eilertsen2021-04-152-22/+37
| | | | | | | | | | | | | | | | | | 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.
* | | Make Band::create return band object, not just id.Harald Eilertsen2021-04-153-14/+35
| | |
* | | Merge remote-tracking branch 'andrea/andreaschanges' into devHarald Eilertsen2021-04-154-62/+172
|\| | | |/ |/|
| * Got rid of find or create for concert and bandAndreaChirulescu2021-04-144-62/+172
| | | | | | | | Refactored band
* | Fix test setup for wp-env.Harald Eilertsen2021-04-157-2130/+104
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | Save city for venues too.Harald Eilertsen2021-04-151-1/+2
| |
* | Fix prepare call in Venue::venues_in_city.Harald Eilertsen2021-04-151-1/+1
| |
* | GiglogAdmin_Venue: Use default args to set default city.Harald Eilertsen2021-04-141-4/+3
| |
* | Use (object) notation to create plain objects.Harald Eilertsen2021-04-142-23/+27
|/
* Add license comment _after_ the php opening tag.Harald Eilertsen2021-04-148-13/+13
|
* Switch to Reuse for license handling.Harald Eilertsen2021-04-1431-247/+498
| | | | Run `reuse lint` to verify that all material is licensed.
* Some nitpicking in editforms.Harald Eilertsen2021-04-141-5/+11
| | | | | | Filter $_POST['edit'] into a boolean we can use to check wether we're editing or not. Then use this rather than other implicit assumptions to detect wether we're an edit form or a new concert form.
* Make Concert constructor public.Harald Eilertsen2021-04-142-5/+7
| | | | | Also allow it to default initialize when passing no args. Not entirely sure about making this interface public.
* Check if $_POST member is set before referencing it.Harald Eilertsen2021-04-141-1/+2
|
* Remove obsolete ref to $wpdb.Harald Eilertsen2021-04-141-1/+1
|
* Concert file wizardry to split create/createAndFind,Find functionsAndreaChirulescu2021-04-133-39/+46
| | | | Adjustments in admin page and import gigs to use the above adjustments