Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add Psalm as dev dependency | Harald Eilertsen | 2021-05-08 | 2 | -2/+2129 |
| | |||||
* | removed older icons. Added new ones created by me | AndreaChirulescu | 2021-04-25 | 10 | -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 cases | AndreaChirulescu | 2021-04-24 | 3 | -16/+63 |
| | | | | CSS for edit form in giglog | ||||
* | Added an uppercase in sql for venue and concert duplicate check | AndreaChirulescu | 2021-04-23 | 3 | -2/+32 |
| | | | | Added test to create duplicate concert with varied cases in string | ||||
* | Merge branch 'dev' of ↵ | AndreaChirulescu | 2021-04-23 | 1 | -40/+27 |
|\ | | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges | ||||
| * | Refactor Concert::create | Harald Eilertsen | 2021-04-23 | 1 | -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 usage | AndreaChirulescu | 2021-04-23 | 1 | -0/+2 |
| | | |||||
* | | Merge branch 'dev' of ↵ | AndreaChirulescu | 2021-04-23 | 4 | -12/+66 |
|\| | | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges | ||||
| * | Fix composer test command. | Harald Eilertsen | 2021-04-22 | 1 | -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 dev | Harald Eilertsen | 2021-04-22 | 3 | -2181/+186 |
| |\ | |||||
| * | | Concert::create return NULL if concert already exist. | Harald Eilertsen | 2021-04-22 | 3 | -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 Eilertsen | 2021-04-22 | 1 | -0/+49 |
| | | | |||||
* | | | no idea what these changes are but got stuck with them, so just commiting ↵ | AndreaChirulescu | 2021-04-23 | 1 | -1652/+10 |
| |/ |/| | | | | | before merging latest changes | ||||
* | | changed oackage.json to use wp-env 4.0.0 | AndreaChirulescu | 2021-04-22 | 3 | -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 Eilertsen | 2021-04-22 | 1 | -1/+1 |
| | |||||
* | Activate db version 5 | Harald Eilertsen | 2021-04-22 | 1 | -1/+1 |
| | |||||
* | Fix typo in register_db_table. | Harald Eilertsen | 2021-04-22 | 1 | -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 dev | Harald Eilertsen | 2021-04-22 | 7 | -128/+73 |
|\ | |||||
| * | Added column for concert title in database | AndreaChirulescu | 2021-04-21 | 7 | -138/+73 |
| | | | | | | | | Modified so that band is no longer in use | ||||
| * | added check that GiglogAdmin_Band::get_band returns something | AndreaChirulescu | 2021-04-21 | 1 | -3/+13 |
| | | |||||
* | | Typo in comment for Concert constructor. | Harald Eilertsen | 2021-04-21 | 1 | -2/+2 |
| | | |||||
* | | Merge remote-tracking branch 'andrea/andreaschanges' into dev | Harald Eilertsen | 2021-04-21 | 1 | -1/+4 |
|\| | |||||
| * | Added an else to return something when concert is empty on return user function | AndreaChirulescu | 2021-04-21 | 1 | -0/+1 |
| | | |||||
| * | Added check if emty concert when fetching assigned user | AndreaChirulescu | 2021-04-21 | 1 | -1/+3 |
| | | |||||
* | | run-tests: Pass any additional args to phpunit. | Harald Eilertsen | 2021-04-21 | 1 | -1/+1 |
| | | | | | | | | | | | | | | Particularly useful for running a specific test case: For example: ./run-tests --filter Band::testCreateBandWithName | ||||
* | | Fix Bands class constructor. | Harald Eilertsen | 2021-04-21 | 2 | -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 dev | Harald Eilertsen | 2021-04-21 | 3 | -3/+62 |
|\ | |||||
| * | Added form fields and functionality for admin to edit assigned users | AndreaChirulescu | 2021-04-19 | 2 | -2/+56 |
| | | |||||
| * | Added NN country for defaulting bands to an unknown country | AndreaChirulescu | 2021-04-18 | 1 | -1/+6 |
| | | |||||
| * | Merge branch 'dev' of ↵ | AndreaChirulescu | 2021-04-18 | 3 | -27/+63 |
| |\ | | | | | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges | ||||
| * \ | Merge branch 'dev' of ↵ | AndreaChirulescu | 2021-04-15 | 8 | -2136/+114 |
| |\ \ | | | | | | | | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges | ||||
* | | | | Refactor BandTest. | Harald Eilertsen | 2021-04-18 | 1 | -13/+22 |
| | | | | | | | | | | | | | | | | Set up a test env before running the test cases. | ||||
* | | | | Add a few more tests for GoglogAdmin_Band class. | Harald Eilertsen | 2021-04-16 | 1 | -0/+28 |
| | | | | | | | | | | | | | | | | | | | | Sidenote: UK is not included in the country list. Did the brexit everything? | ||||
* | | | | Add test script to composer.json | Harald Eilertsen | 2021-04-16 | 1 | -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 ::create | Harald Eilertsen | 2021-04-15 | 2 | -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 Eilertsen | 2021-04-15 | 3 | -14/+35 |
| | | | |||||
* | | | Merge remote-tracking branch 'andrea/andreaschanges' into dev | Harald Eilertsen | 2021-04-15 | 4 | -62/+172 |
|\| | | |/ |/| | |||||
| * | Got rid of find or create for concert and band | AndreaChirulescu | 2021-04-14 | 4 | -62/+172 |
| | | | | | | | | Refactored band | ||||
* | | Fix test setup for wp-env. | Harald Eilertsen | 2021-04-15 | 7 | -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 Eilertsen | 2021-04-15 | 1 | -1/+2 |
| | | |||||
* | | Fix prepare call in Venue::venues_in_city. | Harald Eilertsen | 2021-04-15 | 1 | -1/+1 |
| | | |||||
* | | GiglogAdmin_Venue: Use default args to set default city. | Harald Eilertsen | 2021-04-14 | 1 | -4/+3 |
| | | |||||
* | | Use (object) notation to create plain objects. | Harald Eilertsen | 2021-04-14 | 2 | -23/+27 |
|/ | |||||
* | Add license comment _after_ the php opening tag. | Harald Eilertsen | 2021-04-14 | 8 | -13/+13 |
| | |||||
* | Switch to Reuse for license handling. | Harald Eilertsen | 2021-04-14 | 31 | -247/+498 |
| | | | | Run `reuse lint` to verify that all material is licensed. | ||||
* | Some nitpicking in editforms. | Harald Eilertsen | 2021-04-14 | 1 | -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 Eilertsen | 2021-04-14 | 2 | -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 Eilertsen | 2021-04-14 | 1 | -1/+2 |
| | |||||
* | Remove obsolete ref to $wpdb. | Harald Eilertsen | 2021-04-14 | 1 | -1/+1 |
| | |||||
* | Concert file wizardry to split create/createAndFind,Find functions | AndreaChirulescu | 2021-04-13 | 3 | -39/+46 |
| | | | | Adjustments in admin page and import gigs to use the above adjustments |