Hi.

First: cool module!

I was wondering: PCP engenders one block per profile-type that is marked "required by PCP". Would it be possible to to combine all those blocks in one block so that on the user's profile, it would be more compact. The «Filling out...» links could be in a list, like this new module (http://drupal.org/project/profile_alert) that does not seem to offer a link per field to complete.

I found the "field to complete" link much usefull for users who are not that much at ease with UI.

Thanks.

Comments

deggertsen’s picture

deggertsen commented

Yes, I would also like to see all the Profile 2 profile types combined in one PCP block. Not sure if this will be too difficult, but I will look into it.

deggertsen’s picture

deggertsen commented

Wow. Looking through the code it seems there was a lot of effort put forward in order to integrate Profile2 and have separate blocks for profile percent complete. Maybe I'm wrong, but it seems like lines upon lines of code were written for this single purpose, while it may have actually been easier to create it as all one block. That said, I can't find an easy way to simply create a new block that combines all of the marked fields into one block without rewriting all of the functions to remove the $entity_type and $bundle variables. On the other hand those variables may be necessary for calling the correct url for where to edit the next field that needs completion.

I'm still a bit of a PHP noob, so if the maintainer could shed any light on this it would be greatly appreciated.

BarisW’s picture

BarisW commented
Status: Active » Postponed (maintainer needs more info)

Do you mean combining both PCP core and Profile2? Because that would be very hard: #1864190: Merge Drupal core profile with profile2 .
Combining all Profile2 types into one might be possible.

However, I'm just maintaining the module, I don't have time to add new stuff. Feel free to submit a patch, I'm happy to review and commit it.

Québec’s picture

Québec commented

Yes, that was the idea: combining all profile_2 types in one block, instead of having one block for each profil types. The % could be calculated on all of the fields of all profile types.

Cool if you dont have time for new stuff; I can understand... time is everything.

deggertsen’s picture

deggertsen commented

Right now I just have a separate profile complete block for each profile type and PCP core. We would prefer to have them all combined, but seeing as it would pretty much require a module rewrite to do it so we've just decided to deal with it.

abramm’s picture

abramm
Primary language Ukrainian
Location Lutsk
commented
Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (outdated)

D7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.

Everyone can apply the patch above (not tested by the maintainers, tested by the community) to their D7 projects.
If the issue remains relevant for D10+ versions, merge requests with proposed solutions for a new module version (D10+) are welcome in a new follow-up issue.

Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.