-
-
Notifications
You must be signed in to change notification settings - Fork 825
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
CRM-21225 Allow for display profile alternate - schema changes only #11503
Conversation
ac07e46
to
913a526
Compare
This seems good to me - do we have a precedent anywhere for naming conventions for front end title fields vs back office title fields? |
913a526
to
9205285
Compare
@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 |
Rename column to be frontend_title
9205285
to
bf48be4
Compare
Jenkins re test this please |
We discussed this in chat. The situation is that there is a need for 3 'names' a symbolic, unchanging machine name 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 |
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? |
@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 |
https://gist.github.com/gah242s/2fbebbb638b8a2065de9eeca7e2a989f |
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