Skip to content
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

Privacy consent statements error in registration form multi-journal install #3836

Closed
israelcefrin opened this issue Jun 27, 2018 · 13 comments
Closed
Assignees
Labels
Hosting Bug reports and feature requests from Publishing Services's hosted clients.

Comments

@israelcefrin
Copy link
Collaborator

israelcefrin commented Jun 27, 2018

Multi-journal registering form page doesn't allow to proceed with registration since it doesn't display checkbox with privacy consent. That way it always returns a form validation error message.

See how it shows to Portal registering page:
https://demo.publicknowledgeproject.org/pkpschool/ojs3/index.php/index/user/register

And how it is shown in a journal from this Portal:
https://demo.publicknowledgeproject.org/pkpschool/ojs3/index.php/rlr/user/register

capture

@israelcefrin israelcefrin added the Hosting Bug reports and feature requests from Publishing Services's hosted clients. label Jun 27, 2018
@asmecher asmecher added this to the OJS/OMP 3.1.1-3 milestone Jun 28, 2018
Vitaliy-1 added a commit to Vitaliy-1/pkp-lib that referenced this issue Jun 30, 2018
@NateWr
Copy link
Contributor

NateWr commented Jul 5, 2018

From my memory of previous discussions, we may need:

  1. A separate consent statement field in the site settings for site-wide registration.
  2. When registering in a multi-journal site, we may need to present the consent statement for each journal a user is registering in, since they may be different.
  3. Some multi-journal sites may be able to get by on a single consent statement. Others may need a separate consent statement for each journal. This will usually depend on whether the "host" is a prominent part of a reader's experience with a journal (ie - an institutional publisher) or is in the background (ie - a technical web host or publishing service). For this reason, it might make sense for there to be a config.inc.php option determining whether to enable consent statements on a per-journal basis. When disabled, the site-wide consent statement would be the only thing used in the site or context registrations, and no text field would appear in journal admin areas.

@asmecher
Copy link
Member

@NateWr, would you mind taking a crack at this? There's some discussion over at #3870.

NateWr added a commit to NateWr/orcidProfile that referenced this issue Aug 1, 2018
Fixes a PHP warning that a reference was expected but a value was
passed.
NateWr added a commit to NateWr/pkp-lib that referenced this issue Aug 2, 2018
- Add site-wide privacy statement.
- Add config option to disable context-specific privacy statements.
- Add privacy consent for site and contexts to site-wide registration.
- Display site-wide privacy statement if no context statement exists.
NateWr added a commit to NateWr/ojs that referenced this issue Aug 2, 2018
@NateWr
Copy link
Contributor

NateWr commented Aug 2, 2018

I've implemented the following:

  • Added a site-wide privacy statement.
  • Added a config option to disable context-specific privacy statements (for sites that only need a single, site-wide statement).
  • Added privacy consent checkboxes for site-wide registration. Consent will be requested for a site-wide policy if one exists, and will be requested for any contexts to which a user selects a role and a policy exists.
  • If a context has no privacy statement, but the site does, the context's privacy page will display the site-wide privacy statement.

This has implications for existing themes. @Vitaliy-1 would you take a look and update the existing themes as necessary? If you could also provide a PR for the Bootstrap3 theme (https://github.com/NateWr/bootstrap3/issues/93), I'd really appreciate it. Finally, if you're able, it would be good to document this change for third-party themes, and outline steps they need to take to make their themes compatible.

@bozana would you code review?

PRs:
#3964
pkp/orcidProfile#41
pkp/ojs#2086
(No changes required for OMP, since there is no site/context distinction.)

@NateWr
Copy link
Contributor

NateWr commented Aug 14, 2018

@asmecher can you code review?

asmecher added a commit to pkp/orcidProfile that referenced this issue Aug 15, 2018
pkp/pkp-lib#3836 Fix broken HTML output on registration page.
@asmecher
Copy link
Member

@NateWr, did you see that the tests failed? OTOH, I suspect it's because you're potentially getting the Site object before the system has been installed (thus no DB).

NateWr added a commit to NateWr/pkp-lib that referenced this issue Aug 16, 2018
- Add site-wide privacy statement.
- Add config option to disable context-specific privacy statements.
- Add privacy consent for site and contexts to site-wide registration.
- Display site-wide privacy statement if no context statement exists.
NateWr added a commit to NateWr/pkp-lib that referenced this issue Aug 16, 2018
NateWr added a commit to NateWr/ojs that referenced this issue Aug 16, 2018
@NateWr
Copy link
Contributor

NateWr commented Aug 16, 2018

Thanks! Think I've fixed it. Tests running.

NateWr added a commit to NateWr/pkp-lib that referenced this issue Aug 28, 2018
- Add site-wide privacy statement.
- Add config option to disable context-specific privacy statements.
- Add privacy consent for site and contexts to site-wide registration.
- Display site-wide privacy statement if no context statement exists.
NateWr added a commit to NateWr/pkp-lib that referenced this issue Aug 28, 2018
NateWr added a commit to NateWr/ojs that referenced this issue Aug 28, 2018
@NateWr
Copy link
Contributor

NateWr commented Aug 28, 2018

@asmecher tests have passed here and I've rebased for a clean merge.

@asmecher
Copy link
Member

Reviewed, committed and back-ported to ojs-stable-3_1_1, thanks! I hope we don't have many boilerplate fields with this much of their own baggage, but it makes sense in this case.

asmecher added a commit to pkp/ojs that referenced this issue Aug 28, 2018
pkp/pkp-lib#3836 Add site privacy statement and consent during registration
asmecher added a commit that referenced this issue Aug 28, 2018
#3836 Add site privacy statement and consent during regist…
asmecher pushed a commit that referenced this issue Aug 28, 2018
- Add site-wide privacy statement.
- Add config option to disable context-specific privacy statements.
- Add privacy consent for site and contexts to site-wide registration.
- Display site-wide privacy statement if no context statement exists.
asmecher pushed a commit to pkp/ojs that referenced this issue Aug 28, 2018
ambs pushed a commit to ambs/pkp-lib that referenced this issue Oct 16, 2018
- Add site-wide privacy statement.
- Add config option to disable context-specific privacy statements.
- Add privacy consent for site and contexts to site-wide registration.
- Display site-wide privacy statement if no context statement exists.
@fgnievinski
Copy link
Contributor

the milestone was 3.1.1-4 but it seems that OJS version 3.1.2.0 still gives out this error (user.profile.form.privacyConsentRequired):

image

maybe there's a regression?

@asmecher
Copy link
Member

@fgnievinski, some themes also required adjustment when they overrode affected templates. (See e.g.: NateWr/bootstrap3@0df65c2) What theme are you using?

@fgnievinski
Copy link
Contributor

hi @asmecher , I see the Default theme is selected under Website Settings > Appearance.

There's also a custom journal style sheet that reads simply:

.item.references a {
word-break: break-all;
}

http://www.seer.ufu.br/public/journals/78/styleSheet.css?2019-09-13%2021:18:44

@fgnievinski
Copy link
Contributor

fgnievinski commented Aug 16, 2020

pls let me know if you have any pointers.

I looked in a couple of .less and .tpl files.

thanks

@NateWr
Copy link
Contributor

NateWr commented Aug 17, 2020

Look in lib/pkp/templates/frontend/pages/userRegister.tpl. You should see the privacyConstent field there. You may be able to debug by adding some text in there to see if it is showing up (clear template cache under the admin page in the backend whenever you make a change.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hosting Bug reports and feature requests from Publishing Services's hosted clients.
Projects
None yet
Development

No branches or pull requests

4 participants