Strange load code

[Ducasse]

Why the opening logic is like that?
I would say either the file has metadata and we use it or the file does not have metadata.

loadFileContents: aString for: aNTNote

	| fileContentsPart |
	(aString beginsWith: '{') ifFalse: [ ^ self ].
	fileContentsPart := aString substrings: '}'.
	aNTNote contents:
		(fileContentsPart second withoutPrefix: String crlf , String crlf).
	self loadMetaData: fileContentsPart first for: aNTNote

[moufort]

Yes i think it's not the best logic.

The loadFileConents:for: method is use by

addNoteFromFileReference: aFileReference
	"Add a new note with title aString to the receiver's notes. Answer the receiver"

	| newNote |
	newNote := NTNote newFromFileReference: aFileReference.
	self loadFileContents: aFileReference contents for: newNote.
	self notes add: newNote.

	^ newNote

When I do the " NTNote newFromFileReference: aFileReference ", i give default value of the fileReference. So I use the fileReference contents to the new note, even the metadata. And I use the fileReference creation date and modification date, not using the metadata here.

After that, I do " self loadFileContents: aFileReference contents for: newNote. " ( it's the method you senf in the first message )
I look if the first character is a bracket. If yes, so I load metadata, and I change the note content and date. Else, I touch nothing.

For why I do this way, It's because for loading a note, I need to use NTContainer for the tags. If I initialize tags in NTNote, I need to create a newTag for each tagName, instead if they exist in NTContainer, cause the NTNote doesn't know NTContainer.
I see with Cyril, and he told me if I need to use NTContainer for import metadata, I need to up the import metadata logic to NTContainer.

Maybe the way I implemented it it's not very clean, and i see if there a problem in metadata, like a thing like in Issue #53 :

{<CR>
"lastModificationDate" : "2025-06-12T15:03:18.021427+02:00",<CR>
"creationDate" : "2025-06-12T14:56:18.97218+02:00"<CR>
}<CR>

Then that do a error, because I didn't plan this case