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

EZP-29290: As an Editor, I want to have preconfigured "Author" section while creating the content #251

Conversation

konradoboza
Copy link
Member

@konradoboza konradoboza commented Aug 2, 2018

Question Answer
JIRA issue EZP-29290
Bug/Improvement yes
Target version master
BC breaks no
Tests pass yes
Doc needed yes

JIRA excerpt:

Currently, in v2 there is no preconfigured author section (fields filled in with the name and e-mail of the currently logged user). It would be more convenient for the editors to have this fields set automatically while creating a new content. Also, prefilling section with the default data should be configurable at the Author FieldType level.

Depends on:

TODO:

  • Implement feature / fix a bug.
  • Fix new code according to Coding Standards ($ composer fix-cs).
  • Ask for Code Review.

@barbaragr barbaragr self-assigned this Aug 9, 2018
@konradoboza
Copy link
Member Author

I discovered some problems with saving content with an empty author, some rework here needed.

@konradoboza
Copy link
Member Author

I applied changes according to ezsystems/ezpublish-kernel#2417 - default value should be empty to avoid BC break. Changing that also fixed an issue with saving a content with empty Author fields. Please re-review @mikadamczyk @ViniTou @andrerom. :)

Copy link

@barbaragr barbaragr left a comment

Choose a reason for hiding this comment

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

I can't add Content Type:

  1. Go to Admin/Content Types/Content and click (+) button
  2. Fill Name, Identifier and Content name pattern
  3. Click Add field definition - error 500 is thrown: Undefined class constant DEFAULT_VALUE_EMPTY

@konradoboza konradoboza force-pushed the ezp-29290-prefilling_author_fieldtype_option branch from 5133add to cb37608 Compare August 27, 2018 12:28
@lserwatka lserwatka merged commit 2f1493c into ezsystems:master Aug 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
6 participants