Add option to hide 'View Contact' link on 'Contact Search' output
Merge request reports
Activity
@mattwire how is the backwards compatibility done? I read through your code and it seems that when I already have a data processor and then install this update the link will disappear. It might be nice if the link is available till otherwise configured.
added 1 commit
- deb671b1 - Add upgrader to add 'link_to_view_contact' parameter default to contact_search
@jaapjansma Yes you are right. I have added an upgrader step because the output is retrieved using API in 3 places and there is no good place I could see to add the default. What do you think now?
Not so sure about the upgrader as one might have data processors in code in an extension and then suddenly your data processors have been overridden.
I have added a function called
alterDataProcessorOutput
in that function we can set a default configuration if none is provided. What do you think? (See: 813dec4f)added 21 commits
-
deb671b1...813dec4f - 20 commits from branch
extensions:master
- 18629074 - Add option to hide 'View Contact' link on 'Contact Search' output
-
deb671b1...813dec4f - 20 commits from branch
@jaapjansma Yes, that is ok with me. I updated the PR to use the function
alterDataProcessorOutput
. Should be ok now?mentioned in commit 4ef1ef8a