summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* 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.
* 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
|\
| * 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
* | 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
|/
* 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
* Fixes of LF and extra spaces done by ATOMAndreaChirulescu2021-04-139-531/+531
|
* Moved a lot of concert related logic from import/admin into the ↵AndreaChirulescu2021-04-123-136/+214
| | | | GiglogAdmin_Concert
* fixed a missing = in an if inside the editconcerts form.AndreaChirulescu2021-04-121-11/+12
| | | | Added check if cid has a value
* Fixed calls to cretae band/venue functions in import gigs codeAndreaChirulescu2021-04-121-2/+2
|
* Fixed capital IFs and hopefully formatting issuesAndreaChirulescu2021-04-114-118/+118
|
* Added small forms to add new venue and bandAndreaChirulescu2021-04-114-88/+217
| | | | Made form for concert both functional to edit existing concert or add new concert
* Change venues into proper objects.Harald Eilertsen2021-04-105-15/+174
| | | | | | | | | This means most static functions now either return a venue object, or an array of venue objects. The exception is the `all_cities` method, which still return an array of cities as strings. The constructor has been made private, as it should not be used directly from anywhere but the static methods on the Venue class.
* Install phpunit with composer.Harald Eilertsen2021-04-102-1/+2113
|