Skip to content

[4.0] Namespace System - Privacy Consent plugin#27645

Merged
zero-24 merged 9 commits intojoomla:4.0-devfrom
SharkyKZ:j4/ns/plg-system-privacyconsent
Mar 18, 2020
Merged

[4.0] Namespace System - Privacy Consent plugin#27645
zero-24 merged 9 commits intojoomla:4.0-devfrom
SharkyKZ:j4/ns/plg-system-privacyconsent

Conversation

@SharkyKZ
Copy link
Copy Markdown
Contributor

Summary of Changes

Namespaces System - Privacy Consent plugin.

Testing Instructions

Apply patch. Delete libraries/autoload_psr4.php file.
Enable System - Confirm Consent plugin.
Login to frontend.
When redirected to profile form to accept consent check that Privacy Policy field appears correctly.

Documentation Changes Required

IDK.

@jwaisner
Copy link
Copy Markdown
Member

I have tested this item 🔴 unsuccessfully on ec00dac

Privacy consent at bottom of profile is missing after applying patch and removing libraries/autoload_psr4.php.

I am noticing that when reversing the patch I am getting a warning that plugins/system/privacyconsent/src/Field/PrivacyField.php does not exists. I am not sure if the patch is not applying correctly or something is missing from the patch.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27645.

@SharkyKZ
Copy link
Copy Markdown
Contributor Author

Sounds like the patch wasn't applied correctly. Do you have GitHub Authentication set up in Patchtester? If not, maybe running into request limits.

@jwaisner
Copy link
Copy Markdown
Member

Even using and IDE (Eclipse) to apply from the PR it still has a problem reverting the patch.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27645.

<filename plugin="privacyconsent">privacyconsent.php</filename>
<folder>privacyconsent</folder>
<folder>forms</folder>
<folder>src</folder>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be field?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, contents of field were moved to src/Field.

@jwaisner
Copy link
Copy Markdown
Member

@SharkyKZ Patch is still referencing a non-existant directory. If we need to move data due to a structure change can you please include those instructions in the patch notes? I tried with Patchtester and via my IDE and both did not work properly.

@chmst
Copy link
Copy Markdown
Contributor

chmst commented Mar 17, 2020

I have tested this item ✅ successfully on 6974cae

I tested this successfully had to do applied the patch manually in the IDE.

The structure now is

forms/privacyconsent.php
src/Field/privacy.php
privacyconsent.php
privacyconsent.xml


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27645.

@jwaisner
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on 6974cae

Test is good. Thanks @chmst for the additional instruction.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27645.

@jwaisner
Copy link
Copy Markdown
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27645.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 18, 2020
@zero-24 zero-24 merged commit 2a33620 into joomla:4.0-dev Mar 18, 2020
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 18, 2020
@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Mar 18, 2020

Thanks merged. First merge with the new GitHub App, works awesome :-D

@zero-24 zero-24 added this to the Joomla 4.0 milestone Mar 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants