Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Refactor Concert::find_concerts. | Harald Eilertsen | 2021-09-06 | 1 | -23/+28 |
| | | | | | Make it a bit more compact and fix use of $wpdb->prepare for `currentuser` filter. | ||||
* | Remove logging from Venue class. | Harald Eilertsen | 2021-09-06 | 1 | -8/+0 |
| | | | | Move logging to presentation/logic layer if necessary. | ||||
* | Throw instead of log error from Concert class. | Harald Eilertsen | 2021-09-06 | 1 | -14/+9 |
| | | | | | Should get rid of most of the annoying output during testing, and allow moving error handling and logging to the presentation layer. | ||||
* | Use find_concerts to check for duplicates on create. | Harald Eilertsen | 2021-09-06 | 1 | -5/+24 |
| | | | | This also adds a number of new filters to find_concerts. | ||||
* | Remove obsolete function Concert::find | Harald Eilertsen | 2021-09-06 | 1 | -13/+0 |
| | |||||
* | Use wpdb::get_row in Concert::get() | Harald Eilertsen | 2021-09-06 | 1 | -4/+4 |
| | | | | | Makes it clearer that we're just fetching _one_ row from the database. Also updated doc comment to hopefully be a bit clearer. | ||||
* | Remove unneccesary code to sort single concert by date. | Harald Eilertsen | 2021-09-06 | 1 | -1/+0 |
| | |||||
* | 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 | ||||
* | File to display concerts inside a page by calling getconcerts code | AndreaChirulescu | 2021-09-05 | 1 | -0/+14 |
| | |||||
* | 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 | 2 | -19/+31 |
| | | | | 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 | 4 | -12/+35 |
|\ | |||||
| * | Re-added the changes for looks, and city/venue selector | AndreaChirulescu | 2021-09-05 | 2 | -14/+14 |
| | | |||||
| * | Merge branch 'dev' of ↵ | AndreaChirulescu | 2021-09-05 | 4 | -252/+292 |
| |\ | | | | | | | | | | | | | | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges # Conflicts: # includes/admin/views/giglog_admin_page.php | ||||
| * | | Ordered venues by name in drop down | AndreaChirulescu | 2021-09-05 | 1 | -1/+2 |
| | | | |||||
| * | | Fixed filter by your concerts | AndreaChirulescu | 2021-09-05 | 3 | -11/+34 |
| | | | | | | | | | | | | Added CSS and extra fromatting to table | ||||
| * | | Fixed a call to city() method in concert.php It needed ->venue->city | AndreaChirulescu | 2021-09-04 | 2 | -1/+3 |
| | | | | | | | | | | | | Added order by concert date in concert list | ||||
* | | | 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 |
| |/ |/| | |||||
* | | Drop extra query for venue_id in concert queries. | Harald Eilertsen | 2021-09-04 | 1 | -1/+1 |
| | | | | | | | | | | It's not required since the id is in the concerts table too. That's what links them together. | ||||
* | | 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. | ||||
* | Fix so concert id is not overwritten by venue id. | Harald Eilertsen | 2021-09-04 | 1 | -5/+8 |
| | | | | | | | | | | | | | | | | | | | | | | | We originally had a more specified query, but simplified it to: SELECT * FROM wpg_concerts LEFT JOIN wpg_venues ON ...; But since both the concerts table and the venues table has a column id, the concert id would be overwritten with the venue id. MySQL/MariaDB does not allow columns with the same name in multiple tables when using unqualified column names in the query. So we need to be more explicit again. I was hoping that the following would work: SELECT wpg_concerts.*, wpg_venues.* FROM .... ; I think MySQL/MariaDB would handle that, but now since php turns the result into an array, where each key must be unique, this again overwrites the concert id with the venue id. So thus a more verbose specification of the columns was necessary. | ||||
* | Remove obsolete source files. | Harald Eilertsen | 2021-09-04 | 2 | -127/+0 |
| | |||||
* | 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 obsolete shortcodes. | Harald Eilertsen | 2021-09-04 | 4 | -720/+0 |
| | | | | These are no longer in use, and have been replaced by the admin screens. | ||||
* | 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 | 2 | -21/+23 |
| | |||||
* | 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 | 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. |