Conversation Essentials [Paid] 1.14.1

[Daniel Hood]

Well, I'm not sure why those 2 template mods aren't being applied. Does anyone else have those not being applied?

The post ratings compatibility add-on isn't anything new, it's just been split because not everyone needs it and it kept accidently being removed for people. It's just the same functionality that existed in much earlier versions being separated from the main Conversation Essentials add-on.

 

[The Sandman]

Or am I missing the point entirely, and it won't actually show the post ratings??

TAZ's template modification looks like that and works properly:


Post Ratings aren't available in PC messages, only Likes are, so I'm not sure what isn't showing?

 

[MattW]

I've been able to get them working by editing the template modifications, and copying the specific block in from the template.

<xen:include template="news_feed_item">
                <xen:map from="$content" to="$item" />
                <xen:set var="$itemTemplate"><xen:include template="news_feed_item_postrating_rate"><xen:map from="$content" to="$user" /></xen:include>
                </xen:set>
                <xen:set var="$itemDate">{$content.date}</xen:set>
            </xen:include>
               
               
               
<xen:include template="news_feed_item">
                <xen:map from="$content" to="$item" />
                <xen:set var="$itemTemplate"><xen:include template="news_feed_item_postrating_rate"><xen:map from="$content" to="$user" /></xen:include>
                </xen:set>
                <xen:set var="$itemDate">{$content.date}</xen:set>
            </xen:include>

Those are the two, one from the template, and one from the modification. I don't personally see a difference, but it now works since I copied the block in.

This is the code which is now working

<xen:include template="news_feed_item">
                    <xen:map from="$content" to="$item" />
                    <xen:set var="$itemTemplate"><xen:include template="news_feed_item_postrating_rate"><xen:map from="$content" to="$user" /></xen:include>
                    </xen:set>
                    <xen:set var="$itemDate">{$content.date}</xen:set>
                </xen:include>

So it's just a compatibility fix, and not extending the post ratings into the conversations?

 

[Daniel Hood]

So it's just a compatibility fix, and not extending the post ratings into the conversations?

Correct. I didn't mean to give the impression that it was something more. I've been stating that it's the same functionality, just isolated from the rest of the add-on.

What is the rest of that code changing? Like can you show a before/after screenshot or something? I'm not really sure what's going on here.

 

[The Sandman]

This functionality works great - even better than expected! It should be part of the core!

 

[MattW]

What is the rest of that code changing? Like can you show a before/after screenshot or something? I'm not really sure what's going on here.

The first 2 bits of code are the ones from the original template, and the template modification.

The 3rd one, is the one I copied and pasted into the template modification, and it started working. They all look exactly the same to me, so I've no idea why it wasn't working either.

 

[Mr Lucky]

Unable to upgrade to 1.1o.1

 

[Mr Lucky]

And now if I try to rebuild search index I get this:

Fatal error: Call to undefined method Valhalla_Export_Conversation_Model_Conversation::getMessageIdsInRange() in /home/xxxx/public_html/library/ConvEss/SearchHandler/Message.php on line 115

 

[MattW]

And now if I try to rebuild search index I get this:

Fatal error: Call to undefined method Valhalla_Export_Conversation_Model_Conversation::getMessageIdsInRange() in /home/xxxx/public_html/library/ConvEss/SearchHandler/Message.php on line 115

That looks like it's a conflict between the ConvEss addon, and the Valhalla Export Conversations addon you have installed.

 

[Mr Lucky]

That looks like it's a conflict between the ConvEss addon, and the Valhalla Export Conversations addon you have installed.

Odd because they have both been installed for over a year, and I never had this issue until I attempted (unsuccessfully) to upgrade Conversation Essentials.

Is there a known conflict between these two addons?

 

[MattW]

Odd because they have both been installed for over a year, and I never had this issue until I attempted (unsuccessfully) to upgrade Conversation Essentials.

But, you are going from 1.2.9 to 1.10.1 of ConvEss, and there have been a lot of changes. Something could have been introduced that is causing the conflict.

 

[Daniel Hood]

Can you please confirm all files are uploaded and then try to install it again?

 

[Mr Lucky]

But, you are going from 1.2.9 to 1.10.1 of ConvEss, and there have been a lot of changes. Something could have been introduced that is causing the conflict.

No, I still have 1.2.9 installed, I was unable to upgrade due to this:

 

[Mr Lucky]

Can you please confirm all files are uploaded and then try to install it again?

And now if I try to rebuild search index I get this:

Fatal error: Call to undefined method Valhalla_Export_Conversation_Model_Conversation::getMessageIdsInRange() in /home/xxxx/public_html/library/ConvEss/SearchHandler/Message.php on line 115

OK this is fine now. it appears that after the unsuccessful upgrade attempt to 1.10.0 I reuploaded the old 1.2.9 backup, but some files were in there that shouldn't be.

Have now reuploaded the 1.2.9 files from original folder, and rebuild search index is now no problem

So will now attempt the upgrade again.

 

[Mr Lucky]

Can you please confirm all files are uploaded and then try to install it again?

This time it installed. Strange.

 

[Mr Lucky]

Where wouldf I find the feature to copy a conversation to a thread - I have permission set, but cannot see where I would do that.

 

[Daniel Hood]

When viewing a conversation, check the box next to each message.

 

[Mr Lucky]

OK, found that and it works well.

One small issue (for me) is that when a message is copied into a thread/forum, you get an alert of a new post. That is OK, but if I copy a message from a member into e.g. a private admin forum, the admins of course get an alert that the member just "replied to a thread" they are not allowed to post in.

In one way this makes sense, but in another way it is very disconcerting to see an alert of a reply from a non-staff member in the staff forum.

It would be better if instead of "replied to a thread" the alert could make it clear that it was copied from a PC to avoid confusion.

 

[Daniel Hood]

That is extremely unlikely to change. It would go against a lot of how XenForo works by default. The thing is, these new posts are legit posts and therefore go through all the typical methods. They trigger alerts and emails for the people watching the nodes, they affect post count, they're posts. Assigning them a different/special alert would be extremely difficult, and quite frankly it'd be pretty odd as well.

I get where you're coming from but it's just not feasible to change it.

 

[Mr Lucky]

I get where you're coming from but it's just not feasible to change it.

I understand that, I thought it might be the case.

I think the odd thing is if they are copied into an existing thread. Mostly admins would copy messages to a new thread and make it very clear to other admins (maybe in the title) it is copied from a PC.

I wonder if it is possible with custom fields to automatically insert a notice at the top of copied posts, e.g. Copied from Conversation.