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.

Sign up for GitHub

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

Jump to bottom

Extend image metadata handling #163

Merged
merged 70 commits into from
Mar 22, 2024
Merged

Extend image metadata handling #163

merged 70 commits into from
Mar 22, 2024

Conversation

dersmon
Copy link
Member

@dersmon dersmon commented Jun 26, 2023 •
edited
Loading

This does two things:

  • The image resource creation modal got extended in order to set creators based on the project's list of staff members.
  • The application will try to parse data from the image metadata (EXIF etc.) to fill empty metadata (including creator).

@dersmon
Refactor image display variant creation
fdb78a7
@dersmon
Create function for loading image file's metadata
136f1f7 
This is going to be used to set some of Field Desktop's image metadata.
@dersmon
Parse image file metadata
6d0400a 
- creator
- creation date
- copyright statement
- height
- width

This checks exif, iptc and xmp data, in that order.
@dersmon
Rework image category picker for more metadata
794e3f0
@dersmon
Remove log
86cb71f
@dersmon
Add chechmark for selected category
9ef2040
@dersmon
Rename component according to new function
562a3d9
@dersmon
Evaluate image width and height on file level
b94f026 
...instead of trying to parse metadata provided within the file.
@dersmon
Parse exif DateTime as a creation date fallback
a8cb004
@dersmon
Remove deprecated function
e707a29 
Replaced by `selectMetadata`.
@dersmon
Unify image metadata types used for uploads
18ae237
@dersmon
Remove unused import
bc3a0bf
@dersmon
Add some doc strings
45f7311
@dersmon
Remove log
4c8d8bd
@dersmon
Readr creator from iptc metadata
1a4f2cd
@dersmon
Add return type declaration
3134b43
@dersmon
Add image copyright metadata to db document
04158b4 
...as parsed from exif/iptc/xmp.
@dersmon
Refactor function name and doc string
27a81c5
@dersmon dersmon changed the title Loading image file's internal metadata into database fields Extend image metadata handling Aug 9, 2023
@dersmon dersmon requested review from tkleinke and GuzzoD August 9, 2023 07:32
@dersmon dersmon mentioned this pull request Dec 20, 2023
Closed
@tkleinke
Merge branch 'master' into exif_to_db_metadata
054345d
@tkleinke
Read label of draughtsmen field from configuration
f97da87
@tkleinke
Fix constant name in Labels.getFieldLabel
8b3f270
@tkleinke
Read label of staff field from configuration
2b9c964
@tkleinke
Do not allow setting default values for draughtsmen field if it is no…
40e3804 
…t configured
@tkleinke
Add common field date to form Image:default
012c596
@tkleinke
Add metadata only if fields are configured
a05b0f3
@tkleinke
Format creation date
f955a1c
@tkleinke
Only add draughtsmen values to uploaded image that are included in st…
35b421c 
…aff field
@tkleinke
Adjust layout of image upload metadata modal
596470d
@tkleinke
Set parseFileMetadata per field
e393e92
@tkleinke
Fix toggling draughtsman options
cd3ecde
@tkleinke
Serialize images state
f9898c7
@tkleinke
Wait for images state to be loaded in ImageOverviewComponent
278a63f
@tkleinke
Adjust margin of draughtmen options
cbd6daf
@tkleinke
Fix subsystem helper
d23abb8
@tkleinke
Merge branch 'master' into exif_to_db_metadata
2c95071
@tkleinke
Display checkboxes in image upload metadata modal horiziontally
8498802
@tkleinke
Replace radio buttons in image upload metadata modal with switch
0e78ef4
@tkleinke
Always add draughtsmen from image metadata (even if not present in st…
ae6449c 
…aff field)
@tkleinke tkleinke merged commit df0d47f into master Mar 22, 2024
10 checks passed
@dersmon dersmon deleted the exif_to_db_metadata branch April 29, 2024 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tkleinke tkleinke Awaiting requested review from tkleinke

@GuzzoD GuzzoD Awaiting requested review from GuzzoD

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dersmon @tkleinke