Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Order of tasks are shuffled #318

Closed
lunaire opened this issue Feb 6, 2013 · 51 comments
Closed

Order of tasks are shuffled #318

lunaire opened this issue Feb 6, 2013 · 51 comments

Comments

@lunaire
Copy link

lunaire commented Feb 6, 2013

This has happened seemingly everytime the server crashes or I get disconnected (i.e. today for the upgrade). The order of daily and todos are shuffled in a seemingly random fashion. Quite annoying since I had them ordered in order of importance.

Also, now I cannot re-order the tasks; it gets reset to this current order after I refresh the page.

[Intel/ windows 7 system, chrome browser]

@lefnire
Copy link
Contributor

lefnire commented Feb 6, 2013

Confirmed. Fixing

On Wed, Feb 6, 2013 at 6:10 PM, lunaire [email protected] wrote:

This has happened seemingly everytime the server crashes or I get
disconnected (i.e. today for the upgrade). The order of daily and todos are
shuffled in a seemingly random fashion. Quite annoying since I had them
ordered in order of importance.

Also, now I cannot re-order the tasks; it gets reset to this current order
after I refresh the page.


Reply to this email directly or view it on GitHubhttps://github.com//issues/318.

@StanLindsey
Copy link
Contributor

I get this too, sadface.

@horusofoz
Copy link

