Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix verbiage for Edit Contact Information title #13352

Merged
merged 1 commit into from
Dec 26, 2018

Conversation

yashodha
Copy link
Contributor

@yashodha yashodha commented Dec 26, 2018

Overview

fix verbiage for Edit Contact Information title

Before

Edit Contact Information title is wrongly shown as Edit Relationship

After

title reads Edit Contact Information

Comments

Title is misleading and does not match with the name.

@civibot
Copy link

civibot bot commented Dec 26, 2018

(Standard links)

@civibot civibot bot added the master label Dec 26, 2018
@yashodha
Copy link
Contributor Author

@jitendrapurohit found another one.

@jitendrapurohit
Copy link
Contributor

On my testing - I wasn't able to see the link Edit Contact Information loading on the dashboard. Eg https://dmaster.demo.civicrm.org/civicrm/user?reset=1 displays the link for Dashboard and disable action, but no link to edit the contact details.

On digging into it a little, I see the reason for the link not loading on the page is because the file UserDashboard.php uses array_merge() and adds a disable action link which resets the keys in self::$_links variable.

If I change the line to eg self::$_links = self::$_links + array(....) instead of the existing array_merge(), the link is shown and the this PR correctly modifies the title to Edit Contact Information.

@yashodha If the above make sense, can you modify and fix the link display in this PR itself?

@yashodha
Copy link
Contributor Author

@jitendrapurohit
makes sense :) I would rather have that fix in a different PR so that we have one PR tackle one problem.
#13353 is filed for the same.

@seamuslee001
Copy link
Contributor

I'm going to merge this PR based on @jitendrapurohit 's comment of "the this PR correctly modifies the title to Edit Contact Information." Agree with Yash lets split out the other issue into its own PR

@seamuslee001 seamuslee001 merged commit e97a770 into civicrm:master Dec 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants