I have a use case where a Write only option would be required. Meaning only the reply form would appear, but not the reply thread.

CommentFileSizeAuthor
#12 reply-write-only-option-1552388-12.patch7.33 KBacrazyanimal
#7 reply-write-only-option-1552388-6.patch7.08 KBacrazyanimal
#1 reply-write-only-option-1552388.patch7.2 KBacrazyanimal

Comments

acrazyanimal’s picture

acrazyanimal commented
Status: Active » Needs review
StatusFileSize
new reply-write-only-option-1552388.patch7.2 KB

Here is a simple patch.

james.williams’s picture

james.williams
Location Coventry, UK
commented

I agree that this would be very good. It's something I looked at doing with Onecomment, but went about it a different way (using a separate setting to show/hide the comments, so the form etc could be configured separately without touching the core Comment module settings).

acrazyanimal’s picture

acrazyanimal commented

I had just tried to integrate it as easily as possible into your existing framework and settings without disrupting any of your core functionality. I think the patch above should probably suffice for most use cases.

Just in case you are interested:
My particular use case is that I want people to be able to respond to job postings. However, I don't want a comment stream, but I do want to be able to track those responses. My use case will involve rules that send the job posting author (HR manager) an email when a new application (reply) is received. Also, there will be views on user pages, one showing jobs applied for, for those people filling out applications, and another admin view that would show links to applications received for an HR manager's job postings.

james.williams’s picture

james.williams
Location Coventry, UK
commented

You've got a really interesting use case there - did you consider any other options of how to implement that? Reply seems to be ideal, I'd just be interested to know to see how we can strengthen Reply. Presumably in your use case, it would be good for users to only be able to submit one reply? That's something I implemented in Onecomment, and really want to see in Reply one day. Thanks for the strengthening my resolve to do so :-)

acrazyanimal’s picture

acrazyanimal commented

Yes I looked around and considered more then a few possibilities. The D6 job postings is EOL D6 and recruit went had a very short life and went nowhere. Recruiter looks promising but I don't like how they are not releasing through D.O and its likely overkill for what I actually need.

Also, I had created a module for D6 called node_reply and was going to update it to D7 as an all encompassing entity reply module. However, I decided to search around before I did and came across your D7 'reply' module. It looks like you have a good jump on it so I figured I would try it out.

Anonymous’s picture

Anonymous (not verified) commented

Good idea, I like this feature. I'll look into it when I'll have time.

acrazyanimal’s picture

acrazyanimal commented
StatusFileSize
new reply-write-only-option-1552388-6.patch7.08 KB

Here is an updated copy of the patch applied against the current dev.

james.williams’s picture

james.williams
Location Coventry, UK
commented

I've only glanced at the patch, but the concept is good. I've rewritten the way access checks are done in the module, to use a new reply_access() function, similar to what comment_access() or node_access() do, so this can be called from anywhere (such as the new views field handlers). For this write-only option to work, it would need adding into that function. Perhaps we could be really clever and do things with bitwise masking to determine read/write access? The constants might have to change for this to work. I've used up my time for the week on the module, so you can be assured that I won't change anything more this week - sorry that you've had to re-roll the patch once already!

acrazyanimal’s picture

acrazyanimal commented

The way the patch is written it doesn't affect permissions or access. It defines a new constant for write only and does use bitwise masking to determine if the reply form and/or reply thread should be used. If you have permissions already for reading replies and creating new replies then that should be all that is needed.

mgifford’s picture

mgifford
he/him
Primary language English
commented

@james.williams - any thoughts on getting this into the repository.

james.williams’s picture

james.williams
Location Coventry, UK
commented

I don't have time to prioritise this - but thanks for the nudge, it might help. I certainly want to get something like this committed at some point in the next few weeks.

acrazyanimal’s picture

acrazyanimal commented
StatusFileSize
new reply-write-only-option-1552388-12.patch7.33 KB

Updated the patch again. It includes an update for the write only option in reply_access.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

If you set the permissions so that users can post but not view, don't you have the same thing?