Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I am extremely glad to see someone contributing work towards a viable D8 Biblio module. Thanks!
It occurs to me that the entity class Bibliography actually represents a Citation, not a bibliography. This is also referenced in the interface. This is leading to a lot of confusion browsing the codebase.
Am I incorrect?
Understanding that this would be a lot of string pulling and testing to broadly change namespaces I wonder if this change is something you'd be interested in exploring? It seems like something that should be handled sooner than later if you consider it an issue.
Regards,
Comments
Comment #2
JacobSanfordComment #3
JacobSanfordComment #4
kruhak CreditAttribution: kruhak at ADCI Solutions commentedHello Jacob,
You're right. Bibliography is not appropriate name for the main entity. I want to initiate a process of renaming this entity to something more suitable and will be really helpful if you can help me with it. It will be very good to do it before alpha1 release.
I think that a good names is "Citation" and "Reference". Reference is a more common word. What do you think?
Before renaming:
Bibliography - bibliography
Contributor - bibcite_contributor
Keyword - bibcite_keyword
After renaming:
Reference - bibcite_reference or Citation - bibcite_citation
Contributor - bibcite_contributor
Keyword - bibcite_keyword
Comment #5
bibdoc CreditAttribution: bibdoc commentedDear all,
to my knowledge, citation is the formatted expression of a reference. A reference could be from a simple code (a book signature, for instance) , or an hypertext link to a complex and detailed MARC record expressed in ISBD, as long as it allow us de-reference it and find the bibliographic information, thus the document.
Therefore, I find reference more suitable to a bibliographic record than citation, that would be this reference put down on letters and numbers and punctuation, i.e., formatted.
Let me know if I can give a hand and how (user from D6, never contributed). I'm finishing Librarianship degree, FYI.
Regards,
mg
Comment #6
marqpdx CreditAttribution: marqpdx commentedI too believe Reference is the correct choice in this case.
Just a quick glimpse, we'll need to update about 30 files.
Now, i just tied the following, and it actually seems to work. Have not done extensive testing, but my website and the bibcite module both appear to work correctly.
The `grep -irl` command changes every instance of "Bibliography" to "Reference" (case-sensitive) and the `find` command renames every file in the bibcite directory that contains "Bibliography" to "Reference", also case-sensitive.
I am on a Mac, so if on Linux or Windows, the above might be different, but this brings us much closer.
Thanks,
m
ps, this doesn't change the machine_name (bibcite_bibliography etc), that would be an update function perhaps. we still need to look into that.
pps, ok, yes, we'll likely need to replace "bibliography" with "reference" (lower case) using variations of the above, but too sleepy to do today. will try tomorrow.
Comment #7
kruhak CreditAttribution: kruhak at ADCI Solutions commentedThanks for you comments! It's a very helpful.
It will be good if we can find and rename all confusing things in the code.
We have decide that "Bibliography" and "BibliographyType" should be renamed to "Reference" and "ReferenceType".
I want discuss with all about another confusing names in the code:
It will be good if you can comment this potential changes.
Comment #8
marqpdx CreditAttribution: marqpdx commentedhi kruhak,
OK, have been giving this some thought and i have one concern about using the term Reference.
In Drupal there are lots of kinds of reference, like Entity Reference, View Reference, Node Reference etc... We might cause confusion.
perhaps we consider BibReference instead. I don't like it as much, but it will be much more uniquely identifiable in the Drupal ecosystem then Reference would.
Thoughts?
Comment #9
kruhak CreditAttribution: kruhak at ADCI Solutions commentedI think name "Reference" should not cause any confusion.
Machine name will be "bibcite_reference". Class name "Reference" will be protected by namespace of the module. In the interface this name will appear with context.
So, I'm not see any troubles to rename entity to "Reference".
Comment #10
marqpdx CreditAttribution: marqpdx commentedkruhak, you are correct. i was thinking w/ a Drupal 7 mind.
bibcite_reference and the tight Namespace makes it fine. good call!
Comment #11
bibdoc CreditAttribution: bibdoc commentedHi,
regarding to "Styler", citation style (APA, Vancouver, etc ...) is a common naming, so if you want to put it more specific, citation styler could be a good candidate.
Comment #12
kruhak CreditAttribution: kruhak at ADCI Solutions commentedBased on the discussion, these changes will be applied to module components:
All of these changes will cause many troubles for the existing installations of bibcite module. So, all configuration and database changes should be covered by update hook.
Comment #14
kruhak CreditAttribution: kruhak at ADCI Solutions commentedUpdates from the latest comment have been applied. Changes in a database covered by update hook but additional configuration (like a reference fields) can be broken.
Let's close this issue for now. If anyone have a comments about latest changes or ideas about additional improvements then please create a new issue.