Fixed not-well-formed bcmath issue in av_metadata.html (ticket #24)
Added static method ArchiveStructure::force_decimal_point_char() for this.
https://www.php.net/manual/en/book.bc.php Reason was that bcmul() as part of BCMath "always expects a decimal point" (not comma, as it is using DE locale). Fixed by forcing (search+replace) "." instead of "," in numeric strings before using in bcmatch functions.
"bcmath function argument is not well-formed" in av_metadata.html
Warning "operation not permitted" when importing capture requests
Fixed in r563: Declaring "$rename=false" when calling move_files() for capture requests.
Fixed in rev. 566. NOTE: Before this revision, the case was kept as-is.
Capture requests with UPPERCASE filenames cause issues
Fixed in rev. 566.
Removed DEBUG leftover.
Forcing filename of imported capture-requests to lowercase (fixes #23)
Capture requests with UPPERCASE filenames cause issues
Disable "rename" to force move for capture request import (fixes #22)
Warning "operation not permitted" when importing capture requests
Notice: Trying to access array offset on value of type null in MthkVideoCarrier.php on line 861
fixed in rev.562
Undefined variable: filename in MthkTask_request_video_capture_json.php on line 153
Fixed in rev.563: Error messages and handling in general has been improved (using RuntimeExceptions).
Fixed and improved error handling and reporting with JSON requests (ticket #21)
Indeed, there was $contents, not $filename - but the issue went way deeper: Errors detected during parsing of the JSON request were not properly propagated as error message - and for another reason, failure of load_capture_request() did /NOT/ trigger the task to go to error.
Undefined variable: filename in MthkTask_request_video_capture_json.php on line 153
Fixed missing check for empty stock_brand array values (ticket #20)
Notice: Trying to access array offset on value of type null in MthkVideoCarrier.php on line 861
purge/remove: not deleting workflow-file anymore
Fixed in revision 561. The reason was that "$delete_targets" list in MthkTask::purge_data() method resolved the "CAPTURE_REQUEST" filename patterns for the workflow files: $this->get_filename('CAPTURE_REQUEST', 'CAPTURE_REQUEST', 'TASK/TO_DO'), this worked in the past, because capture_request and workflow_files were /both/ '.v_ingest'. The correct code is now: $this->get_filename('WORKFLOW_FILE', 'WORKFLOW_FILE', 'TASK/TO_DO'),
Fixed purge leftover workflow-file issue (ticket #19)
purge/remove: not deleting workflow-file anymore
purge/remove: not deleting workflow-file anymore
Fixed call-condition for checking of purge-conditions
Fixed issues with proper handling of missing mandatory fields.
Switched to JSON as capture request format (THIS BREAKS COMPATIBILITY with prior versions!)
Added new task type for JSON DublinCore capture request format.
Adding symlink for SD-PAL template (MLT capture export)
Letting go of Windows in favor of Xubuntu as ingest OS! 🌻️
Added new capture request tool template: JSON DublinCore
Added suffix-change on import for capture request formats.
Removed methods which are fine called as-is from abstract parent.
Added check for mandatory fields in capture request.
Enabled process/finalize methods in Abstract request_video_capture base.
Marked certain parts as unused/obsolete.
Split request_video_capture Task, to easier support different request file-formats.
Stable production use in Mthk.
Aborting on existing output files, instead of asking (capture export). Related ticket #17.
Partial fix: Adding a -n option to the FFmpeg-commandline to avoid locking of the task_trigger session? Real fix: Make sure that capture_export task cleans its to-be-created files (leftovers) from previous runs.
Could not move 'xxx.v_ingest' from todo to 'in_progress'
Task capture_export halts with "(target) file already exists. Overwrite? [y/N]
Added missing "hold" on Shotcut snap package to prevent undesired updates.
Installer: Disable Snap auto-updates
Embedded metadata: Map to FADGI guidelines
Fixed in revision 544.
Fixed save/load racing condition (related to #12)
Action "export settings": confusing loading default values instead of JSON
Fixed in revision 543.
export settings: Auto-loading JSON config if it exists (Fixes #12).
embedded metadata ICMT contains quotes around ArchiveSignature
embedded metadata ICMT contains quotes around ArchiveSignature
Action "export settings": metadata strings not escaped
Fixed in revision #540.
Action "export settings": metadata strings not escaped
Fixed in revision 542: Added new $quote parameter to ArchiveStructure::resolve_command() to make surrounding single-quotes (introduced by escapeshellarg()) optional. NOTE: Currently this is /only/ applied to MthkMovie::create_export_copy() to keep the affected lines in this change low. If the patch works and is stable, this shall be applied to all other calls of resolve_command() - and hand-quote their command strings once, too.
Switched to manual/optional cmd args quoting (fixes ticket #14)
Making quoting escaped shell args strings optional.
Fixed missing HTML escaping of metadata strings in export action.
embedded metadata ICMT contains quotes around ArchiveSignature
Export settings: Increase width of metadata input fields
Added in revision #539. New CSS class "input.export_settings" created and assigned.
Export settings: Increase width of metadata input fields
Export Settings: Metadata text input. Added option to set width (by CSS)
Export metadata: Rearranged HTML input elements for cleaner update to CSS class.
Updated revision/version info in trunk.
Filename/path of LowRes is empty in `av_metadata` action.
Seems to be fixed. Unable to reproduce. When hovering over the heading "Access Copy" - the correct filename is shown.
NTSC: generate_preview error "height not divisible by 4 (720x486)"
Embedded metadata: Map to FADGI guidelines
Fully implemented in revision #536. 😁️
Added ISRF (carrier sourceform) as embedded metadata (Hires-copy). Solves ticket #10.
Partially implemented in revision #535. TODO: Resolve ISRF=[@SOURCEFORM@] to depict physical carrier type (VHS, DigiBeta, DV, etc) string - read from carrier.xml.
Added and changed metadata mapping (FADGI AVI ListInfo) See ticket #10.
Action "export settings": confusing loading default values instead of JSON
Accept capture task: Permission denied not caught
Embedded metadata: Map to FADGI guidelines
Fixed in revision #534. There's now a new option "FILTERS_CROP[]" in config.inc.php which allows to set crop-conditions in "generate_preview" based on source resolution.
Added option to crop preview copy by source resolution (solves ticket #9)
Fixed description/title of launcher
Fixed wording (launcher)
Added DVA-specific desktop launchers.
Fixed typo in install-script download URL.
Switched to trunk.
Added remark which user to run install script as.
Switched back to trunk (from devel-branch vrecordx-202311)
Related info links: Are we using the correct PAR settings? (vrecord issue #689) Current vrecord resolutions (2024-05): https://github.com/amiaopensource/vrecord/issues/689#issuecomment-2108177958 https://forum.videohelp.com/threads/405710-Why-does-NTSC-video-usually-get-captured-at-a-resolution-of-720x480
Related info link: https://forum.videohelp.com/threads/405710-Why-does-NTSC-video-usually-get-captured-at-a-resolution-of-720x480
NTSC: generate_preview error "height not divisible by 4 (720x486)"
Merged 202312-vrecordx devel branch back to trunk.
Switched trunk to REL-2.0 (vrecordX/Linux)
Fixed wrong release number.
1st vrecordX Release
Before Linux
Forced using DAR from export_settings in capture_export.