XN Resource Editor 3.0.0.1 Portable Pre-Release 4

I'm pretty sure that the license is wrong.

I've just tested it for a few minutes with vista ultimate sp1, and it works well so far and leaves nothing obvious behind

Finally back on a Windows machine for a bit, so here's some small stuff I picked up on.

1. The title of this post says XNRes..., should be XN Res...
2. Change the ;Website in the source to \XNResourceEditorPortable, as well as the !define WEBSITE.
3. Change the !define PORTABLEAPPNAME to XN Resource Editor Portable, and the !define APPNAME to XN Resource Editor.
4. Change the LegalCopyright to PortableApps.com & Contributors. That was my fail previously.
5. In the help.html, you have a link to an old forum (Node 12321, what a cool number). Replace that with 'Portable App Issues' as seen in the Jooleem Portable help file.
6. The title and the h1 of the help file have 0.5 as the version number.
7. Change the homepage from the appinfo.ini to /XNResourceEditorPortable
8. The appinfo.ini description says XNResourceEditor, throw some spaces in there.

And that's it assuming you compressed and the icon is right.

Nice work... it's handy to have this portable.

I've just tried Pre-release 2 with vista ultimate sp1. I haven't had any noticeable problems so far, and it leaves nothing obvious behind.

but shouldn't be the filename XN_Resource_Editor_Portable_3.0.0.1_PRERELEASE_2_en_us.paf.exe?

• Delete Thumbs.db in \app\appinfo
• Readme.txt: Copyright 2004-2008 John T. Haller

Nice work

Try and load firefox.exe and click on an icon group. Gives "Pixel Format Not Valid for Icons or Cursors." for me, continuously. :/

Also, it hasn't been updated since December, 2005, so I'd say it's not going to be fixed any time soon. In fact, due to the age, it may be better just to drop this

Hey, for me at least the download link doesn't work it brings me to http://patrickpatience.com/nascent-project/powerjuce/XN_Resource_Editor_...
on that page it says :
Not Found
Sorry, but you are looking for something that isn't here.

There's a problem with the registry handling in this launcher.

The HKLM backup/restore section (lines 218 - 220 and 255 - 258) is being skipped if there's no local installation so, if you're logged in as admin, the key "HKEY_LOCAL_MACHINE\Software\Woozle" is left behind. You can generate this key for testing purposes by going to "Help/About..."

The two sections are currently...

RegistryBackup:
	StrCmp $SECONDARYLAUNCH "true" LaunchAndExit
	;=== Backup the registry
	${registry::KeyExists} "HKEY_CURRENT_USER\Software\Woozle-BackupByXNResourceEditorPortable" $R0
	StrCmp $R0 "0" RestoreTheKey
	${registry::KeyExists} "HKEY_CURRENT_USER\Software\Woozle" $R0
	StrCmp $R0 "-1" RestoreTheKey
	${registry::MoveKey} "HKEY_CURRENT_USER\Software\Woozle" "HKEY_CURRENT_USER\Software\Woozle-BackupByXNResourceEditorPortable" $R0
	StrCmp $USERTYPE "Guest" RestoreTheKey
	StrCmp $USERTYPE "User" RestoreTheKey
	${registry::MoveKey} "HKEY_LOCAL_MACHINE\Software\Woozle" "HKEY_CURRENT_USER\Software\Woozle-BackupByXNResourceEditorPortable" $R0
	Sleep 100

In the above, line 220 (MoveKey HKLM) is referencing different keys anyway.

SetOriginalKeyBack:
	${registry::DeleteKey} "HKEY_CURRENT_USER\Software\Woozle" $R0
	Sleep 100
	${registry::KeyExists} "HKEY_CURRENT_USER\Software\Woozle-BackupByXNResourceEditorPortable" $R0
	StrCmp $R0 "-1" TheEnd
	${registry::MoveKey} "HKEY_CURRENT_USER\Software\Woozle-BackupByXNResourceEditorPortable" "HKEY_CURRENT_USER\Software\Woozle" $R0
	StrCmp $USERTYPE "Guest" RestoreTheKey
	StrCmp $USERTYPE "User" RestoreTheKey
	${registry::DeleteKey} "HKEY_LOCAL_MACHINE\Software\Woozle" $R0
	${registry::MoveKey} "HKEY_LOCAL_MACHINE\Software\Woozle-BackupByXNResourceEditorPortable" "HKEY_LOCAL_MACHINE\Software\Woozle" $R0
	Sleep 100
	Goto TheEnd

I have fixed it by changing them to...

RegistryBackup:
	;=== Backup the registry
	${registry::KeyExists} "HKEY_CURRENT_USER\Software\Woozle-BackupByXNResourceEditorPortable" $R0
	StrCmp $R0 "0" RegistryBackup2
	${registry::KeyExists} "HKEY_CURRENT_USER\Software\Woozle" $R0
	StrCmp $R0 "-1" RegistryBackup2
	${registry::MoveKey} "HKEY_CURRENT_USER\Software\Woozle" "HKEY_CURRENT_USER\Software\Woozle-BackupByXNResourceEditorPortable" $R0
	Sleep 100

RegistryBackup2:
	UserInfo::GetAccountType
	Pop $USERTYPE
	StrCmp $USERTYPE "Guest" RestoreTheKey
	StrCmp $USERTYPE "User" RestoreTheKey
	${registry::MoveKey} "HKEY_LOCAL_MACHINE\Software\Woozle" "HKEY_LOCAL_MACHINE\Software\Woozle-BackupByXNResourceEditorPortable" $R0
	Sleep 100

SetOriginalKeyBack:
	${registry::DeleteKey} "HKEY_CURRENT_USER\Software\Woozle" $R0
	Sleep 100
	${registry::KeyExists} "HKEY_CURRENT_USER\Software\Woozle-BackupByXNResourceEditorPortable" $R0
	StrCmp $R0 "-1" SetOriginalKeyBack2
	${registry::MoveKey} "HKEY_CURRENT_USER\Software\Woozle-BackupByXNResourceEditorPortable" "HKEY_CURRENT_USER\Software\Woozle" $R0
	Sleep 100

SetOriginalKeyBack2:
	StrCmp $USERTYPE "Guest" TheEnd
	StrCmp $USERTYPE "User" TheEnd
	${registry::DeleteKey} "HKEY_LOCAL_MACHINE\Software\Woozle" $R0
	${registry::MoveKey} "HKEY_LOCAL_MACHINE\Software\Woozle-BackupByXNResourceEditorPortable" "HKEY_LOCAL_MACHINE\Software\Woozle" $R0
	Sleep 100
	Goto TheEnd

Which seems to sort it out.

Thanks for the program. Hope I'm helping