summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fix pagination of concert table.Harald Eilertsen2022-02-121-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 Eilertsen2022-02-121-124/+134
|
* Fix indentation and add license info.Harald Eilertsen2022-02-122-101/+104
|
* more pagination details fixesAndreaChirulescu2022-02-112-26/+61
|
* css small fixed for table displayAndreaChirulescu2022-02-094-86/+133
| | | | added pagination to concert list
* Made giglog visible for admins onlyAndreaChirulescu2022-02-071-92/+98
|
* Added a div to make table scrollable on mobilesAndreaChirulescu2021-09-241-2/+2
|
* Changed order of date column in concerts tableAndreaChirulescu2021-09-191-4/+3
|
* Padding in table so text doesn't connect between cellsAndreaChirulescu2021-09-192-2/+7
| | | | Added a paragraph to have a bit of space under filters
* Add CSRF checks to new venue formHarald Eilertsen2021-09-172-0/+8
|
* Move New Venue form to a separate partial.Harald Eilertsen2021-09-173-9/+45
|
* Return better error for failed CSRF check on import.Harald Eilertsen2021-09-171-2/+2
|
* Add CSRF checks for edit concert form.Harald Eilertsen2021-09-172-1/+9
|
* Clean up, fix and rename db tables.Harald Eilertsen2021-09-165-95/+72
| | | | | | | | | | | | | | | | | 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.
* Fix ConcertTest::testFetchConcertByVenueAndDate.Harald Eilertsen2021-09-161-1/+1
| | | | | | | | Since the date column has changed types, so has the representation in the concert class. We should really change the property in the Concert class be a proper DateTime object.
* Reinsert created and updated columns for venues.Harald Eilertsen2021-09-161-0/+2
| | | | Was accidentally dropped during rewrite of base schema.
* Facelift of database table creation to use create tables instead of ↵AndreaChirulescu2021-09-131-323/+17
| | | | versioning. First attempt
* Update ConcertsTable tests.Harald Eilertsen2021-09-111-3/+65
| | | | | Test that less privileged users don't see all the controls, and that no controls are rendered on the public facing pages.
* Change how we load the plugin.Harald Eilertsen2021-09-113-23/+28
| | | | | | | | | | | | | | Since we now have code that should be available, both on the public blog and in the admin section, we need to be more graular when loading the various parts of the plugin. We still try to avoid loading admin-only parts for the public blog, but allways load the parts that we need in either case. Also avoid running the db migrations when running unit tests, as the schema is copied over from the dev environment it just caues problems. Finally, don't hardcode unit tests to always be in_admin, but rather determine that for each test.
* tests: Make an actual test for ConcertsTableHarald Eilertsen2021-09-112-8/+27
| | | | | | | | | | | 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 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
* Fix show only own concerts again.Harald Eilertsen2021-09-061-1/+1
| | | | Missed quitation marks around search term.
* Merge remote-tracking branch 'andrea/andreaschanges' into devHarald Eilertsen2021-09-061-2/+2
|\
| * fixed proper class naming for Apply buttonAndreaChirulescu2021-09-061-1/+1
| |
| * removed some extra characters that got lost in table HTMLAndreaChirulescu2021-09-061-1/+1
| |
* | Refactor Concert::find_concerts.Harald Eilertsen2021-09-061-23/+28
|/ | | | | Make it a bit more compact and fix use of $wpdb->prepare for `currentuser` filter.
* Remove logging from Venue class.Harald Eilertsen2021-09-061-8/+0
| | | | Move logging to presentation/logic layer if necessary.
* Throw instead of log error from Concert class.Harald Eilertsen2021-09-062-18/+13
| | | | | 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 Eilertsen2021-09-062-5/+40
| | | | This also adds a number of new filters to find_concerts.
* Remove obsolete function Concert::findHarald Eilertsen2021-09-061-13/+0
|
* Use wpdb::get_row in Concert::get()Harald Eilertsen2021-09-061-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 Eilertsen2021-09-061-1/+0
|
* Fix indentation and whitespace, add braces where missing.Harald Eilertsen2021-09-061-41/+46
|
* Merge remote-tracking branch 'andrea/andreaschanges' into devHarald Eilertsen2021-09-055-4/+13
|\
| * added images for checkbox and rediconAndreaChirulescu2021-09-055-4/+13
| | | | | | | | added class for City dropdown form
* | security: Add proper CSRF checking for the import_gigs form.Harald Eilertsen2021-09-051-3/+7
|/
* used auto ident this timeAndreaChirulescu2021-09-051-165/+165
|
* used is_admin() to check if user shoudl see xtra columnsAndreaChirulescu2021-09-051-184/+184
| | | | Re-idented the file
* File to display concerts inside a page by calling getconcerts codeAndreaChirulescu2021-09-051-0/+14
|
* Extra changes to hide admincolumns on giglog page If logged in but not using ↵AndreaChirulescu2021-09-051-6/+8
| | | | plugin
* CHanges in the concerts table for logged/non logged user. Fixed row header ↵AndreaChirulescu2021-09-052-19/+31
| | | | with city
* Merge branch 'dev' of ↵AndreaChirulescu2021-09-052-4/+98
|\ | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges
| * Add test for ConcertsTableHarald Eilertsen2021-09-051-0/+94
| |
| * Fix some bugs in generated HTML for ConcertsTable.Harald Eilertsen2021-09-051-5/+5
| |
* | no idea why it keeps showing a n extra > but commiting to avoid a headache. ↵AndreaChirulescu2021-09-051-1/+1
|/ | | | This commit is useless
* Merge remote-tracking branch 'andrea/andreaschanges' into devHarald Eilertsen2021-09-057-18/+50
|\
| * Re-added the changes for looks, and city/venue selectorAndreaChirulescu2021-09-052-14/+14
| |
| * Merge branch 'dev' of ↵AndreaChirulescu2021-09-054-252/+292
| |\ | | | | | | | | | | | | | | | | | | https://code.volse.net/wordpress/plugins/gigologadmin.git into andreaschanges # Conflicts: # includes/admin/views/giglog_admin_page.php