Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 | 1 | -22/+20 |
| | |||||
* | Begin move roles and status field to concerts table. | Harald Eilertsen | 2021-06-27 | 2 | -0/+17 |
| | |||||
* | Remove Band class and tests. | Harald Eilertsen | 2021-06-27 | 1 | -151/+0 |
| | |||||
* | Remove some debug output. | Harald Eilertsen | 2021-06-27 | 1 | -1/+0 |
| | |||||
* | Begin move roles and status field to concerts table. | Harald Eilertsen | 2021-06-14 | 2 | -13/+73 |
| | | | | 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 | 1 | -15/+22 |
| | | | | | 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 | 1 | -2/+8 |
| | |||||
* | Add function to generate selection boxes. | Harald Eilertsen | 2021-05-24 | 1 | -0/+26 |
| | |||||
* | Make it possible to list all concerts. | Harald Eilertsen | 2021-05-10 | 1 | -3/+6 |
| | |||||
* | Method to find concerts at given venue | Harald Eilertsen | 2021-05-09 | 1 | -0/+14 |
| | |||||
* | Method to find concerts for a given city. | Harald Eilertsen | 2021-05-09 | 1 | -0/+15 |
| | |||||
* | Fix importing gigs after changes to the concert class. | Harald Eilertsen | 2021-05-09 | 1 | -20/+10 |
| | |||||
* | Fix constructing concerts if only venue id passed. | Harald Eilertsen | 2021-05-09 | 1 | -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 id | Harald Eilertsen | 2021-05-09 | 1 | -0/+16 |
| | |||||
* | Rename strangely named function. | Harald Eilertsen | 2021-05-09 | 1 | -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 Eilertsen | 2021-05-09 | 2 | -23/+34 |
| | | | | | | | | | | | | | | | | | | | | 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. |