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

sort order not being updated on delete #2

Closed
tmkp opened this issue Jun 28, 2012 · 3 comments
Closed

sort order not being updated on delete #2

tmkp opened this issue Jun 28, 2012 · 3 comments

Comments

@tmkp
Copy link

tmkp commented Jun 28, 2012

just a minor thing, the sort order is not being updated when deleting records.

even after refreshing the admin panel i'm getting the old id's.

working great otherwise, definitely a must-have!

@UndefinedOffset
Copy link
Owner

Ya the reason for this is because of the potential for large data sets, it shouldn't be an issue though. Because of how the sort algo works there should never be the possibility of overlapping of the indexes, and the gaps in the index are just ignored anyways ;).


Keelah se'lai
Ed Chipman
http://www.edchipman.ca

On 2012-06-28, at 5:48 AM, [email protected] wrote:

just a minor thing, the sort order is not being updated when deleting records.

even after refreshing the admin panel i'm getting the old id's.

working great otherwise, definitely a must-have!


Reply to this email directly or view it on GitHub:
#2

@tmkp
Copy link
Author

tmkp commented Jun 28, 2012

okay i see.. more of a cosmetical issue then, if i have the SortID column shown in the list view. might work around it with a custom getter. thanks anyway!

@UndefinedOffset
Copy link
Owner

ya it's really not supposed to be displayed lol. The main challenge was performance when updating large data sets so it's easier and more efficient to essentially ignore the gaps in the sort index.

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

No branches or pull requests

2 participants