Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge remote-tracking branch 'andrea/andreaschanges' into dev | Harald Eilertsen | 2021-09-05 | 1 | -11/+14 |
|\ | |||||
| * | Re-added the changes for looks, and city/venue selector | AndreaChirulescu | 2021-09-05 | 1 | -11/+14 |
| | | |||||
* | | ConcertsTable: fix getting city row when changing cities. | Harald Eilertsen | 2021-09-05 | 1 | -1/+1 |
| | | |||||
* | | Don't error if obsolete indexes or keys are already deleted. | Harald Eilertsen | 2021-09-05 | 1 | -2/+2 |
|/ | |||||
* | Move the concerts table and edit form out of AdminPage. | Harald Eilertsen | 2021-09-04 | 3 | -248/+291 |
| | | | | | | | | | | | Currently the AdminPage is still responsible for updating changes to any of the concerts, but I'd like to get that into their respective classes too. That way the AdminPage will just be a simple class to handle the layout of the page, while all the specific functionality is in their own classes. This is also the first step to be able to reuse the concerts table on the public end of the site. | ||||
* | Drop obsolete database tables. | Harald Eilertsen | 2021-09-04 | 1 | -97/+6 |
| | | | | | These tables are no longer being used, so let's remove them and the code to add them. | ||||
* | Remove dangling $wpdb ref in AdminPage. | Harald Eilertsen | 2021-09-04 | 1 | -2/+0 |
| | | | | | AdminPage now references the database only through the Concert (and Venue) models. | ||||
* | Mark recently added concerts as new again. | Harald Eilertsen | 2021-09-04 | 1 | -17/+9 |
| | |||||
* | Add timestamp columns to concerts and venues. | Harald Eilertsen | 2021-09-04 | 1 | -2/+17 |
| | | | | To keep track of creation and modification times for each record. | ||||
* | Make click to unassign from concert work again. | Harald Eilertsen | 2021-09-04 | 1 | -15/+13 |
| | | | | | | 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 | 1 | -8/+24 |
| | |||||
* | 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. | ||||
* | 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 | 1 | -2/+0 |
| | |||||
* | Begin move roles and status field to concerts table. | Harald Eilertsen | 2021-06-14 | 1 | -2/+16 |
| | | | | There's no need to have a separate table (concertlogs) for these fields. | ||||
* | Move method to update Concertlogs to Concertlogs class. | Harald Eilertsen | 2021-06-13 | 1 | -1/+1 |
| | |||||
* | Fix misc Psalm issues. | Harald Eilertsen | 2021-06-12 | 2 | -1/+3 |
| | |||||
* | 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. | ||||
* | Use select_field in AdminPage::get_user | Harald Eilertsen | 2021-05-25 | 1 | -15/+4 |
| | |||||
* | 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. | ||||
* | 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 | 1 | -11/+5 |
| | | | | | 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 |
| | |||||
* | Fix importing gigs after changes to the concert class. | Harald Eilertsen | 2021-05-09 | 1 | -20/+10 |
| | |||||
* | 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 | 1 | -5/+9 |
| | | | | | | | | | | | | | | | | | | | | 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. | ||||
* | Merge remote-tracking branch 'andrea/andreaschanges' into dev | Harald Eilertsen | 2021-05-09 | 1 | -7/+8 |
|\ | |||||
| * | Fixed venue by city filter. With a small exception: cna't get it to select ↵ | AndreaChirulescu | 2021-05-04 | 1 | -10/+11 |
| | | | | | | | | in dropdown | ||||
* | | Autofixes by Psalm | Harald Eilertsen | 2021-05-08 | 2 | -4/+6 |
| | | |||||
* | | Psalm fixes | Harald Eilertsen | 2021-05-08 | 1 | -6/+6 |
| | | |||||
* | | Auto fixup missing return types by Psalm | Harald Eilertsen | 2021-05-08 | 6 | -17/+28 |
|/ | |||||
* | removed older icons. Added new ones created by me | AndreaChirulescu | 2021-04-25 | 1 | -16/+13 |
| | | | | 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 | 1 | -16/+18 |
| | | | | CSS for edit form in giglog | ||||
* | Merge remote-tracking branch 'andrea/andreaschanges' into dev | Harald Eilertsen | 2021-04-22 | 1 | -69/+9 |
|\ | |||||
| * | changed oackage.json to use wp-env 4.0.0 | AndreaChirulescu | 2021-04-22 | 1 | -69/+9 |
| | | | | | | | | 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 Eilertsen | 2021-04-22 | 1 | -8/+11 |
|/ | | | | | | 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. | ||||
* | 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. | ||||
* | Added column for concert title in database | AndreaChirulescu | 2021-04-21 | 3 | -91/+41 |
| | | | | 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 |
| |