Refactor the metadata handling
Bug #962301 reported by
Sebastian Meyer
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Goobi.Presentation |
Fix Released
|
Low
|
Sebastian Meyer |
Bug Description
Currently each configured metadate is handled seperatly. The workflow is as follows:
1. Get configuration from database.
2. Parse metadate value from METS file.
3. Apply TS configuration to it.
4. Process next metadate.
The problem is that sometimes you may want to combine several metadata fields, but the metadata fields don't "know" of each other. So a better workflow would be:
1. Get configuration from database.
2. Parse information for all metadata fields and save it.
3. Apply TS configuration to each field.
Related branches
lp:~sebastian-meyer/goobi-presentation/bug962301
- Sebastian Meyer: Approve
- Diff: 0 lines
lp:goobi-presentation/1.1
- Sebastian Meyer: Approve
-
Diff: 6190 lines (+1826/-1558) (has conflicts)35 files modifieddlf/common/class.tx_dlf_document.php (+427/-367)
dlf/common/class.tx_dlf_helper.php (+189/-137)
dlf/common/class.tx_dlf_indexing.php (+138/-157)
dlf/common/class.tx_dlf_list.php (+51/-15)
dlf/common/class.tx_dlf_mods.php (+0/-12)
dlf/common/class.tx_dlf_module.php (+12/-14)
dlf/common/class.tx_dlf_plugin.php (+22/-10)
dlf/common/class.tx_dlf_solr.php (+14/-3)
dlf/ext_autoload.php (+30/-30)
dlf/ext_emconf.php (+33/-0)
dlf/ext_tables.php (+146/-145)
dlf/ext_tables.sql (+2/-1)
dlf/ext_tables_static+adt.sql (+1/-2)
dlf/hooks/class.tx_dlf_em.php (+166/-166)
dlf/hooks/class.tx_dlf_hacks.php (+21/-21)
dlf/hooks/class.tx_dlf_tceforms.php (+24/-24)
dlf/hooks/class.tx_dlf_tcemain.php (+50/-58)
dlf/locallang.xml (+0/-2)
dlf/modules/indexing/index.php (+2/-2)
dlf/modules/newclient/index.php (+1/-0)
dlf/modules/newclient/metadata.inc.php (+18/-3)
dlf/plugins/collection/class.tx_dlf_collection.php (+19/-15)
dlf/plugins/feeds/class.tx_dlf_feeds.php (+6/-6)
dlf/plugins/listview/class.tx_dlf_listview.php (+100/-93)
dlf/plugins/listview/locallang.xml (+2/-0)
dlf/plugins/metadata/class.tx_dlf_metadata.php (+91/-87)
dlf/plugins/navigation/class.tx_dlf_navigation.php (+2/-2)
dlf/plugins/oai/class.tx_dlf_oai.php (+124/-91)
dlf/plugins/pageview/class.tx_dlf_pageview.php (+33/-26)
dlf/plugins/search/class.tx_dlf_search.php (+39/-30)
dlf/plugins/search/locallang.xml (+2/-2)
dlf/plugins/statistics/class.tx_dlf_statistics.php (+5/-1)
dlf/plugins/toc/class.tx_dlf_toc.php (+23/-19)
dlf/plugins/toolbox/class.tx_dlf_toolbox.php (+8/-8)
dlf/tca.php (+25/-9)
Changed in goobi-presentation: | |
status: | New → In Progress |
importance: | Undecided → Low |
assignee: | nobody → Sebastian Meyer (sebastian-meyer) |
milestone: | none → 1.1.rc1 |
Changed in goobi-presentation: | |
status: | In Progress → Fix Committed |
Changed in goobi-presentation: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.