summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * Add instance methods to Concertlogs.Harald Eilertsen2021-05-261-0/+33
| | | | | | | | | | | | This allows us to instantiate a Concertlogs objects just as with Concert and Venue objects. Also add a few instance methods to get the assigned user for a given role, and get the role assigned to a given user.
| * Use select_field in AdminPage::get_userHarald Eilertsen2021-05-251-15/+4
| |
| * Remove debug log from concertlogsHarald Eilertsen2021-05-251-1/+0
| |
| * Simplify AdminPage::get_userHarald Eilertsen2021-05-251-14/+5
| | | | | | | | | | Now queries the user from the concertlogs table instead of going by generating a form that is thrown away.
| * Add method to get assigned user for concertlogsHarald Eilertsen2021-05-251-0/+20
| |
| * Refactor the AdminPage::get_user method.Harald Eilertsen2021-05-251-11/+23
| | | | | | | | No functional change, just trying to make sense of it.
| * Use select_field in adminactions form.Harald Eilertsen2021-05-252-11/+17
| | | | | | | | | | Also add a `get_status` method to the Concertlogs class, returning the press status for a given concert_id.
| * Use select_field helper for city/venue filter.Harald Eilertsen2021-05-251-33/+16
| |
| * Use select_field for venue selector in new/edit formHarald Eilertsen2021-05-241-13/+4
| |
| * Allow custom text for no selection in select_fields.Harald Eilertsen2021-05-242-2/+25
| |
| * Add function to generate selection boxes.Harald Eilertsen2021-05-243-0/+80
| |
| * Make it possible to list all concerts.Harald Eilertsen2021-05-102-3/+29
| |
* | Merge branch 'dev' of ↵AndreaChirulescu2021-05-092-0/+110
|\| | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges
| * Method to find concerts at given venueHarald Eilertsen2021-05-092-0/+54
| |
| * Method to find concerts for a given city.Harald Eilertsen2021-05-092-0/+56
| |
* | Recommitted, with removed head<<<<<< and >>>>AndreaChirulescu2021-05-093-67/+9
| |
* | Tried to fix the psalm local changes done when I manually installed itAndreaChirulescu2021-05-0920-107/+455
|\|
| * Fix importing gigs after changes to the concert class.Harald Eilertsen2021-05-091-20/+10
| |
| * Fix constructing concerts if only venue id passed.Harald Eilertsen2021-05-091-7/+12
| | | | | | | | | | | | | | | | | | The previous changes required the full venue data to be included in the constructor. This patch ensures that we handle the old case, where only the venue id was passed in. We still should get the full concert object including the venue name and city back.
| * New method Venue::get to get venue by idHarald Eilertsen2021-05-091-0/+16
| |
| * Rename strangely named function.Harald Eilertsen2021-05-091-2/+2
| | | | | | | | | | | | GoglogAdmin_AdminPage::get_allvenues did not get all venues, but presented a selection control for selecting a venue. Renamed it as get_venue_selector instead.
| * Include info from venue in concerts.Harald Eilertsen2021-05-093-24/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes the concert a full object containing all relevant info, while we can still segment the data in the db. Instead of this: $concert = GiglogAdmin_Concert::get($concert_id); $venue = GiglogAdmin_Venue::get($concert->venue()); echo "{$concert->name()} @ {$venue->name()} : {$concert->cdate()}" You can now do: $concert = GiglogAdmin_Concert::get($concert_id); echo "{$concert->name()} @ {$concert->venue()->name()} : {$concert->cdate()}" And yeah, renamed Concert::find_cid() to Concert::get() and changed it's semantics somewhat. It now either returns the given concert if it exists, or NULL if it does not. Simpler function; simpler to use.
| * Make venue constructor public.Harald Eilertsen2021-05-091-1/+1
| |
| * Merge remote-tracking branch 'andrea/andreaschanges' into devHarald Eilertsen2021-05-091-7/+8
| |\
| * | Venue::venues_in_city wrong type annotationHarald Eilertsen2021-05-081-1/+1
| | |
| * | Add config for Psalm as well...Harald Eilertsen2021-05-081-0/+28
| | |
| * | Merge branch 'psalm' into devHarald Eilertsen2021-05-0818-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 PsalmHarald Eilertsen2021-05-085-8/+10
| | | |
| | * | Psalm fixesHarald Eilertsen2021-05-082-8/+12
| | | |
| | * | Autofix missing property types by Psalm.Harald Eilertsen2021-05-081-7/+42
| | | |
| | * | Fix return types for abstract class GeneratorBase.Harald Eilertsen2021-05-081-11/+5
| | | |
| | * | Auto fixup missing return types by PsalmHarald Eilertsen2021-05-0816-43/+91
| | | |
| | * | Add Psalm as dev dependencyHarald Eilertsen2021-05-082-2/+2129
| | | |
* | | | After installing psalm, I got these files modified with, what seems to be, ↵AndreaChirulescu2021-05-094-5/+2003
| |_|/ |/| | | | | | | | local hashkeys?
* | | Fixed venue by city filter. With a small exception: cna't get it to select ↵AndreaChirulescu2021-05-041-10/+11
|/ / | | | | | | in dropdown
* / Add missing license tags to files.Harald Eilertsen2021-04-264-0/+24
|/
* 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
|