View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0000180 | OpenMPT | User Interface | public | 2011-09-07 07:03 | 2012-03-08 23:09 |
| Reporter | jmichae3 | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | no change required | ||
| Platform | xp pro sp3 32-bit | OS | xp pro sp3 32-bit | OS Version | xp pro sp3 |
| Product Version | OpenMPT 1.19.03.00 (upgrade first) | ||||
| Summary | 0000180: shortcut to web site is not proper | ||||
| Description | shortcut to web site, if NSIS was made was using probably the CreateShortCut command, which is wrong. there is a better way. | ||||
| Steps To Reproduce | install openmpt | ||||
| Additional Information | the code below I basically deroved from AfroUK's counter code. it creates shortcuts for 32 and 64-bit programs where you have the need for a default path (working directory) you can set for each shortcut and 32-bit DLL's and EXE's to be stored in say, a 32\ directory, while 64-bit executables and DLL's can be stored in a 64\ directory. this allows different 32/64-bit runtime DLL's with the same name as is the case with the mingw-w64 compiler. it also happens to create URL shortcuts the proper way. ;Author: Jim Michaels jmichae3@yahoo.com, Stu (AfroUK) ;credit for using Push and Pop for $OUTDIR and using SetOutPath to set the working dir goes to Stu. ;shortcut - macro that makes a basic shortcut with a working directory or an internet shortcut with no working directory. !ifndef _SHORTCUTCOUNTER | ||||
| Tags | No tags attached. | ||||
| Has the bug occurred in previous versions? | don't know | ||||
| Tested code revision (in case you know it) | |||||
|
1) We do not use NSIS but InnoSetup The shortcut in the start menu is clickable and opens the website, as it should do. I don't see anything that's wrong with it? |
|
|
as I discovered by myself some time ago, this method is functional. but it is not proper. real Internet Shortcuts are an .INI file renamed as a .url (have you ever opened one in a text editor?). and so they are in the start menu. If innosetup has any functions for writing .ini files, that would be the proper function for writing a .url file. as I stated before, the icon's look is broken. normally a web icon will look like the icon of the default browser. doing it this way does not. I used to do the same thing in my programs until I learned how to do it correctly. If innosetup does not have the write-an-INI facility, then I guess you can close this bug. I use NSIS. |
|
|
The source code I just quoted writes an INI file that has an URL extension. Windows probably just has a problem finding an icon for the linked URL file in the start menu. That is probably the least problem I could care about, since it looks like OpenMPT is not the only program with that "problem" - I got plenty of plain white icons in my start menu, mostly pointing to URLs, TXTs and similar documents. It is probably by design, since neither OpenMPT nor those other programs specifiy an explicit icon file to use for the link. |
|
|
this should be the appropriate innosetup code. I am not used to innosetup, but I don't mind learning something new. [INI] so here's your patch. what's in the latest release does not write an INI file, it creates a binary shortcut file. .ini files are text files. it basically looks like this: [InternetShortcut] anything after that may be binary or it may be text. |
|
|
Uhm, sorry, but the URL file is perfectly valid. You don't have to create sections before adding keys to them. Just look at the URL file itself and you'll see that it's valid. It's merely the start menu item that doesn't show an icon, like so many other start menu items of other programs, as I've said. |
|
|
I won't argue. |
|
Yes, there is a valid INI file in OpenMPT's directory. And yes, the start menu item is a shortcut. |
|
|
hmm. interesting: on your installation it comes out proper. but on mine it does not. I am on xp pro sp3 32-bit (which may make a difference, I don't know). my shortcut contents are char garbage (but opens up to web site). I could put contents here but mantis would probably choke on it. mine is not an INI file format, but shortcut format. yours is in INI file format. take a second look at your source. I am using 1.19.03.00 |
|
|
I've just tried it on WinXP myself, and, in contrast to Win7, it even shows an Internet Explorer icon in the start menu there, so the (binary) .lnk file in the start menu is correctly pointing to the (INI text) .url file there, just like on Win7. And as long as it does that, everything's OK, I guess? |
|
|
[Icons] here, you should not be creating an icon for the .url file. the .url file created in the [INI] section IS the icon/shortcut. when you do this right, windows will automatically create the icon for this when you open the start menu and it will look like the default web browser's icon. [INI] here for the .url file, {app} should be {group} because the .INI format .url file is supposed to be created in the start menu, not in the application directory! both .lnk and .url files are shortcuts (different kinds, one is a document/program shortcut, and the other is an internet shortcut). I hope this helps. |
|
|
Placing LNK files instead of the actual URL file in the start menu is common among all kinds of software and there is no Windows guidline that forbids doing that. |
|
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2011-09-07 07:03 | jmichae3 | New Issue | |
| 2011-09-07 10:23 | Saga Musix | Note Added: 0000417 | |
| 2011-09-08 20:45 | jmichae3 | Note Added: 0000440 | |
| 2011-09-08 20:48 | Saga Musix | Note Added: 0000441 | |
| 2011-09-08 21:20 | jmichae3 | Note Added: 0000443 | |
| 2011-09-08 21:22 | Saga Musix | Note Added: 0000444 | |
| 2011-09-08 21:39 | jmichae3 | Note Edited: 0000443 | |
| 2011-09-08 21:48 | jmichae3 | Note Added: 0000445 | |
| 2011-09-08 22:00 | Saga Musix | Note Added: 0000446 | |
| 2011-09-08 22:01 | Saga Musix | Note Edited: 0000446 | |
| 2011-09-08 22:40 | jmichae3 | Note Added: 0000447 | |
| 2011-09-08 23:17 | Saga Musix | Note Added: 0000448 | |
| 2011-09-09 00:00 | jmichae3 | Note Added: 0000449 | |
| 2011-09-09 11:02 | Saga Musix | Note Added: 0000450 | |
| 2012-03-08 23:09 | Saga Musix | Status | new => closed |
| 2012-03-08 23:09 | Saga Musix | Resolution | open => no change required |