Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Move New Venue form to a separate partial. | Harald Eilertsen | 2021-09-17 | 3 | -9/+45 |
| | |||||
* | Return better error for failed CSRF check on import. | Harald Eilertsen | 2021-09-17 | 1 | -2/+2 |
| | |||||
* | Add CSRF checks for edit concert form. | Harald Eilertsen | 2021-09-17 | 2 | -1/+9 |
| | |||||
* | Clean up, fix and rename db tables. | Harald Eilertsen | 2021-09-16 | 1 | -49/+19 |
| | | | | | | | | | | | | | | | | | This patch got a bit more involved than what was originally planned, but since we're messing with the tables I decided to do it all right away. - Moves the constraint definition to the CREATE TABLE statement for the concerts table. This replaces the existing KEY definition that it had. - Make sure the venues table is created before the concerts table so that the above mentioned constraint definition works. - Rename the tables. Use the wpdb-prefix and make the name a bit prettier. This caused some changes in the Concert and Venue classes, and for slightly silly reasons some test classes. The code actually turned out better (for the most part), but some refactoring can still be done. The column names remains unchanged for now. | ||||
* | Reinsert created and updated columns for venues. | Harald Eilertsen | 2021-09-16 | 1 | -0/+2 |
| | | | | Was accidentally dropped during rewrite of base schema. | ||||
* | Facelift of database table creation to use create tables instead of ↵ | AndreaChirulescu | 2021-09-13 | 1 | -323/+17 |
| | | | | versioning. First attempt | ||||
* | tests: Make an actual test for ConcertsTable | Harald Eilertsen | 2021-09-11 | 1 | -3/+3 |
| | | | | | | | | | | | There's a bit of setup to make this work as it should, we need to ensure that the current user and current screen is set to proper values so that the WordPress api's `is_admin()` and `current_user_can()` work as they should. This first test just tests that all the expected forms are being rendered for the admin user accessing the table through the site admin interface. | ||||
* | Fix indentation again. | Harald Eilertsen | 2021-09-09 | 1 | -8/+10 |
| | |||||
* | changes reagding mail sending on concert status changes | AndreaChirulescu | 2021-09-07 | 1 | -14/+13 |
| | |||||
* | Fixed wording in mails sent when concerts are assigned/unassinged | AndreaChirulescu | 2021-09-06 | 1 | -4/+29 |
| | | | | Added mailing for when concert status changes. Not sure how to get line break in text mails AND concert status as text | ||||
* | fixed proper class naming for Apply button | AndreaChirulescu | 2021-09-06 | 1 | -1/+1 |
| | |||||
* | removed some extra characters that got lost in table HTML | AndreaChirulescu | 2021-09-06 | 1 | -1/+1 |
| | |||||
* | Fix indentation and whitespace, add braces where missing. | Harald Eilertsen | 2021-09-06 | 1 | -41/+46 |
| | |||||
* | Merge remote-tracking branch 'andrea/andreaschanges' into dev | Harald Eilertsen | 2021-09-05 | 1 | -4/+5 |
|\ | |||||
| * | added images for checkbox and redicon | AndreaChirulescu | 2021-09-05 | 1 | -4/+5 |
| | | | | | | | | added class for City dropdown form | ||||
* | | security: Add proper CSRF checking for the import_gigs form. | Harald Eilertsen | 2021-09-05 | 1 | -3/+7 |
|/ | |||||
* | used auto ident this time | AndreaChirulescu | 2021-09-05 | 1 | -165/+165 |
| | |||||
* | used is_admin() to check if user shoudl see xtra columns | AndreaChirulescu | 2021-09-05 | 1 | -184/+184 |
| | | | | Re-idented the file | ||||
* | Extra changes to hide admincolumns on giglog page If logged in but not using ↵ | AndreaChirulescu | 2021-09-05 | 1 | -6/+8 |
| | | | | plugin | ||||
* | CHanges in the concerts table for logged/non logged user. Fixed row header ↵ | AndreaChirulescu | 2021-09-05 | 1 | -19/+30 |
| | | | | with city | ||||
* | Fix some bugs in generated HTML for ConcertsTable. | Harald Eilertsen | 2021-09-05 | 1 | -5/+5 |
| | |||||
* | 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. |