We are using SteemJ for the SteemitWorldMap project and we recently found out that some comments are failing on user contaning a dot in the account name.
This led to some investigation in the SteemJ source code.
After some digging I found that issue comes from the generated permlink for the reply. Indeed, it is using the author account name to build the permlink therefore but this character is invalid in permlinks.
Looking at the condenser implementation I saw that is was removing the dot is such cases.
I implemented the same behavior in this PR. This should be the only character to remove since account names can only contain alphanumerical characters (which are valid) plus dot (which is not).
On the practical level, the fix is quite easy, it just remove the dot, if any, from the author name in the function that builds the permlink.
This has been tested successfully and the patched version is now powering SteemitWorldMap. The Pull Request was kindly merged by @dez1337 who is maintaining the SteemJ project.
Proof of work
- PR link should already be in the post but here it is, just in case.
- Here is a comment made using the patched version. (view it on steemd)
Thank you for your interest!
Posted on Utopian.io - Rewarding Open Source Contributors
Again, thanks for helping out making Steemitworldmap better and better!
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Sure ! @deez1337 and its @steemj project definitely helped to focus on business code :-D
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
I didn't know about @steemitworldmap, it looks very good. Thanks for sharing and for your contributions to the project.
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Glad you like it :-D More is comming !
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Congratulations @oroger! You have received a personal award!
Click on the badge to view your own Board of Honor on SteemitBoard.
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Hey @oroger I am @utopian-io. I have just upvoted you!
Achievements
Community-Driven Witness!
I am the first and only Steem Community-Driven Witness. Participate on Discord. Lets GROW TOGETHER!
Up-vote this comment to grow my power and help Open Source contributions like this one. Want to chat? Join me on Discord https://discord.gg/Pc8HG9x
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Thank you for the contribution. It has been approved.
Bug fixes are always quite easy in hindsight - but finding them and fixing them without causing more damage takes time and patience. Great work!
You can contact us on Discord.
[utopian-moderator]
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Hehe that's true :-)
Thanks !
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Great job dear friend we appreciate your efforts here thank you
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Congratulations @oroger! You have received a personal award!
Click on the badge to view your Board of Honor.
Do not miss the last post from @steemitboard:
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Congratulations @oroger! You received a personal award!
Click here to view your Board of Honor
Do not miss the last post from @steemitboard:
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Congratulations @oroger! You received a personal award!
You can view your badges on your Steem Board and compare to others on the Steem Ranking
Do not miss the last post from @steemitboard:
Vote for @Steemitboard as a witness to get one more award and increased upvotes!
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit