Opened 4 years ago
Closed 4 years ago
#8391 closed defect (bug) (fixed)
BP Nouveau: improve Groups Loop and single Groups Headers (with & without cover) templates
Reported by: | imath | Owned by: | imath |
---|---|---|---|
Milestone: | 7.0.0 | Priority: | normal |
Severity: | normal | Version: | 3.0.0 |
Component: | Templates | Keywords: | has-patch |
Cc: |
Description
I was testing the improvements we brought to the BP REST API when I've noticed the Group Types were not included into the regular Groups single item header.
Exploring the issue, I've noticed there was a weird function in Nouveau used to fill the template container having the meta
class. I believe I've improved this function, following the @todo
that was inside the code. I had to create a new class BP_Nouveau_Group_Meta
as this function was used like an object, eg: <?php echo esc_html( bp_nouveau_group_meta()->status ); ?>
to safely deprecate it.
I've fixed the issue about Group Types display in Group headers.
Then, I've noticed another bug about the live update of dates (via moment.js). The "Active" word was disappearing in Group Headers and wasn't refreshing in Loops.
The patch adresses all this + some improvements on code indentation.
In 12789: