Ok, after upgrading to Oxwall 1.8.2  I (once again) commented out that line in /ow_plugins/newsfeed/components/feed_item.php as per Bobby's post of Feb 22, 2013. 

Note: In Oxwall 1.8.2, the relevant line in feed_item.php is line 192

Since then, I'm getting this error message in my error.log file, over and over again:

[ow_core_log] [Notice] Message: Undefined variable: shouldExtend File: /home/hypwatch/public_html/ow_plugins/newsfeed/components/feed_item.php Line:195

Have I missed something?

The reason you are getting that notice (not fatal error) is that the variable $shouldExtend is not initialized before using it.  

I do not recommend commenting out lines in the core. However if you do then atleast initialize the variable so that the notice you get will go away. 

right ABOVE this 

 $shouldExtend = $this->displayType == NEWSFEED_CMP_Feed::DISPLAY_TYPE_ACTIVITY && $lastActivity !== null;

just add this 

if(!isset($shouldExtend)) { $shouldExtend = ''; }

that should take care of it. :)

Is this possible for comments as well? Right now, if somebody posts a picture, and somebody else comments on it, the picture shows up on the top of the newsfeed but with the commenter's avatar next to it instead of the author's.

I don't understand enough php to find this in the code :-/

Is it weird that no one will fix it? Even devs?

Is it a big problem? Or a small problem? Or not a problem?

..............    :(

Interestingly, I have not had to re-do the modification to avoid this problem after updating to Oxwall 1.8.4

not sure then unless $shouldExtend is an array for which it would be 

$shouldExtend = array();

But it sounds like its null and there is no check for that so when it tries to display it pops the notice because its null. 

@dave
Sorry, I wasn't clear.

I haven't had to re-do the modification to avoid the initial "Poster's avatar replaced by avatar who liked" problem in 1.8.4.

I did have to do it with EVERY other upgrade since I started using Oxwall (1.8.0)

Same problem of misunderstanding for my English, American, Russian, Ukrainian, Belorus, Polish and other language users. When will it be fixed?

Couldn't you fix this confusion in CSS and have it display like this:

How did You do this?

Quote from dave The reason you are getting that notice (not fatal error) is that the variable $shouldExtend is not initialized before using it.  

I do not recommend commenting out lines in the core. However if you do then atleast initialize the variable so that the notice you get will go away. 

right ABOVE this 

 $shouldExtend = $this->displayType == NEWSFEED_CMP_Feed::DISPLAY_TYPE_ACTIVITY && $lastActivity !== null;

just add this 

if(!isset($shouldExtend)) { $shouldExtend = ''; }

that should take care of it. :)

Thanks! Working!

hi itested code some time not corect work have new update .??

It would be really, really nice (and well overdue) if this could be resolved in the next release of Oxwall (seeing there's finally some movement to make Oxwall great again - thanks @Dave !), or at least by a plugin

Change has been commited to gitHub

In North Vancouver, reliable hot water tank services are essential for maintaining a comfortable living environment. Residents depend on these services to ensure their hot water systems operate efficiently year-round. From routine maintenance to repairs and replacements, North Vancouver offers a variety of professional hot water Hot water tank services north vancouver to address any issues promptly. Skilled technicians possess the expertise to diagnose problems accurately and provide effective solutions, ensuring residents have consistent access to hot water whenever needed. With their commitment to quality service and customer satisfaction, hot water tank services in North Vancouver play a crucial role in keeping homes functional and comfortable, no matter the season.