creatubbles / ctb-mcmod

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Revert screenshot grabbing order for upload

swilgosz opened this issue · comments

This need to be checked first, but the issue is:

by default, running /ctb-upload test1 will grab the oldest upload to proceed. We need to revert that, to grab the newest one instead.

  • ctb-upload "Title 1" 0 (or 'Tittle 1' or no attribute at all) would refer to the latest screenshot available in the queue of screenshots in memory (should be the most recent screenshot taken)
  • ctb-upload "Title 2" 1 > to refer to the screenshot taken before the last.
  • ctb-upload "Title 3" N > to refer to the screenshot taken N screenshots before

I think it would be more intuitive for minecraft users

Is this not how it works? This was the exact purpose of the ArrayUtils.reverse() call

No.
Check File.listFiles() documentation

There is no guarantee that the name strings in the resulting array will appear in any specific order; they are not, in particular, guaranteed to appear in alphabetical order.

now upload command supports:
ctb-upload Name
ctb-upload Name 1
ctb-upload "Name" 1
ctb-upload Name 4
ctb-upload "Very very long name"
ctb-upload "Very long name" 3

etc