The socialfeed_PROVIDER_posts( ) functions now return an associative array, delegating the creation of HTML to socialfeed_posts_render( ) function.

With this way you can use socialfeed_PROVIDER_posts() to take an associative array to be used in other modules or blocks.

I hope that you feel useful this change.

Bye
I.

CommentFileSizeAuthor
#8 return_associative_array-2756339-8.patch2.78 KBhilatti
#6 return_associative_array-2756339-6.patch2.54 KBacrosman
#5 return_associative_array-2756339-4.patch2.23 KBacrosman
#2 return_associative_array-2756339-2.patch3.25 KBYoris00

Comments

Yoris00 created an issue. See original summary.

Yoris00’s picture

Yoris00 commented
StatusFileSize
new return_associative_array-2756339-2.patch3.25 KB
acrosman’s picture

acrosman
Primary language English
Location South Carolina, US
commented
Title: Get associative array un other module » Get associative array in other module
Status: Needs review » Needs work
acrosman’s picture

acrosman
Primary language English
Location South Carolina, US
commented

I think this patch is a great idea, but I'm struggling to get it to apply. Looks like it's in need of a re-roll at this point.

acrosman’s picture

acrosman
Primary language English
Location South Carolina, US
commented
Status: Needs work » Needs review
StatusFileSize
new return_associative_array-2756339-4.patch2.23 KB

Attached is a reroll of the patch.

acrosman’s picture

acrosman
Primary language English
Location South Carolina, US
commented
StatusFileSize
new return_associative_array-2756339-6.patch2.54 KB

I found a couple small issues during testing of the previous patch. Here's a new version.

acrosman’s picture

acrosman
Primary language English
Location South Carolina, US
commented
Assigned: Yoris00 » Unassigned
hilatti’s picture

hilatti commented
StatusFileSize
new return_associative_array-2756339-8.patch2.78 KB

Re-rolled patch to work with latest version.

hemangi.gokhale’s picture

hemangi.gokhale
she/her/hers
Primary language English
Location Portland, 🇺🇸
commented
Status: Needs review » Reviewed & tested by the community

  • Hemangi Gokhale committed 5423056 on 7.x-2.x 
    git commit -m 'Issue #2756339 by acrosman, Yoris00, hilatti, Hemangi...
hemangi.gokhale’s picture

hemangi.gokhale
she/her/hers
Primary language English
Location Portland, 🇺🇸
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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