Weird bug with discourse-assign

Firepup650 · August 21, 2023, 10:15pm

We have this random topic on our forum that has somehow managed to become invisible. Deleting and restoring the topic via tha API has had no effect. It seems like discourse-assign causes it to fail to load, as it loads perfectly fine when plugin JS is disabled using safe mode.
You can find this buggy post here:

Here’s a stack trace for one of the errors we get when opening the topic:

Cannot read properties of undefined (reading 'includes')
at n.<anonymous> (https://amcforum.wiki/assets/plugins/discourse-assign-e7635afd56f3ebe38bb2866d6121e1e726de50fbfc472646a6e738a3d44b3930.js:208:1008)
at https://amcforum.wiki/assets/discourse-12f38708c935010f61959d1ee5b74a9770c91ed393b88e3a7a5fe0a3de467a89.js:9931:74
at Array.forEach (<anonymous>)
at n.buildClasses (https://amcforum.wiki/assets/discourse-12f38708c935010f61959d1ee5b74a9770c91ed393b88e3a7a5fe0a3de467a89.js:9931:51)
at n.draw (https://amcforum.wiki/assets/discourse-12f38708c935010f61959d1ee5b74a9770c91ed393b88e3a7a5fe0a3de467a89.js:10348:34)
at n.render (https://amcforum.wiki/assets/discourse-12f38708c935010f61959d1ee5b74a9770c91ed393b88e3a7a5fe0a3de467a89.js:10324:54)
at a (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:331:20)
at e.exports (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:333:34)
at e (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:337:6)
at Object.e [as render] (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:342:48)
at https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:976:62
at e.exports (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:977:40)
at u (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:952:8)
at Object.l [as patch] (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:949:3)
at e.exports (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:981:29)
at u (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:952:8)
at Object.l [as patch] (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:949:3)
at e (https://amcforum.wiki/assets/chunk.660.cfa44cc656d26c5d04fa-3aa2531cc98e968234ee473ed8fc2ef14895f4ba22e888d7d542e7d3620f8056.js:953:114)
at e.rerenderWidget (https://amcforum.wiki/assets/discourse-12f38708c935010f61959d1ee5b74a9770c91ed393b88e3a7a5fe0a3de467a89.js:1844:163)
at p.invoke (https://amcforum.wiki/assets/vendor-d19c5d3106737be843b2f1f98da81f661a0e985585b4a1fd14cab77cbfa60e37.js:4339:182)
at p.flush (https://amcforum.wiki/assets/vendor-d19c5d3106737be843b2f1f98da81f661a0e985585b4a1fd14cab77cbfa60e37.js:4331:141)
at h.flush (https://amcforum.wiki/assets/vendor-d19c5d3106737be843b2f1f98da81f661a0e985585b4a1fd14cab77cbfa60e37.js:4346:207)
at q._end (https://amcforum.wiki/assets/vendor-d19c5d3106737be843b2f1f98da81f661a0e985585b4a1fd14cab77cbfa60e37.js:4410:9)
at _boundAutorunEnd (https://amcforum.wiki/assets/vendor-d19c5d3106737be843b2f1f98da81f661a0e985585b4a1fd14cab77cbfa60e37.js:4358:595)

Lilly · August 21, 2023, 11:10pm

yea the console is showing errors here.

github.com

discourse/discourse-assign/blob/0cbf10b8055370445bd36536e51986bf48bdc57e/assets/javascripts/discourse/initializers/extend-for-assigns.js#L482C1-L486C6


      
          api.addPostSmallActionClassesCallback((post) => {
            if (post.actionCode.includes("assigned") && !siteSettings.assigns_public) {
              return ["private-assign"];
            }
          });

sam · August 21, 2023, 11:38pm

Its a small action with no action code!

You can fix this using the console, enter the container find the small actions on the topic, one of them is missing an action code.

We should probably add a sql validation to ensure this is impossible.

hello-smile6 · August 21, 2023, 11:39pm

I think those can be created using the API (we have a theme component that creates one), will we have to manually run some kind of database migration?

hello-smile6 · August 21, 2023, 11:42pm

Here’s an example of another one created using our theme component:

hello-smile6 · August 21, 2023, 11:48pm

It seems like 25 different topics all have posts that are breaking discourse-assign, according to the results of this SQL query (using data explorer):

SELECT id topic_id FROM topics topic WHERE topic.id IN (SELECT topic_id FROM posts WHERE post_type = 3 AND action_code IS NULL)

sam · August 21, 2023, 11:48pm

GPT bot to the rescue…

On another topic

Thank you for the additional information. Given the code snippet from the topic.rb file, it’s clear that small actions are indeed created as a type of post with post_type set to Post.types[:small_action] and an action_code associated with it.

If we want to find small actions with no action_code, we can adjust the query to look for posts where post_type is 3 (small action) and action_code is nil.

Here are the revised steps:

Enter the container: You can enter the container by using the command ./launcher enter app from the command line in your Discourse directory.

Run rails c: Once inside the container, you can start the Rails console by running the command rails c.

Find the Topic: To find the topic, you can use the Topic.find_by(id: your_topic_id) command. Replace your_topic_id with the ID of the topic you are looking for.

Find the Small Action with no action_code: Small actions are stored in the posts table with post_type set to 3 and action_code is nil. You can find the small action related to your topic with sa = Post.where(topic_id: your_topic_id, post_type: 3, action_code: nil).

Delete the Small Action: Once you have the small action, you can delete it by running sa.destroy_all.

Please make sure to replace your_topic_id with the actual ID of your topic. Also, be careful when deleting records, as it can’t be undone.

Let me know if you have any questions or need further assistance!

Firepup650 · August 22, 2023, 1:25am

If that’s the check, why not just add !!post.actionCode to the if block?

Firepup650 · September 12, 2023, 11:07am

Finally got around to this, and PR’d a change:

Topic		Replies	Views
I could not edit tags for old topics Support	10	640	May 6, 2022
Default pinned topic does not show due to policy plugin Support	10	649	September 17, 2019
Edits not displayed and timing out Bug fixed	12	164	November 9, 2024
Reactions plugin: 500 Internal Server Error Bug reactions	9	485	October 28, 2022
Endless spinner on some posts Bug fixed	14	142	September 12, 2024

Weird bug with discourse-assign

Related topics