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

Annotation install FieldException error from reserved word use #5

Closed
amandavisconti opened this issue Dec 8, 2014 · 2 comments
Closed
Assignees
Labels

Comments

@amandavisconti
Copy link
Contributor

In modules/custom/annotation/annotation.install lines 49 and 57, I changed 'name' in
'field_name' => 'name',
to some other identifier. Use of 'name' is causing an error on module install for me on several installation tests, and the error was fixed by replacing 'name' with a non-reserved word. I didn't see any other lines that would require corresponding changes, but 'name' is a hard word to search for.
The error message was:
"FieldException: Attempt to create field name name which is reserved by entity type taxonomy_term. in field_create_field() (line 93 of /mysite.com/modules/field/field.crud.inc).

@mwidner mwidner self-assigned this Dec 8, 2014
@mwidner
Copy link
Member

mwidner commented Dec 8, 2014

Our first issue! How exciting. Thanks for pointing this out, Amanda. What did you change it to? I'm thinking "label" probably makes sense. If you'd like to send a pull request so that we properly credit your contribution, please feel free. Thanks!

@amandavisconti
Copy link
Contributor Author

Excellent--just sent the request (I think label works well).

On Sun, Dec 7, 2014 at 11:15 PM, Michael Widner notifications@github.com
wrote:

Our first issue! How exciting. Thanks for pointing this out, Amanda. What
did you change it to? I'm thinking "label" probably makes sense. If you'd
like to send a pull request so that we properly credit your contribution,
please feel free. Thanks!


Reply to this email directly or view it on GitHub
#5 (comment)
.

@mwidner mwidner closed this as completed Dec 8, 2014
tzhu10 pushed a commit that referenced this issue Mar 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants