| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | | Merge branch 'psalm' into dev | Harald Eilertsen | 2021-05-08 | 18 | -64/+2274 | |
| |\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | 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. | |||||
| | * | Autofixes by Psalm | Harald Eilertsen | 2021-05-08 | 5 | -8/+10 | |
| | | | ||||||
| | * | Psalm fixes | Harald Eilertsen | 2021-05-08 | 2 | -8/+12 | |
| | | | ||||||
| | * | Autofix missing property types by Psalm. | Harald Eilertsen | 2021-05-08 | 1 | -7/+42 | |
| | | | ||||||
| | * | Fix return types for abstract class GeneratorBase. | Harald Eilertsen | 2021-05-08 | 1 | -11/+5 | |
| | | | ||||||
| | * | Auto fixup missing return types by Psalm | Harald Eilertsen | 2021-05-08 | 16 | -43/+91 | |
| | | | ||||||
| | * | Add Psalm as dev dependency | Harald Eilertsen | 2021-05-08 | 2 | -2/+2129 | |
| | | | ||||||
| * | | Add missing license tags to files. | Harald Eilertsen | 2021-04-26 | 4 | -0/+24 | |
| |/ | ||||||
| * | 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 | |
| |/ | ||||||
 |
index : plugins/gigologadmin.git | |
| WordPress plugin for handling concert assignments for Eternal-Terror.com | Harald Eilertsen |
| summaryrefslogtreecommitdiffstats |