GenoPro Home
GenoPro Home  |  Get Started With My Genealogy Tree  |  Buy  |  Login  |  Privacy  |  Search  |  Site Map
 

GenoPro Support Forum




Author Default version should not be hard-coded
Posted Tuesday, October 09, 2018 - Post #38984
Famous Writer

Famous WriterFamous WriterFamous WriterFamous WriterFamous WriterFamous WriterFamous WriterFamous WriterFamous Writer

Customers
FamilyTrees.GenoPro.com
GenoPro version: 3.0.1.4

Last Login: Yesterday @ 10:49 PM
Posts: 352, Visits: 2,277
I recently realised that the default value for the IrfanView image is always the 32 bit version, even if GenoPro is running on a 64 bit machine.

I know it is not easy to detect this architectural detail in a program but maybe it would be possible to look for the presence of the 64 or 32 bit image in the %ProgramFiles%\IrfanView folder to determine which image is present, or if indeed it is missing?

There is actually an error message generated when the IrfanView image is not found, but it is difficult to spot in the log window.  Maybe a different color could be used and/or a final "done with errors" message could be considered?

Posted Wednesday, October 10, 2018 - Post #38988
Legendary Master

Legendary MasterLegendary MasterLegendary MasterLegendary MasterLegendary MasterLegendary MasterLegendary MasterLegendary MasterLegendary Master

Administrators
Customers
Important Contributors
FamilyTrees.GenoPro.com
GenoPro version: 3.0.1.4

Last Login: Yesterday @ 8:38 PM
Posts: 2,659, Visits: 19,338
Yes I should be able to sort something out to improve this.  

As GenoPro is 32bit then on a 64bit OS the environment variable %Program Files% always points at %SystemDrive%\Program Files (x86) therefore I will need to check the path %ProgramW6432%, which is only present in a 64bit OS.

I will need to change the way a default is obtained for parameters of type 'file'. Currently it is just read from the ConfigMsg.. file.

I did try 64bit IrfanView some time ago and found it very buggy and so I abandoned it. Hopefully it has improved by now.


'lego audio video erro ergo disco' or "I read, I listen, I watch, I make mistakes, therefore I learn"
Posted Friday, October 12, 2018 - Post #38991
Legendary Master

Legendary MasterLegendary MasterLegendary MasterLegendary MasterLegendary MasterLegendary MasterLegendary MasterLegendary MasterLegendary Master

Administrators
Customers
Important Contributors
FamilyTrees.GenoPro.com
GenoPro version: 3.0.1.4

Last Login: Yesterday @ 8:38 PM
Posts: 2,659, Visits: 19,338
Ok I think I have a fix for this.

Essentially I have added an extra attribute Default64 to the Irfanview config parameter in ConfigMsgBase.xml to specify the default 64bit executeable.

The scripts then check for it and if found test to see if the file exists and use that value if it does, otherwise the value of the standard Default attribute is used.

I have changed the message displayed in the log should the file not be found to be a red error message rather than a comment.  Not sure why it was a comment in the first place Ermm

There are also some cosmetic changes to the way the file parameters are presented in the Configuration dialog.

The attached zip contains the 3 amended files, ConfigMsgBase.xml, Init.htm & ParameterSettings.hta  that should be unzipped and copied into the skins folder Narrative Common\Code


'lego audio video erro ergo disco' or "I read, I listen, I watch, I make mistakes, therefore I learn"


Code.zip (8 views, 38.04 KB)
Posted Saturday, October 13, 2018 - Post #38995
Famous Writer

Famous WriterFamous WriterFamous WriterFamous WriterFamous WriterFamous WriterFamous WriterFamous WriterFamous Writer

Customers
FamilyTrees.GenoPro.com
GenoPro version: 3.0.1.4

Last Login: Yesterday @ 10:49 PM
Posts: 352, Visits: 2,277
Congratulations.  Tested on both 32 and 64 bit platforms.  Works OK for me.


Similar Topics

Click to view RSS...
Expand / Collapse

Reading This Topic

Expand / Collapse