Archive/README.md

@@ -154,6 +154,6 @@ Leading `"-"` indicates a not fully sorted shoot.
 - <date> - <info>/
 ```
 
-This is the default as created by `sync-flash.sh`, renaming (removing the leading `"- "`) should be done by the user.
+This is the defailt as creatrd by `sync-flash.sh`, renaming (removing the leading `"- "`) should be done by the user.
 
 

Archive/media/README

@@ -1,4 +1,5 @@
-# Base photo archive directory
+Base photo archive directory
+----------------------------
 
 This tree is created based on the specification from:
 	https://github.com/flynx/ImageGrid/blob/master/Archive/README.md

Archive/media/img/my/work/compress-archive.sh

@@ -1,4 +1,4 @@
-#!/usr/bin/env bash
+#!/bin/bash
 
 # TODO make this runnable from anywhere...
 #	- prepend paths with './' only if local/relative

Archive/media/img/my/work/process-archive.sh

@@ -1,4 +1,4 @@
-#!/usr/bin/env bash
+#!/bin/bash
 #
 #######################################################################
 #

Archive/media/img/my/work/sync-flash.sh

@@ -1,4 +1,4 @@
-#!/usr/bin/env bash
+#!/bin/bash
 
 VERSION=1.0
 

Archive/media/img/my/work/update-exif.sh

@@ -1,4 +1,4 @@
-#!/usr/bin/env bash
+#!/bin/bash
 
 
 

Viewer/features/metadata.js

@@ -113,9 +113,6 @@ var MetadataReaderActions = actions.Actions({
 			- force is true
 
 
-		NOTE: this will read metadata from both the image file as well as sidecar 
-			(xmp) files, if available
-		NOTE: sidecar metadata fields take precedence over image metadata.
 		NOTE: also see: .cacheMetadata(..)
 		`,
 		core.sessionQueueHandler('Read image metadata', 
@@ -138,17 +135,14 @@ var MetadataReaderActions = actions.Actions({
 				if(!image && !img){
 					return false }
 
-				if(!force 
-						&& (img.metadata || {}).ImageGridMetadata == 'full'){
-					return Promise.resolve(img.metadata) }
-
 				//var full_path = path.normalize(img.base_path +'/'+ img.path)
 				var full_path = this.getImagePath(gid)
 
+				return new Promise(function(resolve, reject){
+					if(!force 
+							&& (img.metadata || {}).ImageGridMetadata == 'full'){
+						return resolve(img.metadata) }
 
-				var readers = []
-				// main image...
-				readers.push(new Promise(function(resolve, reject){
 					fs.readFile(full_path, function(err, file){
 						if(err){
 							return reject(err) }
@@ -163,57 +157,36 @@ var MetadataReaderActions = actions.Actions({
 							img.ctime = stat.ctime
 							img.birthtime = stat.birthtime
 
-							img.size = stat.size }
+							img.size = stat.size
+						}
 
 						// read image metadata...
 						exiftool.metadata(file, function(err, data){
 							if(err){
 								reject(err)
+
 							} else if(data.error){
 								reject(data)
-							} else {
-								resolve(data) } }) }) }) ) 
-				// XMP sidecar files...
-				// NOTE: sidecar files take precedence over image metadata...
-				for(var ext of ['xmp', 'XMP']){
-					var xmp_path = full_path.replace(/\.[a-zA-Z0-9]*$/, '.'+ ext)
-					if(fs.existsSync(xmp_path)){
-						readers.push(new Promise(function(resolve, reject){
-							fs.readFile(xmp_path, function(err, file){
-								if(err){
-									// XXX log error...
-									resolve({}) }
-								exiftool.metadata(file, function(err, data){
-									if(err){
-										// XXX log error...
-										resolve({})
-									} else if(data.error){
-										// XXX log error...
-										resolve({})
-									} else {
-										console.log("@@@@", data)
-										resolve(data) } }) }) }))
-						break } }
 
-			// merge the data...
+							} else {
-			return Promise.all(readers)
+								// convert to a real dict...
-				.then(function(data){
+								// NOTE: exiftool appears to return an array 
-					// convert to a real dict...
+								// 		object rather than an actual dict/object
-					// NOTE: exiftool appears to return an array 
+								// 		and that is not JSON compatible....
-					// 		object rather than an actual dict/object
+								that.images[gid].metadata =
-					// 		and that is not JSON compatible....
+									Object.assign(
-					that.images[gid].metadata =
+										// XXX do we need to update or overwrite??
-						Object.assign(
+										that.images[gid].metadata || {},
-							// XXX do we need to update or overwrite??
+										data,
-							that.images[gid].metadata || {},
+										{
-							...data,
+											ImageGridMetadataReader: 'exiftool/ImageGrid',
-							{
+											// mark metadata as full read...
-								ImageGridMetadataReader: 'exiftool/ImageGrid',
+											ImageGridMetadata: 'full',
-								// mark metadata as full read...
+										})
-								ImageGridMetadata: 'full',
+								that.markChanged 
-							})
+									&& that.markChanged('images', [gid]) }
-					that.markChanged 
+
-						&& that.markChanged('images', [gid]) }) })],
+							resolve(data) }) }) }) })],
 	// XXX Q: should this be a linked task???
 	// 		...on one hand yes, on the other, saving after this may 
 	// 		unintentionally save other state from the main object...

Viewer/package.json

@@ -1,7 +1,7 @@
 {
   "name": "ImageGrid.Viewer.g4",
   "main": "index.html",
-  "version": "4.0.11a",
+  "version": "4.0.10a",
   "author": "Alex A. Naanou <alex.nanou@gmail.com> (https://github.com/flynx)",
   "contributors": [],
   "repository": "github:flynx/ImageGrid",