tuxxy/simpl

Cannot update an entry's comment with more than one word.

Closed this issue · 2 comments

tuxxy commented

Currently, the way the arguments are passed in Simpl._update_entry make it impossible to update an entry's comment with more than one word.

It should be an easy fix and I'll get around to it eventually.

tuxxy commented

To elaborate on the issue revealed in commit 8494da8, if a user makes a query like update test comment=This does indeed work!, then the comment will be set as it was designed.

The issue is revealed when a query is made like update test username=newUsername, comment=This doesn't work!. Instead of setting the entry's comment, comment is defaulted to NoneType; thus, not changing anything as per the specs in Locker.update.

This should be a simple fix and I will fix it when I have the chance!

tuxxy commented

Upon investigation, I have found that this issue is not just specific for comment. It is for any attribute that is after the first given one.

I am working on a fix for this now.