feature request #6658
closedMake mediaCreated editable in Media view
100%
Description
Media has beside CdmBase.created another data called Media.mediaCreated.
This needs to be editable but currently isn't in the media view.
Related issues
Updated by Andreas Müller almost 7 years ago
- Related to feature request #6659: Show mediaCreated in dataportal added
Updated by Andreas Müller almost 7 years ago
- Related to feature request #6607: Implement WebSite.accessed in TaxEditor added
Updated by Andreas Müller almost 7 years ago
- Assignee changed from Patrick Plitzner to Katja Luther
- Priority changed from New to Highest
- Target version changed from Unassigned CDM tickets to Release 4.8
Updated by Katja Luther almost 7 years ago
- Status changed from New to Resolved
- % Done changed from 0 to 50
Applied in changeset taxeditor|6430b43e4a5e3def3a2778bf51f5114606cda156.
Updated by Andreas Müller almost 7 years ago
- Assignee changed from Katja Luther to Andreas Müller
Updated by Andreas Müller almost 7 years ago
- Assignee changed from Andreas Müller to Katja Luther
Sorry maybe it was too early to pass to assign to me. With current version the media is not yet stored in DB and even not in session as far as I can see.
Updated by Katja Luther almost 7 years ago
- Assignee changed from Katja Luther to Andreas Müller
the problem was that the start value of the mediaCreated field was not set correctly. It was saved but not displayed.
Now this is fixed, please review.
Updated by Andreas Müller almost 7 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
Generally it works, but there is no check for correct format.
However, there are a couple of problems related to the pure text input:
- No user visible format check exists: Compare e.g. with URI for format checking (red text if format is not correct). E.g. the user is not informed about entering just "xxx" has no effect on the data.
- It is not possible to enter time. Sometimes time might be an interesting information which gets completely lost. Also the user is not informed about the fact that it gets lost.
- If entering only a year this is transformed to the year on Jan. 1st. But the user is not informed about this changed.
- Entering e.g. 2008-02-30 may result in 2008-02-03 without the user being informed. This is because only the intermediate state is changed.
- Generally it is often more comfortable using a calendar entry component. Could you please leave a message in any of the DateTime tickets what the problem with the Calendar element e.g. from nebula is. Also leave a message where else the current solution is used so it can be changed in bulk once we have a better solution.
If using a Calendar component is to much work for now, we may stay with the text element but we should as a minimum implement a format check. If we allow only yyyy-mm-dd the data should be changed ONLY if it comes EXACTLY in this format. And the user should be informed in RED that the data is not stored or completely deleted if this red label appears.
By the way the label says "yyyy-MM-dd". Can we change this to "yyyy-mm-dd". Even if maybe not consistent with DateTime format rules it is easier to understand by ordinary users.
Updated by Katja Luther almost 7 years ago
- Assignee changed from Katja Luther to Andreas Müller
now there is a calendar widget to choose the date and the time.
The DateElement is used for DNAQualityCheck (qualityCheckDate), Reference (accessed) and MediaSpecimen(preparationDate).
The DateElement provides the possibility to edit also the textfield, but as long as we do not have a proper handling for wrong formated strings, this is disabled.
Updated by Andreas Müller almost 7 years ago
- Status changed from Feedback to Resolved
Updated by Andreas Müller almost 7 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
It looks much better now.
Some smaller issues:
most important: if there is already data available the dialog should be set to this data. Currently it always shows the current date time (which is annoying if you have to move back e.g. 10 years only because you want to change the time)is it possible to add a title to the dialog "such as "Media created" (depending on the use case).the seconds can not be set but still appear in the result. So either we should allow editing seconds or we should remove them from the result and always set to 00currently it is not possible to edit data in the text field therefore we can remove the showing of the format "yyyy-mm-dd hh:mm:ss", however, once we allow editing via text fieldcan you open a new ticket for remaining issue to edit via text field?- for later when we show format again, hours should be hh, not HH
The textfield should start at the same position as "Title", "Artist" and "Media URI". Currently there is a little identation
Updated by Katja Luther almost 7 years ago
- Related to feature request #6741: Edit Date in TextField added
Updated by Katja Luther almost 7 years ago
- Status changed from Feedback to Resolved
- Assignee changed from Katja Luther to Andreas Müller
created new ticket for editing in textfield and set this ticket to resolved.
Updated by Andreas Müller over 6 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
if there is already data available the dialog should be set to this data. Currently it always shows the current date time (which is annoying if you have to move back e.g. 10 years only because you want to change the time)
This is fixed if data is persisted, if data is not yet persisted or persisted with a different date (because you recently changed it) than the persisted or the current date is taken. But it should always open with the currently shown date.
Updated by Andreas Müller over 6 years ago
The textfield should start at the same position as "Title", "Artist" and "Media URI". Currently there is a little indentation.
This is still an open issue. The label is correct, but the text field has an indentation
Updated by Katja Luther over 6 years ago
- Target version changed from Release 4.8 to Release 4.10
needs to be fixed completely in 4.10
Updated by Andreas Müller over 6 years ago
- Severity changed from normal to critical
There is another critical issue. If you select e.g. July than June is written into the textfield. Looks like incorrect counting start is taken. Can you fix this immediately (should not take much time I guess).
Updated by Katja Luther over 6 years ago
Andreas Müller wrote:
There is another critical issue. If you select e.g. July than June is written into the textfield. Looks like incorrect counting start is taken. Can you fix this immediately (should not take much time I guess).
This is strange because the toString-method of joda time is used to set the text:
this.textDate.setText(initialDateTime.toString(pattern));
I will have a look at this.
Updated by Katja Luther over 6 years ago
Katja Luther wrote:
Andreas Müller wrote:
There is another critical issue. If you select e.g. July than June is written into the textfield. Looks like incorrect counting start is taken. Can you fix this immediately (should not take much time I guess).
This is strange because the toString-method of joda time is used to set the text:
this.textDate.setText(initialDateTime.toString(pattern));
I will have a look at this.
swt.DateTime starts with 0 as the first month, so adding 1 to fit with joda time.
Updated by Andreas Müller over 6 years ago
you also need to substract 1 when reading joda.DateTime to swt.DateTime. Otherwise you increase existing data by 1 month each time you open the editor and store the data. Implemented with taxeditor|754477ecee0a6cc9e77686b533ceee623702ee30 and taxeditor|b18418d0b56a957b3c0490fc72dc7957a6952f53
Updated by Andreas Müller over 6 years ago
- Target version changed from Release 4.10 to Release 4.8
I don't understand why you moved this to 4.10 (98% seems to be implemented in 4.8 therefore we should keep it there for documentation). "Edit Date in TextField" has been moved to a new ticket #6741 and some major bug (has been fixed in 4.9.0 and 4.9.1).
Only remaining issue to me seems: "This is still an open issue. The label is correct, but the text field has an indentation". But please read the full ticket carefully if there is maybe still something else open. I didn't check intensively.
And another new open issue is: #6558
Updated by Andreas Müller over 6 years ago
- Related to bug #6860: Use consistent calendar system for date picker added
Updated by Katja Luther over 6 years ago
- Status changed from Feedback to Resolved
Applied in changeset taxeditor|e38ddfeee4a59c876fedd950b64cb99147e4f865.
Updated by Katja Luther over 6 years ago
- Assignee changed from Katja Luther to Andreas Müller
removed the indention. So we can close the ticket?
Updated by Andreas Müller over 6 years ago
- Status changed from Resolved to Closed
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 50 to 100
Updated by Andreas Müller over 6 years ago
- Related to bug #7074: Use TimePeriod for Media.mediaCreated added