legend Posted February 12, 2014 Posted February 12, 2014 I'm wondering why my script isn't working properly, right click on a file, and select : Encrypt, and it shows the full path to the file you clicked on, but the Decrypt option, does not work, and does not return the full math, ideas why? RegWrite("HKEY_CLASSES_ROOT\*\shell\Encrypt\command", '', 'REG_SZ', '"' & @ScriptFullPath & '" "%1"') RegWrite("HKEY_CLASSES_ROOT\*\shell\Encrypt", '', 'REG_SZ', "Encrypt") RegWrite("HKEY_CLASSES_ROOT\*\shell\Decrypt\command", '', 'REG_SZ', '"' & @ScriptFullPath & '" "%2"') RegWrite("HKEY_CLASSES_ROOT\*\shell\Decrypt", '', 'REG_SZ', "Decrypt") If $CmdLine[0]=1 Then MsgBox ('', 'Test', 'encrypt ' & $CmdLine[1]) EndIf If $CmdLine[0]=2 Then MsgBox ('', 'Test', 'decrypt ' & $CmdLine[2]) EndIf
guinness Posted February 12, 2014 Posted February 12, 2014 Change %2 to %1. UDF List: Reveal hidden contents _AdapterConnections() • _AlwaysRun() • _AppMon() • _AppMonEx() • _ArrayFilter/_ArrayReduce • _BinaryBin() • _CheckMsgBox() • _CmdLineRaw() • _ContextMenu() • _ConvertLHWebColor()/_ConvertSHWebColor() • _DesktopDimensions() • _DisplayPassword() • _DotNet_Load()/_DotNet_Unload() • _Fibonacci() • _FileCompare() • _FileCompareContents() • _FileNameByHandle() • _FilePrefix/SRE() • _FindInFile() • _GetBackgroundColor()/_SetBackgroundColor() • _GetConrolID() • _GetCtrlClass() • _GetDirectoryFormat() • _GetDriveMediaType() • _GetFilename()/_GetFilenameExt() • _GetHardwareID() • _GetIP() • _GetIP_Country() • _GetOSLanguage() • _GetSavedSource() • _GetStringSize() • _GetSystemPaths() • _GetURLImage() • _GIFImage() • _GoogleWeather() • _GUICtrlCreateGroup() • _GUICtrlListBox_CreateArray() • _GUICtrlListView_CreateArray() • _GUICtrlListView_SaveCSV() • _GUICtrlListView_SaveHTML() • _GUICtrlListView_SaveTxt() • _GUICtrlListView_SaveXML() • _GUICtrlMenu_Recent() • _GUICtrlMenu_SetItemImage() • _GUICtrlTreeView_CreateArray() • _GUIDisable() • _GUIImageList_SetIconFromHandle() • _GUIRegisterMsg() • _GUISetIcon() • _Icon_Clear()/_Icon_Set() • _IdleTime() • _InetGet() • _InetGetGUI() • _InetGetProgress() • _IPDetails() • _IsFileOlder() • _IsGUID() • _IsHex() • _IsPalindrome() • _IsRegKey() • _IsStringRegExp() • _IsSystemDrive() • _IsUPX() • _IsValidType() • _IsWebColor() • _Language() • _Log() • _MicrosoftInternetConnectivity() • _MSDNDataType() • _PathFull/GetRelative/Split() • _PathSplitEx() • _PrintFromArray() • _ProgressSetMarquee() • _ReDim() • _RockPaperScissors()/_RockPaperScissorsLizardSpock() • _ScrollingCredits • _SelfDelete() • _SelfRename() • _SelfUpdate() • _SendTo() • _ShellAll() • _ShellFile() • _ShellFolder() • _SingletonHWID() • _SingletonPID() • _Startup() • _StringCompact() • _StringIsValid() • _StringRegExpMetaCharacters() • _StringReplaceWholeWord() • _StringStripChars() • _Temperature() • _TrialPeriod() • _UKToUSDate()/_USToUKDate() • _WinAPI_Create_CTL_CODE() • _WinAPI_CreateGUID() • _WMIDateStringToDate()/_DateToWMIDateString() • Au3 script parsing • AutoIt Search • AutoIt3 Portable • AutoIt3WrapperToPragma • AutoItWinGetTitle()/AutoItWinSetTitle() • Coding • DirToHTML5 • FileInstallr • FileReadLastChars() • GeoIP database • GUI - Only Close Button • GUI Examples • GUICtrlDeleteImage() • GUICtrlGetBkColor() • GUICtrlGetStyle() • GUIEvents • GUIGetBkColor() • Int_Parse() & Int_TryParse() • IsISBN() • LockFile() • Mapping CtrlIDs • OOP in AutoIt • ParseHeadersToSciTE() • PasswordValid • PasteBin • Posts Per Day • PreExpand • Protect Globals • Queue() • Resource Update • ResourcesEx • SciTE Jump • Settings INI • SHELLHOOK • Shunting-Yard • Signature Creator • Stack() • Stopwatch() • StringAddLF()/StringStripLF() • StringEOLToCRLF() • VSCROLL • WM_COPYDATA • More Examples... Updated: 22/04/2018
legend Posted February 12, 2014 Author Posted February 12, 2014 If I do that, it will still run this line: MsgBox ('', 'Test', 'encrypt ' & $CmdLine[1]) Instead of this one: MsgBox ('', 'Test', 'decrypt ' & $CmdLine[2])
kylomas Posted February 12, 2014 Posted February 12, 2014 Try what guinness suggested and change your logic. I have never done this but it seems that each time you right click on a file you should be getting a new instance of the script. Forum Rules Procedure for posting code "I like pigs. Dogs look up to us. Cats look down on us. Pigs treat us as equals." - Sir Winston Churchill
legend Posted February 12, 2014 Author Posted February 12, 2014 I already tried that, but I don't see how it would fix it
BrewManNH Posted February 12, 2014 Posted February 12, 2014 What do you get from the msgbox using the decrypt menu option? If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag GudeHow to ask questions the smart way! Reveal hidden contents I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from. Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays. - ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script. - Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label. - _FileGetProperty - Retrieve the properties of a file - SciTE Toolbar - A toolbar demo for use with the SciTE editor - GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI. - Latin Square password generator
legend Posted February 12, 2014 Author Posted February 12, 2014 (edited) getting this for decrypt: and this for encrypt: Edited February 12, 2014 by legend
LongBowNZ Posted February 12, 2014 Posted February 12, 2014 (edited) Wouldn't the number of parameters in $CmdLine be the same in both. $CmdLine[0] returns the number of parameters on the command line. Try RegWrite("HKEY_CLASSES_ROOT\*\shell\Encrypt\command", '', 'REG_SZ', '"' & @ScriptFullPath & '" "%1"') RegWrite("HKEY_CLASSES_ROOT\*\shell\Encrypt", '', 'REG_SZ', "Encrypt") RegWrite("HKEY_CLASSES_ROOT\*\shell\Decrypt\command", '', 'REG_SZ', '"' & @ScriptFullPath & '" "%2"') RegWrite("HKEY_CLASSES_ROOT\*\shell\Decrypt", '', 'REG_SZ', "Decrypt") If $CmdLine[1]=1 Then MsgBox ('', 'Test', 'encrypt ' & $CmdLine[2]) EndIf If $CmdLine[1]=2 Then MsgBox ('', 'Test', 'decrypt ' & $CmdLine[2]) EndIf or RegWrite("HKEY_CLASSES_ROOT\*\shell\Encrypt\command", '', 'REG_SZ', '"' & @ScriptFullPath & '" "%1"') RegWrite("HKEY_CLASSES_ROOT\*\shell\Encrypt", '', 'REG_SZ', "Encrypt") RegWrite("HKEY_CLASSES_ROOT\*\shell\Decrypt\command", '', 'REG_SZ', '"' & @ScriptFullPath & '" "%2"') RegWrite("HKEY_CLASSES_ROOT\*\shell\Decrypt", '', 'REG_SZ', "Decrypt") If $CmdLine[2]=1 Then MsgBox ('', 'Test', 'encrypt ' & $CmdLine[1]) EndIf If $CmdLine[2]=2 Then MsgBox ('', 'Test', 'decrypt ' & $CmdLine[1]) EndIf Edited February 12, 2014 by LongBowNZ
BrewManNH Posted February 12, 2014 Posted February 12, 2014 Do this. RegWrite("HKEY_CLASSES_ROOT\*\shell\Encrypt\command", '', 'REG_SZ', '"' & @ScriptFullPath & '" "encrypt %1"') RegWrite("HKEY_CLASSES_ROOT\*\shell\Encrypt", '', 'REG_SZ', "Encrypt") RegWrite("HKEY_CLASSES_ROOT\*\shell\Decrypt\command", '', 'REG_SZ', '"' & @ScriptFullPath & '" "decrypt %1"') RegWrite("HKEY_CLASSES_ROOT\*\shell\Decrypt", '', 'REG_SZ', "Decrypt") If $CmdLine[0] = 1 Then MsgBox('', 'Test', $CmdLine[1]) EndIf If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag GudeHow to ask questions the smart way! Reveal hidden contents I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from. Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays. - ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script. - Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label. - _FileGetProperty - Retrieve the properties of a file - SciTE Toolbar - A toolbar demo for use with the SciTE editor - GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI. - Latin Square password generator
legend Posted February 12, 2014 Author Posted February 12, 2014 that way, it would still return the same thing, I need to make it do 2 different things
BrewManNH Posted February 12, 2014 Posted February 12, 2014 Then write your script to do that, you can't do it the way you tried it originally because you did it wrong. If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag GudeHow to ask questions the smart way! Reveal hidden contents I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from. Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays. - ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script. - Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label. - _FileGetProperty - Retrieve the properties of a file - SciTE Toolbar - A toolbar demo for use with the SciTE editor - GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI. - Latin Square password generator
legend Posted February 12, 2014 Author Posted February 12, 2014 (edited) got it solved, thanks everyone Edited February 12, 2014 by legend
kylomas Posted February 12, 2014 Posted February 12, 2014 legend, Please post the solution, if you can. kylomas Forum Rules Procedure for posting code "I like pigs. Dogs look up to us. Cats look down on us. Pigs treat us as equals." - Sir Winston Churchill
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now