Commit Graph

577 Commits

Author SHA1 Message Date
Dylan Wagstaff 9d9ab35cc6
Merge pull request #294 from AdamSawoscianik/patch-1
Update Installing.md
2019-10-04 10:07:48 +13:00
AdamSawoscianik d5b9a2818c
Update Installing.md
Adding information about $CommentsForm tag to help people with this module.
2019-10-03 10:32:22 +02:00
Guy Marriott e21a09234a
Update translations 2019-08-20 16:37:58 +12:00
Jason Irish fbfead9c9e BUGFIX CommentAdmin implements PermissionProvider
fixes #288
2019-08-15 09:50:36 +12:00
Will Rossiter b3fe6d82b4 Add legacy YAML for upgrading 2019-08-15 09:50:27 +12:00
Heath Dunlop 617501efed comments extension filters on Parent Class
This allows both pages and DataObjects to have comments without the ParentID clashing and showing comments on a page where the ParentID is the Same as a DataObjects or even two DataObjects having the sameID
2019-08-15 09:50:19 +12:00
Robbie Averill f4c82f238e
Merge pull request #292 from jsirish/bugfix/adminPermissionProvider
BUGFIX CommentAdmin implements PermissionProvider
2019-07-15 22:03:28 +01:00
Jason Irish c0f0c99e56 BUGFIX CommentAdmin implements PermissionProvider
fixes #288
2019-07-15 12:02:50 -05:00
Dylan Wagstaff f8eeeee4b8
Merge pull request #290 from GOVTNZ/pulls/legacy
Add legacy YAML for upgrading
2019-07-03 12:47:47 +12:00
Will Rossiter cad9369f7b
Add legacy YAML for upgrading 2019-07-03 11:40:42 +12:00
Robbie Averill c16ca56408
Merge pull request #289 from creative-commoners/pulls/3.2/focus-comment-submission-message
NEW Add tabindex="-1" to comment submission message for a11y support
2019-06-28 16:15:27 +12:00
Garion Herman 67167c0809 NEW Add tabindex="-1" to comment submission message for a11y support 2019-06-28 15:56:09 +12:00
Robbie Averill 8397bb5792
Merge pull request #287 from creative-commoners/pulls/master/update-vuln-dependencies
Update @silverstripe/webpack-config to patch vulnerable deps, add NVM config
2019-06-14 15:04:14 +12:00
Garion Herman 2d0d949fc3 Update @silverstripe/webpack-config to patch vulnerable deps 2019-06-14 14:47:09 +12:00
Garion Herman 14675b53bf Add NVM config 2019-06-14 13:59:23 +12:00
Robbie Averill 9c4ec173fd
Merge pull request #285 from dunatron/patch-2
comments extension filters on Parent Class
2019-05-16 15:33:09 +12:00
Heath Dunlop 82845626bc
comments extension filters on Parent Class
This allows both pages and DataObjects to have comments without the ParentID clashing and showing comments on a page where the ParentID is the Same as a DataObjects or even two DataObjects having the sameID
2019-05-16 13:46:28 +12:00
Robbie Averill 780ea22b93 Merge branch '3.2' 2019-05-10 09:35:00 +12:00
Robbie Averill f65f5b5697 FIX Remove reliance on translations in fieldLabels test, run textcollector, remove deprecated code from CommentsTest 2019-05-10 09:33:18 +12:00
Guy Marriott 74a317ab46
Update translations 2019-05-09 15:44:57 +12:00
Robbie Averill 161f5f88bd Merge branch '3.2' 2019-05-08 11:28:18 +12:00
Robbie Averill 67faa822f7 Update dependencies 2019-05-08 11:26:56 +12:00
Guy Marriott aec3e2ba5b
FIX "Allow Comments" label now reads "Allow comments" - should not have been capitalised (#283)
FIX "Allow Comments" label now reads "Allow comments" - should not have been capitalised
2019-05-02 13:12:10 +12:00
Robbie Averill 89204c0c1c FIX "Allow Comments" label now reads "Allow comments" - should not have been capitalised 2019-05-02 11:31:32 +12:00
Robbie Averill d8602d0e02 Merge branch '3.2' 2019-04-15 15:34:34 +12:00
Robbie Averill 4d19e6c43e Merge branch '3.1' into 3.2 2019-04-15 15:33:53 +12:00
Robbie Averill 303bdef1c8 Remove obsolete branch alias and add PHP 7.3 to Travis builds 2019-04-15 15:33:44 +12:00
Guy Marriott 1358594070
Merge pull request #275 from creative-commoners/pulls/3.1/update-jquery-validator
Update jQuery.validation to 1.19.0
2019-04-05 09:17:06 +13:00
Dylan Wagstaff 6fcc040485 Update jQuery.validation to 1.19.0
jQuery 3.3.1 is still the latest release
2019-03-25 09:56:58 +13:00
Dylan Wagstaff 8b178b63ac
Merge pull request #280 from creative-commoners/pulls/master/improve-compat-with-bulk-editing-tools
Adjust the output of CommentHandler to use HTTPBulkToolsResponse
2019-03-24 13:34:21 +13:00
Garion Herman 1e4378d212 Tighten version constraint for Bulk Editing Tools dependency 2019-03-22 15:53:12 +13:00
Robbie Averill f0998faafb
Allow translation of 'Changes applied' message on CommentHandler
Co-Authored-By: Cheddam <cheddam@me.com>
2019-03-22 15:43:19 +13:00
Garion Herman 74afde6d9b Adjust the output of CommentHandler to use HTTPBulkToolsResponse
GridFieldBulkEditingTools has introduced a common output schema
in the form of HTTPBulkToolsResponse. This commit adopts that API.
2019-03-22 15:00:19 +13:00
Garion Herman 9ff1db403e
Merge pull request #277 from creative-commoners/pulls/4.0/infix-spam-flicks
NEW relocate spam and approve admin actions
2019-03-22 11:10:29 +13:00
Robbie Averill ce1e934d57
Merge pull request #278 from SpeksForks/doc-link
Fix documentation link
2019-02-19 05:09:33 +07:00
Peter Thaleikis 005f58f1ee
Fix documentation link 2019-02-19 02:32:09 +07:00
Robbie Averill 647c634dc4
Update .travis.yml
Co-Authored-By: NightJar <NightJar@users.noreply.github.com>
2019-02-11 09:22:02 +13:00
Guy Marriott 7d60fc2fe2
Update src/Admin/CommentsGridFieldSpamAction.php
Co-Authored-By: NightJar <NightJar@users.noreply.github.com>
2019-02-11 09:19:31 +13:00
Guy Marriott 18fbdfc94b
Update src/Admin/CommentsGridFieldApproveAction.php
Co-Authored-By: NightJar <NightJar@users.noreply.github.com>
2019-02-11 09:19:18 +13:00
Robbie Averill c4cfc44643 Update Scrutinizer configuration to use new build engine 2019-02-10 07:01:57 +03:00
Dylan Wagstaff b6a2c608ac NEW relocate spam and approve admin actions
Previous versions had individual spam and approve buttons for a comment
in the admin area on the GridField row. However with the upgrade to
SilverStripe 4, and particularly 4.2, these were having layout issues
with the new GridField Action Menu component that groups actions
together.

The solution here is to put them into aforementioned gridfield action
menu component, with the other actions for that row. However this
requires two separate grid field components (one each for the two
comment actions) - previously these were a single component that output
two buttons instead of one each. This also reduces coupling, which is
nice :)

