Magnolia Forum Module

Allow paragraphs to create a thread if it doesn't exist

Details

  • Type: New Feature New Feature
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.0
  • Component/s: None
  • Labels:
  • Description:

    Similar to MGNLFORUM-30, we should have the possibility to automatically create threads if they don't exist. This would in effect give us "discuss this article" and "comments" features.

  1. MGNLFORUM-71.txt
    (2 kB)
    Grégory Joseph
    05/Feb/09 4:17 PM

Issue Links

Activity

Hide
Grégory Joseph added a comment - 27/Aug/08 1:12 PM

Got a local p.o.c working. Need to see how it behaves with regards to permissions and on a public instance (typically for a page commenting scenario)

Show
Grégory Joseph added a comment - 27/Aug/08 1:12 PM Got a local p.o.c working. Need to see how it behaves with regards to permissions and on a public instance (typically for a page commenting scenario)
Hide
Jan Haderka added a comment - 04/Feb/09 11:55 PM

Can you make patch from that POC and attach it here? Ta.

Show
Jan Haderka added a comment - 04/Feb/09 11:55 PM Can you make patch from that POC and attach it here? Ta.
Hide
Grégory Joseph added a comment - 05/Feb/09 4:17 PM
  • extracted setupThreadIdFromContent() and setupForumIdFromContent() from ThreadList and ThreadView
  • added ThreadViewAutoCreate
Show
Grégory Joseph added a comment - 05/Feb/09 4:17 PM
  • extracted setupThreadIdFromContent() and setupForumIdFromContent() from ThreadList and ThreadView
  • added ThreadViewAutoCreate
Hide
Jan Haderka added a comment - 18/Feb/09 2:04 PM

Proposed solution worked fine for page commenting which handles all its needs in one action/model, but it is not enough for general forums where thread viewing in handled by action and message creation is handled solely by ForumController and first message needs to be created together with the forum because of the "firstMessage" reference property. Either we make this property non mandatory so thread can be created without it or we need to rework the flow so the "auto create" threads can be recognized and handled by actions appropriately (i.e. return empty list of messages for the thread that is to be auto-created and doesn't exist yet).

Show
Jan Haderka added a comment - 18/Feb/09 2:04 PM Proposed solution worked fine for page commenting which handles all its needs in one action/model, but it is not enough for general forums where thread viewing in handled by action and message creation is handled solely by ForumController and first message needs to be created together with the forum because of the "firstMessage" reference property. Either we make this property non mandatory so thread can be created without it or we need to rework the flow so the "auto create" threads can be recognized and handled by actions appropriately (i.e. return empty list of messages for the thread that is to be auto-created and doesn't exist yet).
Hide
Jan Haderka added a comment - 12/Jun/09 3:39 PM

Forum Module provide methods for paragraphs to use to create new Threads when necessary. The only limitation of the current solution is the fact that Thread has to be created together with the first message.

Show
Jan Haderka added a comment - 12/Jun/09 3:39 PM Forum Module provide methods for paragraphs to use to create new Threads when necessary. The only limitation of the current solution is the fact that Thread has to be created together with the first message.

People

Dates

  • Created:
    31/Jul/08 12:26 PM
    Updated:
    12/Jun/09 3:39 PM
    Resolved:
    12/Jun/09 3:39 PM