Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add phpcs config and do automatic fixups. | Harald Eilertsen | 2022-06-18 | 5 | -328/+291 |
| | |||||
* | Filtered out concerts older than today | AndreaChirulescu | 2022-03-27 | 1 | -1/+1 |
| | | | | Concerts should show with upper case | ||||
* | Move update new venue form to class. | Harald Eilertsen | 2022-03-12 | 2 | -18/+20 |
| | | | | Also fix nonce checking. | ||||
* | Move update edit concert form code to class. | Harald Eilertsen | 2022-03-12 | 2 | -57/+73 |
| | | | | | | Also ensure that nonce checking is performed before both adding and editing concerts, and escape concert data before outputing it in the form. | ||||
* | Security: Escape event and link urls before using. | Harald Eilertsen | 2022-03-12 | 1 | -2/+2 |
| | |||||
* | ConcertsTable: Initialize more properties in constructor. | Harald Eilertsen | 2022-03-12 | 1 | -0/+5 |
| | |||||
* | Security: Escape band and venue name in concerts table. | Harald Eilertsen | 2022-03-12 | 1 | -2/+2 |
| | |||||
* | ConcertsTable: Initialize page_no on construct. | Harald Eilertsen | 2022-03-12 | 1 | -1/+1 |
| | | | | | Just to make sure it has a default value and is guaranteed initialized befor it is being used. | ||||
* | Security: Add CSRF checks when updating concerts table. | Harald Eilertsen | 2022-03-12 | 1 | -1/+17 |
| | |||||
* | Move code to update concert table to concert table class. | Harald Eilertsen | 2022-03-10 | 2 | -108/+107 |
| | | | | | | 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 Eilertsen | 2022-03-10 | 1 | -7/+0 |
| | |||||
* | Refactor gig import code. | Harald Eilertsen | 2022-03-10 | 1 | -63/+64 |
| | | | | | | | | | Split into two functions, and drop the nesting of the ifs. Technically the ifs didn't need to be nested in the first place, since each error case would break the execution flow using `continue` anyways. Throw parsing errors instead of having to keep pass all the state into the function that only need to parse one line. | ||||
* | Fix misc errors detected by psalm. | Harald Eilertsen | 2022-03-10 | 1 | -17/+34 |
| | | | | | - Lacking type info in venue.php - Potentially dereference null-objects in giglog_admin_page.php | ||||
* | Security: Escape imported dates before outputing in error messages. | Harald Eilertsen | 2022-03-10 | 1 | -3/+2 |
| | |||||
* | Fix more psalm issues | Harald Eilertsen | 2022-03-09 | 3 | -1/+5 |
| | |||||
* | Merge remote-tracking branch 'andrea/andreaschanges' into dev | Harald Eilertsen | 2022-03-09 | 3 | -26/+83 |
|\ | | | | | | | And fixup most whitespace issues while at it. | ||||
| * | import giglog checks for empty values and ivnalid date | AndreaChirulescu | 2022-03-01 | 1 | -32/+53 |
| | | |||||
| * | iCal file download fixes to generate one file per concert | AndreaChirulescu | 2022-02-27 | 4 | -325/+209 |
| | | | | | | | | giglog import fixes to catch errors and ignore old concerts | ||||
| * | beginning of ical setup with kigkonsults calendar | AndreaChirulescu | 2022-02-21 | 1 | -0/+162 |
| | | |||||
| * | Pagination fix to resest page if out of range | AndreaChirulescu | 2022-02-13 | 1 | -1/+6 |
| | | |||||
* | | Fix various psalm errors. | Harald Eilertsen | 2022-03-09 | 4 | -3/+5 |
| | | |||||
* | | Some cleanup in render_concerts_table | Harald Eilertsen | 2022-02-22 | 1 | -33/+23 |
|/ | |||||
* | Fix so pagination remembers filters in shortcode too. | Harald Eilertsen | 2022-02-13 | 1 | -4/+4 |
| | |||||
* | Remember filter settings when paginating. | Harald Eilertsen | 2022-02-13 | 1 | -33/+60 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch does changes quite a bit of how things work under the hood. It changes the filter settings from using POST to using query args in a GET request instead. This has some challenges: 1. The query args from the form will overwrite any query args in the original URL, so we have to keep the query args that was passed in as hidden fields in the form. 2. Since we try to keep the passed in query args, we need to filter out the query args that we expect to set in the form, otherwise query args that are dropped by the form will still be present as hidden args. Not too happy about how this works tbh, but at least it seems to work reasonably well for now. I've also renamed the args set by the form to make the URL a bit nicer, i.e "city" instead of "selectcity" etc for the query arg keys. I've also refactored quite a bit to make the code more manageable. Like using the view helper to create the month selector, so it's consistent with the others, and remembers it's selection. (I also simplified it to simply list the months from jan to dec, instead of starting at the current month. If it's important to keep the old behaviour I'll change it back.) There's a few problems still with this implementation. 1. It only works in the admin-section for now. Not sure why. Probably because shortcodes... 2. In the admin section it gains the `cid` and `pid` query args that should not be there. Not sure why. 3. If we change the filters while being on a page larger than the total number of pages after the filter change, we stay on the page even if it's not a valid page after the filter change. This should be trivial to fix. | ||||
* | Refactor rendering of the concerts table. | Harald Eilertsen | 2022-02-12 | 1 | -58/+79 |
| | | | | | | | | The main render function was getting too large and difficult to work with. Splitting it up a little to make it more manageable. Also fix styling to make the pagination links fall on one line a bit nicer. | ||||
* | Fix pagination of concert table. | Harald Eilertsen | 2022-02-12 | 1 | -6/+7 |
| | | | | | | | There's potential vulnerabilities here still, and I'm unsure if get_permalink() is entirely safe to use in this context. Seems to do the right thing, but the docs for the function indicates it has some edge cases. | ||||
* | More indentation and format if statements. | Harald Eilertsen | 2022-02-12 | 1 | -124/+134 |
| | |||||
* | more pagination details fixes | AndreaChirulescu | 2022-02-11 | 1 | -26/+57 |
| | |||||
* | css small fixed for table display | AndreaChirulescu | 2022-02-09 | 2 | -75/+116 |
| | | | | added pagination to concert list | ||||
* | Added a div to make table scrollable on mobiles | AndreaChirulescu | 2021-09-24 | 1 | -2/+2 |
| | |||||
* | Changed order of date column in concerts table | AndreaChirulescu | 2021-09-19 | 1 | -4/+3 |
| | |||||
* | Padding in table so text doesn't connect between cells | AndreaChirulescu | 2021-09-19 | 1 | -2/+2 |
| | | | | Added a paragraph to have a bit of space under filters | ||||
* | Add CSRF checks to new venue form | Harald Eilertsen | 2021-09-17 | 2 | -0/+8 |
| | |||||
* | 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 |
| | |||||
* | 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 |