I just ran composer update, followed by drush updb.

Now the teasers in the streams, both on homepage and on groups, fail to properly display the author name and picture, displaying in exchange the full profile.

See the 2 pictures below for better understanding.

A quick look at the block_stream_homepage View shows that the View is properly generated.
(admin/structure/views/view/activity_stream/edit/block_stream_homepage)

So I'm guessing this is either a theming or a preprocessing issue.

I'll be reverting to a previous backup.

Home Before Upgrade:
Home before upgrade

Home After Upgrade
Home after upgrade

The View looks fine in the Views page:
Activity Stream View

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Carlos Miranda Levy created an issue. See original summary.

Carlos Miranda Levy’s picture

Title: Broken display of teasers in Streams » Broken display of teasers in Streams (home pages) after upgrading to 8.x-1.0
jochemvn’s picture

Carlos,
Running a composer update and a drush updatedb is not enough to update Open Social. Please read this this article about installing and updating Open Social: https://github.com/goalgorilla/drupal_social/wiki/Installing-and-updatin...

Carlos Miranda Levy’s picture

Closer inspection, indicates that it is the profile image not being displayed (the social_medium style), however, the image is physically accessible from the url used by the img tag, so it's not a permissions issue:

/sites/civ/files/styles/social_medium/public/2017-06/Carlos_Miranda_Levy-in-Ishinomaki-Manga_Statues%20%287%29.jpg?itok=rmONgP78

Carlos Miranda Levy’s picture

Gotcha, I ran drush -y fra --bundle=social and everything seems normal now.

I suggest to make a quick correction to the upgrading instructions.

One, for us well-behaved users, who started with composer and didn't download from drupal, etc.

And list the steps for updating.

As I understand now, in my case (Linux box, installing using composer), the steps are, in this order:

composer update
drush cr
drush updb
drush entup
drush -y fra --bundle=social

... and just in case...

drush cr
drush cron

:-)

Anything else?

Thanks for the quick response above.

jaapjan’s picture

Status: Active » Fixed

Hi Carlos, great that it works now!

I don't think this last part is needed:

drush cr
drush cron

Especially if you do the update steps right after each other it should not be necessary to do an extra cache clear (there is already a cache clear in drush updb).

Perhaps we should think about splitting the update instructions for users who installed with Composer and installed from tarball on drupal.org to make the distinction a bit clearer.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.