The previous class is still maintained for backwards compatibilty, but
is deprecated.
2019-02-08 15:19:45 +13:00
Robbie Averill d504d4a847 Merge branch '3.1' 2019-01-11 09:13:31 +01:00
Guy Marriott f5c7024aaf
Merge pull request #276 from creative-commoners/pulls/3.1/deprotocol-my-site
Validate protocol-less URLs
2019-01-11 15:18:27 +13:00
Dylan Wagstaff 3812057b00 Add ability for commenters to enter a URL without a protocol
When posting a comment on the a page with this module applied, there is
an optional input for the commenter to give their URL,presumably their
website. However this input currently validates (via JavaScript) to
allow URLs only iff they have a protocol.

Common use cases when someone is asked for their website in my
experience is to then receive a URL without a protocol, confounded in
that most web browsers will accept this form and automatically add the
http protocol, where a webserver may then redirect to https by default.
This means that all the magic happens behind the scenes and most folks
don't particularly care to think about protocols when entering web
addresses.

Yest this input will only validate true and allow comment submission if
they do.

So now it will allow a protocol-less entry into the URL box.
2019-01-11 15:01:34 +13:00
Guy Marriott 20c6a50f12
Merge pull request #273 from creative-commoners/pulls/3.1/name-that-member
FIX show name of members in summary tables
2019-01-11 14:56:20 +13:00
Dylan Wagstaff 0535d29f8d
Merge pull request #274 from creative-commoners/pulls/3.1/i-de-test-those-labels
FIX Correcting inconsistent capitalisation of label and cleaning up tests
2019-01-11 14:49:02 +13:00
Guy Marriott 7691237fe5
FIX Correcting inconsistent capitalisation of label 2019-01-11 14:17:36 +13:00
Guy Marriott a57c83c121
Removing superflous i18n test and updating label test to only test labels specifically added 2019-01-11 12:22:31 +13:00
Dylan Wagstaff dc1f8622e0 FIX show name of members in summary tables
Anonymous comments (posted by the public at large) must have a name
and an email address associated with them. On the other hand, a logged
in user will have the `Member` record details used for this information,
via the Author relationship.

However the summary fields do not allow for this, and only reference
Name and Email on the Comment model directly, so any comment posted by a
logged in member has no data for name and email displayed in the various
GridFields in the CMS for administering comments (either per page, or in
the global ModelAdmin).

To recitfy this we can change the summary fields to use getter methods
that will return the Comment model info, or fall back to the Author
associated Member record if Name and Email are unset on the Comment.
2019-01-11 11:48:51 +13:00
Dylan Wagstaff 4ee9720081
Merge pull request #269 from creative-commoners/pulls/3.1/moderation-labels
FIX Capitalise comment moderation action labels and update i18n syntax in templates
2018-11-28 14:36:30 +13:00