summaryrefslogtreecommitdiffstats
path: root/includes/admin/views/giglog_admin_page.php
Commit message (Collapse)AuthorAgeFilesLines
* Rename and reorganize more source files.Harald Eilertsen2023-01-201-91/+0
|
* Rename and restructure source files to conform to common namin schemes.Harald Eilertsen2023-01-191-1/+1
| | | | | | | | | - Source files containing a class should only contain _one_ class. - Source files containing a class should be names class-[name of the class].php - Use dashes instead of underscores in file names. - Fix source file comments - Some nitpicking...
* Add phpcs config and do automatic fixups.Harald Eilertsen2022-06-181-20/+20
|
* Move update new venue form to class.Harald Eilertsen2022-03-121-17/+3
| | | | Also fix nonce checking.
* Move update edit concert form code to class.Harald Eilertsen2022-03-121-49/+3
| | | | | | Also ensure that nonce checking is performed before both adding and editing concerts, and escape concert data before outputing it in the form.
* Move code to update concert table to concert table class.Harald Eilertsen2022-03-101-106/+5
| | | | | | Puts everything that belongs together into the same place. This means the concerts table is getting a tad large, but let's refactor that later.
* Drop some unused variables from the admin page update.Harald Eilertsen2022-03-101-7/+0
|
* Fix misc errors detected by psalm.Harald Eilertsen2022-03-101-17/+34
| | | | | - Lacking type info in venue.php - Potentially dereference null-objects in giglog_admin_page.php
* Fix various psalm errors.Harald Eilertsen2022-03-091-1/+1
|
* Add CSRF checks to new venue formHarald Eilertsen2021-09-171-0/+7
|
* Move New Venue form to a separate partial.Harald Eilertsen2021-09-171-2/+9
|
* Add CSRF checks for edit concert form.Harald Eilertsen2021-09-171-1/+8
|
* Fix indentation again.Harald Eilertsen2021-09-091-8/+10
|
* changes reagding mail sending on concert status changesAndreaChirulescu2021-09-071-14/+13
|
* Fixed wording in mails sent when concerts are assigned/unassingedAndreaChirulescu2021-09-061-4/+29
| | | | Added mailing for when concert status changes. Not sure how to get line break in text mails AND concert status as text
* Move the concerts table and edit form out of AdminPage.Harald Eilertsen2021-09-041-248/+10
| | | | | | | | | | | 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.
* Remove dangling $wpdb ref in AdminPage.Harald Eilertsen2021-09-041-2/+0
| | | | | AdminPage now references the database only through the Concert (and Venue) models.
* Mark recently added concerts as new again.Harald Eilertsen2021-09-041-17/+9
|
* Make click to unassign from concert work again.Harald Eilertsen2021-09-041-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 Eilertsen2021-09-041-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 Eilertsen2021-09-041-29/+25
|
* Editing concerts now work again.Harald Eilertsen2021-09-031-8/+24
|
* Handle updating concert status again.Harald Eilertsen2021-09-031-6/+7
|
* Rename AdminPage::get_user -> user_dropdown_for_roleHarald Eilertsen2021-09-031-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 Eilertsen2021-09-031-7/+10
|
* Drop query to press status table in adminactions.Harald Eilertsen2021-09-031-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 Eilertsen2021-09-031-9/+9
| | | | Clicking the buttons don't work quite yet.
* Redo concert list to use the Concert class.Harald Eilertsen2021-09-031-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.
* Move method to update Concertlogs to Concertlogs class.Harald Eilertsen2021-06-131-1/+1
|
* Fix misc Psalm issues.Harald Eilertsen2021-06-121-0/+2
|
* Drop obsolete line from AdminPage::get_concerts.Harald Eilertsen2021-06-121-1/+0
| | | | This must have been forgotten in the previous commit.
* Get current users username in AdminPage constructor.Harald Eilertsen2021-05-291-19/+9
|
* Make AdminPage a proper object.Harald Eilertsen2021-05-291-23/+34
|
* Refactor and reformat AdminPage::returnuser.Harald Eilertsen2021-05-261-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_userHarald Eilertsen2021-05-251-15/+4
|
* 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.
* 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-251-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 Eilertsen2021-05-251-33/+16
|
* Use select_field for venue selector in new/edit formHarald Eilertsen2021-05-241-13/+4
|
* 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-091-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 devHarald Eilertsen2021-05-091-7/+8
|\
| * Fixed venue by city filter. With a small exception: cna't get it to select ↵AndreaChirulescu2021-05-041-10/+11
| | | | | | | | in dropdown
* | Autofixes by PsalmHarald Eilertsen2021-05-081-3/+3
| |
* | Psalm fixesHarald Eilertsen2021-05-081-6/+6
| |
* | Auto fixup missing return types by PsalmHarald Eilertsen2021-05-081-10/+16
|/
* removed older icons. Added new ones created by meAndreaChirulescu2021-04-251-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 casesAndreaChirulescu2021-04-241-16/+18
| | | | CSS for edit form in giglog
* Merge remote-tracking branch 'andrea/andreaschanges' into devHarald Eilertsen2021-04-221-69/+9
|\