Any client-side hook before new topic submit?

[Bruce Lee]

Use-case

1. User create a new topic.
2. User click submit button.
3. Client-side check if the content is empty. If content is empty, insert "xxx" to content.

Is there any client-side hook that invoke upon topic submission (#2) before going to server?

Thanks!

[yariplus]

No, afaik, there are no filter hooks for the client-side.

Is there a reason why this can't be server-side? It seems to me to be a natural fit for the server hooks.

In any case, you could attach a click handler to the submit button on during the composer load hook, and edit the post there.

[Bruce Lee]

@yariplus Thanks for your suggestions. I just found out that the default composer validates for 0 content length on the client-side, which mean I have to modify the codes there. And the reason it has to be on client-side is because the composer will block the topic submission if the content is empty.

[yariplus]

Oh, interesting. In that case I would use jquery to attach a .click listener on the submit button, checking for and inserting the textarea content. A little messy, but it should work.

[Bruce Lee]

@yariplus hmm you are right, thanks. Somehow I have to make the ".click' listener trigger first (ordering) before the composer listener.

[yariplus]

Uh, hmm, proxy composer.enhance, this is called after the button is created but before the event is.

require(['composer'], function (composer) {
  var orig = composer.enhance;
  composer.enhance = function (postContainer, post_uuid, postData) {
    postContainer.find('.composer-submit').click(function () {
      postContainer.find('textarea').val('This is automagically posted.');
    });
    orig(postContainer, post_uuid, postData);
  };
});

[Bruce Lee]

@yariplus Great, this is what i need

[baris]

We have a hook that is fired client side right before the data is sent to the server.

$(window).trigger('action:composer.submit', {composerEl: postContainer, action: action, composerData: composerData});

https://github.com/NodeBB/nodebb-plugin-composer-default/blob/master/static/lib/composer.js#L577

[yariplus]

@baris The issue here is, the code will never reach that point if the post content is less than the minimum post length/empty.

[baris]

Maybe we should move those checks after the hook then?