Inconsistent treatment of NotifyScript between Box Backup and Boxi

[achim]

Inconsistent treatment of NotifyScript between Boxi and BB

NotifyScript = "C:\Documents and Settings\Administrador\NotifySysAdmin?.cmd" SyncAllowScript? = C:\Documents and Settings\Administrador\SyncAllowScript?.cmd

Please note that NotifyScript needs qutation marks around filenames with spaces, whereas SyncAllowScript? does not. NotifyScript under bbackupd.exe will result in errors like this when no "" are used around filenames with spaces:

C:\Documents is not recognised as an internal or external command, program or batch file. WARNING: Notify script returned error code: 1 ('C:\Documents and Settings\Administrador\NotifySysAdmin?.cmd read-error')

So we add spaces around the NotifyScript to please BB, but then Boxi complains with

The Notify Script does not exist ("C:\Documents and Settings\Administrador\NotifySysAdmin?.cmd")

In addition, when ignoring Boxi's error above (we know that the file exists, and that BB needs spaces around it!) and trying to restore (which should not require NotifyScript at all), the following error is given upon trying to log into the server on tab "Restore Files" to selec the files:

---

Boxi Error

---

Error connecting to server: Connection SocketNameLookupError? (Check hostname specified.)

---

OK

---

Possible solution: check in the Box Backup source why Notify and SyncAllow? treat spaces in filenames differently, and make sure they both accept filenames with spaces without the need for quotation marks.

[achim]

Fix so that bbackup will accept paths with spaces for NotifyScript: change BackupDaemon?.cpp line 2196 from

// Script to run std::string script(conf.GetKeyValue?("NotifyScript") + ' ' +

sEventNames[Event]);

to

// Script to run std::string script("\"" + conf.GetKeyValue?("NotifyScript") + "\"" + ' ' +

sEventNames[Event]);