Here too :(

@wc8
Copy link
Contributor

wc8 commented Feb 7, 2013

Mine got rearranged up and is insisting on the new order of things. That is to say, after refresh, it reverts to the wrong order.

@lefnire
Copy link
Contributor

lefnire commented Feb 7, 2013

I've found the issue, but I'm not sure how to fix it. Here's the backstory:

Derby uses refLists to maintain order of object lists. So you have user.tasks stored willy-nilly, then you have user.idLists.habits which filters tasks by type (habit), and maintains sort order. Well, a while back people were just losing their tasks wholesale. Turns out they weren't losing anything from user.tasks, they were just losing user.idLists.*. My hack to that can be found here, which looks at user.tasks and puts it back into the refLists if they went missing for some reason - it only happened to some people, not everyone. Now, however, it's happening to everyone. So every refresh causes the idLists to go missing, and this hack puts all the taks back into place - in the order tasks were added to the user object. In other words, this hack can no longer be employed and we need to find the root of corrupt user.idLists.*

I'm babbling in case devs (@SlappyBag) are listening :) And thinking out loud.

@lefnire lefnire mentioned this issue Feb 9, 2013
@StanLindsey
Copy link
Contributor

As I mentioned on gTalk instead of having two objects; one the tasks themselves and one for the task lists. Could you put the an extra variable in the actual task itself with its position. Then it could be uploaded/downloaded as one whole object instead of two and if the tasks display you'd know that variable was coming down the pipes. Then you'd just need the JS to display them in the order of that variable?

@lefnire
Copy link
Contributor

lefnire commented Feb 9, 2013

fixed 5ea0dfe

@lefnire lefnire closed this as completed Feb 9, 2013
@lefnire
Copy link
Contributor

lefnire commented Feb 9, 2013

I was lucky enough to have read @SLaks' commit message for this just today, I never would have figured this one out on my own SLaks/racer@8792107

@lefnire
Copy link
Contributor

lefnire commented Feb 15, 2013

some people are still finding this to be an issue, re-opening

@lefnire lefnire reopened this Feb 15, 2013
@ghost
Copy link

ghost commented Feb 15, 2013

Happened to me just now...

@StanLindsey
Copy link
Contributor

_< I blame the date pickers.

@lefnire
Copy link
Contributor

lefnire commented Feb 16, 2013

naw, we're reverted back to a commit before due-dates. It's actually been an issue for the last few days still. I blame refLists. Also, I'm keeping a keen eye on https://github.com/SLaks/racer - this guy's been fixing up refLists and model queries like woa

@StanLindsey
Copy link
Contributor

I was making a funny =(

Many projects seem to be having problems withth derby refLists if I recal

@SLaks
Copy link

SLaks commented Feb 17, 2013

If you can reproduce this, can you try to determine whether the IDs are wrong (in which case the problem would probably be in listenerQueryMutator or types/find.js) or whether the refList is broken?

@mattrozak
Copy link

i went to assign a due date on a todo. As soon as I clicked on the date textbox, all habits, dailies, todos, and rewards vanished. After I refreshed, everything was still there but was shuffled.

Whenever my tasks do get shuffled, it's usually preceded by everything vanishing momentarily/until a refresh.

@kwilson328
Copy link

Haven't touched due dates yet, but experiencing the same event order as @mattrozak, vanishing -> refresh -> shuffling

Chrome v24.0.1312.57
OSX 10.7.5

@lefnire
Copy link
Contributor

lefnire commented Feb 20, 2013

possible clue codeparty/racer-db-mongo#6 (comment)

@SLaks I'll give your suggestions a try moment I can, thanks!

@ericwilmoth
Copy link

very much looking forward to the solution...

@l--l
Copy link

l--l commented Feb 23, 2013

Even if the order of tasks is not shuffled, most days I can't even drag and drop my tasks to arrange them in the order I want. This function seems to be working only intermittently. Any idea when this might be stable?

Chrome 25.0.1364.99
Mac OS 10.8.2

@horusofoz
Copy link

Today this is not happening for me. Can anyone else confirm if it appears resolved?

@horusofoz horusofoz reopened this Feb 23, 2013
@horusofoz
Copy link

Closed in error.

@ghost
Copy link

ghost commented Feb 23, 2013

It is still happening to me

@ericwilmoth
Copy link

Does it matter if we publicly post them, or should it be a P.M?
I've been avoiding using RPG till there was a clear fix for this...so short of reordering things and seeing if it fails again i can only assume it's still a problem...unless there was an attempted fix I'm unaware of?

@lefnire
Copy link
Contributor

lefnire commented Mar 5, 2013

email it to me directly: link (though, since api-tokens were enacted it is safe to show UUIDs publicly)

@lefnire
Copy link
Contributor

lefnire commented Mar 5, 2013

I haven't fixed anything directly, but I've been merging in SLaks' Racer fork commits periodically, many of which deal directly with refLists (refLists definitely being the issue with this ticket). So I check in here from time to time to see if any recent Racer fixes solved the problem..

@ericwilmoth
Copy link

Thanks--i just sent my id to you. Given your last comment, does that mean I should give it a shot and see what happens, or wait till you've had a chance to play with it further?

@ericwilmoth
Copy link

incidentally, i just tried moving a couple items, the page hiccuped...i refreshed, and the changes i made were discombobulated again, so it's definitely still happening.

@horusofoz
Copy link

If it helps, I'm not experiencing any shuffling issues or trouble reordering tasks, however I found I can move tasks around with out issue but still can't move rewards. Same behaviour as before where it selects text rather than move around the item.

@lefnire
Copy link
Contributor

lefnire commented Mar 5, 2013

You can't move items, those aren't sortable. reward-rewards?

On Tuesday, March 5, 2013, horusofoz wrote:

If it helps, I'm not experiencing any shuffling issues or trouble
reordering tasks, however I found I can move tasks around with out issue
but still can't move rewards. Same behaviour as before where it selects
text rather than move around the item.


Reply to this email directly or view it on GitHubhttps://github.com//issues/318#issuecomment-14437768
.

@Pandoro
Copy link

Pandoro commented Mar 5, 2013

@lefnire, no I can confirm this, sometimes I can drag and drop custom
rewards, sometimes it will just select them. This seems to be related to
the duplicate reward bug though.

On Tue, Mar 5, 2013 at 2:07 PM, Tyler Renelle [email protected]:

You can't move items, those aren't sortable. reward-rewards?

On Tuesday, March 5, 2013, horusofoz wrote:

If it helps, I'm not experiencing any shuffling issues or trouble
reordering tasks, however I found I can move tasks around with out issue
but still can't move rewards. Same behaviour as before where it selects
text rather than move around the item.


Reply to this email directly or view it on GitHub<
https://github.com/lefnire/habitrpg/issues/318#issuecomment-14437768>
.


Reply to this email directly or view it on GitHubhttps://github.com//issues/318#issuecomment-14438949
.

@lefnire
Copy link
Contributor

lefnire commented Mar 5, 2013

Roger that, I'll work on rewards today then

On Tuesday, March 5, 2013, Pandoro wrote:

@lefnire, no I can confirm this, sometimes I can drag and drop custom
rewards, sometimes it will just select them. This seems to be related to
the duplicate reward bug though.

On Tue, Mar 5, 2013 at 2:07 PM, Tyler Renelle <[email protected]<javascript:_e({}, 'cvml', '[email protected]');>>wrote:

You can't move items, those aren't sortable. reward-rewards?

On Tuesday, March 5, 2013, horusofoz wrote:

If it helps, I'm not experiencing any shuffling issues or trouble
reordering tasks, however I found I can move tasks around with out
issue
but still can't move rewards. Same behaviour as before where it
selects
text rather than move around the item.


Reply to this email directly or view it on GitHub<
https://github.com/lefnire/habitrpg/issues/318#issuecomment-14437768>
.


Reply to this email directly or view it on GitHub<
https://github.com/lefnire/habitrpg/issues/318#issuecomment-14438949>
.


Reply to this email directly or view it on GitHubhttps://github.com//issues/318#issuecomment-14439582
.

@lunaire
Copy link
Author

lunaire commented Mar 5, 2013

Just a few extra info: I'm the one that opened this issue, and this issue
is fixed for me. No more shuffling of tasks or rewards.

I am currently unable to move tasks/rewards manually on one of my
computers(it only selects the text when I click & drag), but I think this
is a separate issue.

On Tue, Mar 5, 2013 at 8:32 AM, Tyler Renelle [email protected]:

Roger that, I'll work on rewards today then

On Tuesday, March 5, 2013, Pandoro wrote:

@lefnire, no I can confirm this, sometimes I can drag and drop custom
rewards, sometimes it will just select them. This seems to be related to
the duplicate reward bug though.

On Tue, Mar 5, 2013 at 2:07 PM, Tyler Renelle <[email protected]<javascript:_e({},
'cvml', '[email protected]');>>wrote:

You can't move items, those aren't sortable. reward-rewards?

On Tuesday, March 5, 2013, horusofoz wrote:

If it helps, I'm not experiencing any shuffling issues or trouble
reordering tasks, however I found I can move tasks around with out
issue
but still can't move rewards. Same behaviour as before where it
selects
text rather than move around the item.


Reply to this email directly or view it on GitHub<
https://github.com/lefnire/habitrpg/issues/318#issuecomment-14437768>
.


Reply to this email directly or view it on GitHub<
https://github.com/lefnire/habitrpg/issues/318#issuecomment-14438949>
.


Reply to this email directly or view it on GitHub<
https://github.com/lefnire/habitrpg/issues/318#issuecomment-14439582>
.


Reply to this email directly or view it on GitHubhttps://github.com//issues/318#issuecomment-14439928
.

@BenMilford
Copy link

Just to add to the mix, I've been able to order items without shuffling since yesterday.

@derekantrican
Copy link

While this is still open, I'd like to mention another bug. (It seemed better to post it here because this thread is mostly related and I didn't think it was necessary to open a new one. But if I should open a new thread, let me know).

Basically: I can't rearrange rewards.

@lefnire
Copy link
Contributor

lefnire commented Mar 5, 2013

reward re-arrange should be fixed now (note you can't rearrange store items)

@horusofoz
Copy link

Confirmed now able to rearrange custom rewards

@ericwilmoth
Copy link

I've been back in the saddle for a few days now and everything seems to be staying sorted, so far as I can tell, it's fixed. Thanks!

@lefnire
Copy link
Contributor

lefnire commented Mar 10, 2013

cross our fingers!

@Pandoro
Copy link

Pandoro commented Mar 10, 2013

Hate to say it, but this just happened for me. Was refreshing as some of the stuff did not check correctly and after several refreshs, eventually everything got messed up :x

It is not so critical, as it is a side case, but I think it is still a problem ;x

@lefnire
Copy link
Contributor

lefnire commented Mar 10, 2013

we're exploring moving away from refLists, I'll chime in here if that happens

@SLaks
Copy link

SLaks commented Mar 10, 2013

Note that I'm in the middle of some very heavy performance work on refLists (see https://github.com/SLaks/racer/commits/refList-perf), which affects the behavior of nested refLists and may help here.

I'd like to take a look at this issue and see if I can figure out what's going on here, but I won't have any time for that for at least a week or two.

@lefnire
Copy link
Contributor

lefnire commented Mar 10, 2013

@SLaks I don't think it's performance issues per se effecting this ticket. We're experiencing some hard-to-explain funky-ness with refLists in other areas of the app too. Eg, we're trying to add categories (see commit), and adding new items to the refList bugs out which task has the category assigned to it - seems the :task.id reference gets shifted. It could be we're not using them properly, or bugs in our own app - but we're thinking of just moving our refLists to arrays instead. @toebu knows more some of the refList issues.

I see you've been just strong-arming the Racer codebase though, I'm a huge fan of your fork!

@SLaks
Copy link

SLaks commented Mar 10, 2013

I realize that performance is not what you're looking for, but I'm reworking enough of the code that it might help.

Two particular weaknesses in the current Racer code that I've noticed but am not working on right now is handling changes to parent paths (see my recent Racer issues) and translating paths when performing set operations on refLists (which may be causing some of your issues)

@lefnire
Copy link
Contributor

lefnire commented Mar 10, 2013

Here's a description of above issue: #634 - totally don't meant to rope you into our troubles, I know you've got a lot of unroll.me on your plate :) But it could provide some handy info anyway.

Actually, that last bit sounds exactly like the issue - translating paths when performing set operations.

@paglias
Copy link
Contributor

paglias commented Sep 9, 2013

Hi! The site has been totally rewritten from the ground up, I'm closing the issue since it's probably fixed. Check if it's still happening and in case reopen the bug, thanks! Anyway sorting has still to be ported to new site

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests