Closed (fixed)
Project:
CRM Core
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
21 Mar 2016 at 12:20 UTC
Updated:
19 Apr 2016 at 10:54 UTC
Jump to comment: Most recent
Activity entity currently doesn't declare views data handler.
Implement views data handler for activity entity. Rely on EntityViewsData for sane defaults and build on top of that. See what data and which views handlers activity used in D7 (files in modules/crm_core_activity/views). Goal of this issue is to achieve feature-parity with D7 version.
Patch will be complete when:
- we have feature parity with D7 version
- D7 code is either ported or removed
- we have test coverage
Comments
Comment #2
slashrsm commentedComment #3
CTaPByK commentedThis issue is similar to what i'm working right now, to provide views support for contact, so i will take this one.
Comment #4
CTaPByK commentedHere is new patch with some cleanup and new views support for activity.
Comment #5
sanja_m commentedIt seems ok.
Comment #6
slashrsm commentedThis 3 fields don't make sense any more as we removed their handlers (since core provides same functionality in a different way).
This field is also provided by core.
We won't be using core search.
And types are not in SQL any more so we can't join.
Comments above make ActivityViewsData unneeded so we use EntityViewsData here instead.
At the end of this comment block there is a @todo which can be removed with this patch.
This module should depend on crm_core_activity too.
Comment #7
CTaPByK commentedHere is new patch with fixes requested by latest review.
Comment #8
sanja_m commentedLooks good to me.
Comment #9
slashrsm commentedCommitted. Thanks!
Comment #12
mbovan commentedI guess this should be fixed.
Comment #13
CTaPByK commentedYes this is fixed but somehow after commit it changed status to active and assigned to unassigned.