Closed (fixed)
Project:
EU Cookie Compliance (GDPR Compliance)
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
1 Aug 2019 at 15:30 UTC
Updated:
10 Feb 2020 at 08:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
svenryen commentedFor now the solution is to use a database tool and inspect the table.
The plan is to expose the data to views in both D7 and D8. Thanks for the reminder.
Comment #3
junkunczI attached the D8 related patch. It simply exposes its database table and also adds a relationship with the user table by uid.
If you are okay with that I'd be happy to implement the D7 version as well.
Bálint
Comment #4
junkunczSorry it's a new file fix in the patch.
Comment #5
junkunczComment #6
svenryen commentedThanks @junkuncz. Yes I'd be really happy to get a patch for D7 as well.
Comment #7
junkunczAlright I will manage it soon will create a new issue and link it.
Thanks!
Comment #8
junkunczD7 views integration added here: https://www.drupal.org/project/eu_cookie_compliance/issues/3073989
I also upload a new patch (contains tiny refacts).
Comment #10
junkunczComment #11
junkunczHmm I have no idea about this phpcs error, all seems good to me with the last patch:
eu_cookie_compliance.views.inc
line 1 Missing file doc comment
@edit: shit, ignore me I uploaded the wrong patch. :)
retrieve_what_is_stored-3071961-13.patch is the winner!
Comment #12
junkunczComment #13
junkunczComment #15
junkunczComment #16
svenryen commentedLooked mostly good. I changed the name and description of the view, since we're not providing a view of EU Cookie Compliance's but of the Consents.
Here's the final patch. Committing this now. Thanks for the work!
Comment #17
svenryen commentedSeems like I have to commit it later. I'm aboard a plane and the wifi seems to have only port 80 and no port 22 support for git.
Comment #18
junkunczAlright thanks for the feedback and have a good flight! :)
Comment #19
ttamniwdoog commented@svenryen, hope you had a great flight. Could you provide a timetable for the next release of eu_cookie_compliance and if it could contain this patch? Thanks again!
Comment #21
svenryen commentedLooking at this patch again, it needs work.
Some fields are clearly numeric (such as uid) and the timestamp needs to be recognized as such, instead of merely a string.
I also need to make a patch for Drupal 7.
Comment #22
svenryen commentedHere's an updated patch for Drupal 8.
Time and numeric fields are properly recognized.
It's nice if somebody can review.
Comment #23
svenryen commentedI found a better way to identify the uid field, here's an updated patch.
I'll go ahead and merge. If you spot something astray, you can reopen the issue and let me know.
Comment #25
svenryen commentedPatch for D7.
Comment #27
svenryen commented