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

possibly incorrect migration of dam_pages (migraterelations) #73

Closed
dneuge opened this issue Jul 15, 2015 · 1 comment
Closed

possibly incorrect migration of dam_pages (migraterelations) #73

dneuge opened this issue Jul 15, 2015 · 1 comment

Comments

@dneuge
Copy link
Contributor

dneuge commented Jul 15, 2015

While rewriting caption/title/alt handling for images I noticed that migraterelations may query for tt_content.uid using a page ID (tx_dam_mm_ref.uid_foreign when tx_dam_mm_ref.tablenames is pages or pages_language_overlay), then uses the result as captions. This does seem wrong unless I'm missing something. According to history, this was introduced for migration of dam_pages extension providing file ("media") references on page properties and should only be effective for $insertData['fieldname'] === 'media' (mapped field name according to getColForFieldName). As it appears to query the wrong table for metadata, however, it may never have worked.

Merging pull request #74 will limit image metadata processing to tt_content, media metadata is still queried the same way as before but now has a comment with a reference to this issue.

Edit: Column mapping for dam_pages is of course media, not image. Changed fieldname above.

dneuge pushed a commit to glutrot/t3ext-dam_falmigration that referenced this issue Jul 15, 2015
@dneuge
Copy link
Contributor Author

dneuge commented Sep 18, 2015

Closing this issue as the pull request has been merged.

@dneuge dneuge closed this as completed Sep 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant