Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Update db tables if needed on every load. | Harald Eilertsen | 2021-09-04 | 1 | -1/+2 |
| | | | | | No need to deactivate/activate plugin to get the right version of the tables. | ||||
* | Make click to unassign from concert work again. | Harald Eilertsen | 2021-09-04 | 2 | -15/+18 |
| | | | | | | As a user can only be assigned to one role at the time, we remove the current user from any role that they may have when clearing the assignment. | ||||
* | Rename AdminPage::returnuser to assign_role_for_user_form. | Harald Eilertsen | 2021-09-04 | 1 | -5/+5 |
| | | | | It did not return any users, but a form so name it for what it does. | ||||
* | Make click to assign concert/role to self work again. | Harald Eilertsen | 2021-09-04 | 1 | -29/+25 |
| | |||||
* | Editing concerts now work again. | Harald Eilertsen | 2021-09-03 | 2 | -24/+63 |
| | |||||
* | Handle updating concert status again. | Harald Eilertsen | 2021-09-03 | 1 | -6/+7 |
| | |||||
* | Rename AdminPage::get_user -> user_dropdown_for_role | Harald Eilertsen | 2021-09-03 | 1 | -5/+5 |
| | | | | | | The original name did not make much sense. The function didn't return a user, but a dropdown list of users, where the user currently holding the given role for the given concert was preselected in the list. | ||||
* | Drop ref to ConcertLogs in AdminPage::get_user method. | Harald Eilertsen | 2021-09-03 | 1 | -7/+10 |
| | |||||
* | Drop query to press status table in adminactions. | Harald Eilertsen | 2021-09-03 | 1 | -6/+11 |
| | | | | | | | | | | | | Replace the table with hardcoded strings in the AdminPage class. This makes it a pure presentation issue, while the statuses themselves are just mnemonics. There's one smell here, and that is that the status values and their textual representation is split across two modules. (Values in Concert, and textual representation in AdminPage.) This should probably be addressed later by refactoring both into a separate AccredStatus class or something. | ||||
* | Shift value of Concert statuses to be 0-based. | Harald Eilertsen | 2021-09-03 | 1 | -7/+7 |
| | | | | | This will trip up any existing records in the db, but that should not matter, since we're changing how this entire stuff works now. | ||||
* | Render adminactions in concert list view. | Harald Eilertsen | 2021-09-03 | 1 | -9/+9 |
| | | | | Clicking the buttons don't work quite yet. | ||||
* | Redo concert list to use the Concert class. | Harald Eilertsen | 2021-09-03 | 1 | -38/+32 |
| | | | | | | | | | | | | Instead of directly accessing the database with a custom query, we now just use the Concert::find_concerts method to fetch the concerts that are to be displayed. This became much easier now that we don't rely on the extra concertlogs table. There's still stuff missing to be functionally equivalent to the old code, but this should be a lot easier to get in place now. | ||||
* | emove dependency on ConcertLogs in gig import code. | Harald Eilertsen | 2021-09-03 | 2 | -13/+0 |
| | |||||
* | Allow empty concerts to be created. | Harald Eilertsen | 2021-09-03 | 1 | -5/+5 |
| | |||||
* | Make name and city required attributes for Venues. | Harald Eilertsen | 2021-09-02 | 1 | -6/+21 |
| | | | | It does not make sense to have anonymous venues nowhere. | ||||
* | Fix more type issues in Concert and Venue classes | Harald Eilertsen | 2021-09-02 | 2 | -6/+6 |
| | |||||
* | psalm: Add type info to attrs and constructor for Concert. | Harald Eilertsen | 2021-09-02 | 1 | -8/+18 |
| | | | | Also make sure we explicitly set the venue attribute in the constructor. | ||||
* | psalm: Ad types to attrs and constructor for Venue | Harald Eilertsen | 2021-09-02 | 1 | -6/+6 |
| | |||||
* | psalm: No need for null $opts param in select_field. | Harald Eilertsen | 2021-09-02 | 1 | -1/+1 |
| | | | | | It's initialized to an empty array if not specified, that should be good enough, and don't trip up iterating over it. | ||||
* | Fix saving and fetching roles from Concerts table. | Harald Eilertsen | 2021-09-01 | 2 | -41/+24 |
| | |||||
* | Begin move roles and status field to concerts table. | Harald Eilertsen | 2021-06-27 | 4 | -0/+43 |
| | |||||
* | Remove Band class and tests. | Harald Eilertsen | 2021-06-27 | 2 | -220/+0 |
| | |||||
* | Remove some debug output. | Harald Eilertsen | 2021-06-27 | 1 | -1/+0 |
| | |||||
* | Refactor ConcertTest some more. | Harald Eilertsen | 2021-06-27 | 1 | -58/+25 |
| | | | | | Create more of the concerts used by tests into the wpSetupBeforeClass hook. | ||||
* | Refactor ConcertTests and some cleanup | Harald Eilertsen | 2021-06-27 | 1 | -68/+75 |
| | | | | | | | | | | | The commit changes the way we populate the database for the tests by creating more entries up front. This reduces the amount of duplicated code between the tests, but also introduce some challenges. As modifications to the database done in the wpSetUpBeforeClass hook are not cleaned up automatically by the WP_PHPUnit framework, we also have to add a wpTearDownAfterClass hook so anything we set up in this class does not disturb any other tests in other classes. | ||||
* | Clean tests env before running tests | Harald Eilertsen | 2021-06-27 | 2 | -0/+2 |
| | | | | | | Since we're using our own database tables, these are not cleaned by the default WP_PHPUnit setup. To ensure that we start on a clean slate when running the tests, clean the env before the tests are run. | ||||
* | Begin move roles and status field to concerts table. | Harald Eilertsen | 2021-06-14 | 3 | -13/+97 |
| | | | | There's no need to have a separate table (concertlogs) for these fields. | ||||
* | Fix detecting error in update_concert method. | Harald Eilertsen | 2021-06-13 | 1 | -1/+1 |
| | |||||
* | Move method to update Concertlogs to Concertlogs class. | Harald Eilertsen | 2021-06-13 | 3 | -24/+23 |
| | |||||
* | Streamline Concert api. | Harald Eilertsen | 2021-06-12 | 2 | -22/+29 |
| | | | | | Reduce to one find_concerts function taking a filter to limit the selection. | ||||
* | Fix misc Psalm issues. | Harald Eilertsen | 2021-06-12 | 5 | -4/+8 |
| | |||||
* | Drop obsolete line from AdminPage::get_concerts. | Harald Eilertsen | 2021-06-12 | 1 | -1/+0 |
| | | | | This must have been forgotten in the previous commit. | ||||
* | Get current users username in AdminPage constructor. | Harald Eilertsen | 2021-05-29 | 1 | -19/+9 |
| | |||||
* | Make AdminPage a proper object. | Harald Eilertsen | 2021-05-29 | 1 | -23/+34 |
| | |||||
* | Refactor and reformat AdminPage::returnuser. | Harald Eilertsen | 2021-05-26 | 1 | -18/+31 |
| | | | | | Now use a Concertlog object to render the correct subform instead of messing with the db directly. | ||||
* | Add instance methods to Concertlogs. | Harald Eilertsen | 2021-05-26 | 1 | -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_user | Harald Eilertsen | 2021-05-25 | 1 | -15/+4 |
| | |||||
* | Remove debug log from concertlogs | Harald Eilertsen | 2021-05-25 | 1 | -1/+0 |
| | |||||
* | Simplify AdminPage::get_user | Harald Eilertsen | 2021-05-25 | 1 | -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 concertlogs | Harald Eilertsen | 2021-05-25 | 1 | -0/+20 |
| | |||||
* | Refactor the AdminPage::get_user method. | Harald Eilertsen | 2021-05-25 | 1 | -11/+23 |
| | | | | No functional change, just trying to make sense of it. | ||||
* | Use select_field in adminactions form. | Harald Eilertsen | 2021-05-25 | 2 | -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 Eilertsen | 2021-05-25 | 1 | -33/+16 |
| | |||||
* | Use select_field for venue selector in new/edit form | Harald Eilertsen | 2021-05-24 | 1 | -13/+4 |
| | |||||
* | Allow custom text for no selection in select_fields. | Harald Eilertsen | 2021-05-24 | 2 | -2/+25 |
| | |||||
* | Add function to generate selection boxes. | Harald Eilertsen | 2021-05-24 | 3 | -0/+80 |
| | |||||
* | Make it possible to list all concerts. | Harald Eilertsen | 2021-05-10 | 2 | -3/+29 |
| | |||||
* | Method to find concerts at given venue | Harald Eilertsen | 2021-05-09 | 2 | -0/+54 |
| | |||||
* | Method to find concerts for a given city. | Harald Eilertsen | 2021-05-09 | 2 | -0/+56 |
| | |||||
* | Fix importing gigs after changes to the concert class. | Harald Eilertsen | 2021-05-09 | 1 | -20/+10 |
| |