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

CRM-21225 Allow for display profile alternate - schema changes only #11503

Merged
merged 1 commit into from
Jan 12, 2018

Conversation

seamuslee001
Copy link
Contributor

@seamuslee001 seamuslee001 commented Jan 10, 2018

Overview

this picks up just the schema changes from https://github.com/civicrm/civicrm-core/pull/11370/files this is in the hope to try to get this part merged through

Before

No diplay_title column on civicrm_uf_group

After

display_title added

ping @eileenmcnaughton @agileware @JoeMurray


@eileenmcnaughton
Copy link
Contributor

This seems good to me - do we have a precedent anywhere for naming conventions for front end title fields vs back office title fields?

@seamuslee001
Copy link
Contributor Author

@eileenmcnaughton @totten @jusfreeman I have now renamed the column frontend_title as per discussion on MM. I'll work on the main code once the schema change has been merged

@seamuslee001
Copy link
Contributor Author

Jenkins re test this please

@eileenmcnaughton
Copy link
Contributor

eileenmcnaughton commented Jan 12, 2018

We discussed this in chat. The situation is that there is a need for 3 'names'

a symbolic, unchanging machine name
a backend/administrative title (used to browse/select among profiles)
a frontend/page title (like the '<h1'> on a public page)

We have a standard of name for the first & title kinda covers 2 & 3. After discussion we preferred adding 'frontend_title' to display_title (display when?)

Happy with this change now - merge on pass - it could be extended to other entities like custom data group etc

@eileenmcnaughton eileenmcnaughton merged commit 786109a into civicrm:master Jan 12, 2018
@eileenmcnaughton eileenmcnaughton deleted the CRM-21225-schema branch January 12, 2018 02:23
@mlutfy mlutfy added this to the 4.7.31 milestone Feb 9, 2018
@gah242s
Copy link
Contributor

gah242s commented Apr 23, 2018

This is breaking multilingual + logging upgrades. The select statement generates FROM civicrm_uf_group_en_US, which I believe should be civicrm_uf_group. Here's where the field name shows up in the error: [nativecode=1054 ** Unknown column 'a.frontend_title' in 'field list']. Going with a very uneducated guess, I'd think that the error may lie in line 493 of CRM/Upgrade/Incremental/php/FourSeven.php. Does there need to be another FALSE argument at the end?
(Edited to replace installs with upgrades.)

@seamuslee001
Copy link
Contributor Author

@gah242s are you able to put the backtrace into a gist or similar? there could well be something going strange it could also be that the multilingual views haven't properly re-generated

@gah242s
Copy link
Contributor

gah242s commented Apr 25, 2018

https://gist.github.com/gah242s/2fbebbb638b8a2065de9eeca7e2a989f
Here's a gist, also posted in MM. Hopefully it's useful?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants