https://github.com/jrswab/nebulus
The following issues where brought to my attention during the last review. Since the past week was spent away with family no new features were able to be added.
This was set by me in the html markup. While this may not cause an issue (or at least work) it is better to use the fix. This way the URL encoded characters are used correctly. This will also save time in the future if there needs to be a URL change for SteemConnect.
PHP's http_build_query takes an array of data and uses that to construct the link with the proper encoded characters. Our array looks as follows:
$info = array('to' => 'nebulus', 'from' => $steemName,
'amount' => '1.000 STEEM', 'memo' => 'pin '.$pinHash,
'redirect_uri' => 'https://'.$domain.'/welcome.php');
Then we assign the HTTP build method to a variable or easy use in the URL:
$sc2Link = http_build_query($info);
The build method uses both the assigner and the assignee in the link building. So 'to' => 'nebulus' becomes to=nebulus in the URL with any needed encoding. A good example of this is the 'memo' => pin '.$pinHash. In this example, the space between the word pin and the hash variable is encoded with a + symbol.
So, instead of hard-coding:
https://steemconnect.com/sign/transfer?to=nebulus&amount=1.000+STEEM&from='.$steemName.'&memo=pin+'.$pinHash.'&redirect_uri=https://'.$domain.'/welcome.php
we now use:
https://steemconnect.com/sign/transfer?'.$sc2Link.'
This markup is much less likely to get screwed when changing parameters or updating the link as a whole.
roundRobin in the global scopeI am glad 
All that we needed to do was to move tho variable out of the global scope and into a function. Now within the robin function we define roundRobin instead of calling the global variable.
def robin(bLimit):
roundRobin = 0
...
However, we do need that variable in other functions but since the robin function returns roundRobin we can call the robin function in place of the roundRobin global variable.
So instead of executing two functions at the bottom of the Python code we now only call one (sort of):
robin(blockNum)
pin(roundRobin, blockNum)
Becomes:
pin(robin(bLimit), bLimit)
robin() is still called but as an argument of pin() and Python runs robin() to get the return value and use that as the roundRobin variable within pin().
I am still thinking of a suitable way to define bLimit and remove it from the global scope. If you have any ideas, please let me know on Discord or steem.chat.
Another great spot by Justyy, the two functions pinAdd and pidKill had no checks to make sure the PID or hash was present upon running the code. The lack of testing created risks in the code that could have forced the execution to quit and no subsequent hashes to get pinned.
To fix this, I added some true and false returns to the two functions so that we can check them within the pin() function's execution.
if pgreped is None:
return None
else:
return pgreped
if pgreped is None:
print("Nothing to kill")
I chose to add a print statement in pidKill for logging purposes.
Looking to take your Steem based creations to the next level?
Join us over at the Creators' Guild Discord group! We are here to encourage, support and increase the creation of quality content.
If you have any questions about the future of Steem
or my witness please feel free to message jrswab#3134 on Discord.
Click here to vote with SteemConnect!
Or go to https://steemit.com/~witnesses
and type jrswab in the box at the bottom.
You can see all active witnesses on 
Click here to join the mailing list and get exclusive SDB/STEEM giveaways!
Looking to support my content creation efforts outside of the Steem Blockchain?
Check out jrswab.com/support
Mastodon | Keybase | Twitter | Gab | Minds | Hacker Culture Podcast