[FR] Upload files from cache to remote server even when they aren't open

[Nickduino]

The doc says:

When downloading files from a server, they are by default stored in the cache. When a file in the cache is saved, it will automatically be uploaded.

But it only works with plain "Save" and "Save as", not with "Save a copy as".

More details and the use case:

I have script.py and want to save a copy (script.py.bak) to my remote server.

If I use "Save as", it works but now I'm editing the backup copy instead of the file...which is not how you manage backup copies.

If I use "Save a copy as", it will offer to save it in the cache folder but nothing on the interface allows to sync it with the remote server:

• it won't do it automatically (and I really think it should);
• hitting "Refresh" in the NppFTP window will only refresh the refresh the remote directory (it's expected but you could argue it would make sense to sync both ways too);
• hitting "Upload file" in the NppFTP window will only upload the currently open file (so, script.py, not script.py.bak).

Currently, the only way for the NppFTP plugin to sync the copy to the remote directory is to open the file in Notepad++, either by using "Save as" and closing it to manually open the original file again afterwards, or by using "Save a copy as" and manually opening the copy afterwards.

[chcg]

Would need to be checked if

NppFTP/src/PluginInterface.cpp

Lines 123 to 132 in 3fb6792

	case NPPN_FILESAVED: {
	BufferID buf = (BufferID)scNotification->nmhdr.idFrom;
	int size = ::SendMessage(nppData._nppHandle, NPPM_GETFULLPATHFROMBUFFERID, (WPARAM)buf, (LPARAM)NULL);
	if (size == -1)
	break;
	TCHAR * path = new TCHAR[size+1];
	::SendMessage(nppData._nppHandle, NPPM_GETFULLPATHFROMBUFFERID, (WPARAM)buf, (LPARAM)path);
	nppFTP.OnSave(path);
	delete [] path;
	break; }

is triggered in case of "Save a copy as" from N++ . From code inspection:

https://github.com/notepad-plus-plus/notepad-plus-plus/blob/6ea75e5d108c23d0ffeef77ff128da0206db93b1/PowerEditor/src/NppIO.cpp#L675-L679

that seems to be not the case by a first look.

So likely this way it could not be done without adding a notification at N++ level.