Deleting multiple selected files from right to left, results in IndexError

[GiorgosXou]

Issue:

TUIFIManager/TUIFIManager/__init__.py

Lines 667 to 682 in 0390f03

	else: # if self.__count_selected > 1: # Why do i even > 1 very sus
	if self.__clicked_file:
	temp_i = self.__index_of_clicked_file - self.__count_selected
	else:
	temp_i = 0 # VERY SUS BUT NVM NOW
	for f in self.files:
	if f.is_selected:
	self.__delete_file(f)
	if self.__count_selected == 0:
	self.reload(keep_search_results=True)
	self.__index_of_clicked_file = temp_i
	self.__clicked_file = self.files[temp_i]
	self.__pre_clicked_file = None # hmm.. sus
	self.select(self.__clicked_file)
	break

Fixed solution:

else: # if self.__count_selected > 1:  # Why do i even > 1 very sus
    temp_i=0
    for f in self.files:
        if f.is_selected: # first file is never selected because it is the .. one
            temp_i-=1
            self.__delete_file(f)
            if self.__count_selected == 0:
                break
        temp_i+=1
    self.reload(keep_search_results=True)
    self.__index_of_clicked_file = temp_i
    self.__clicked_file          = self.files[temp_i]
    self.__pre_clicked_file      = None # hmm.. sus
    self.select(self.__clicked_file)

[GiorgosXou]

Improved and fixed

    def delete(self):
        """
        Deletes the selected file(s). | Not fully implemented yet
        """
        if not self.has_write_access(self.directory): return
        if self.__count_selected == 1 and self.__clicked_file :
            # checking under __delete_file too but nvm cause i have no time right now
            if self.__clicked_file.name != '..':
                self.__delete_file(self.__clicked_file)
                self.files.pop(self.__index_of_clicked_file)
                temp_i = self.__index_of_clicked_file - 1
                self.resort() # replaced -> self.reload(keep_search_results=True)
                self.__index_of_clicked_file = temp_i
                self.__clicked_file          = self.files[temp_i]
                self.__pre_clicked_file      = None # hmm.. sus?
                self.__pre_hov               = None           
                self.select(self.__clicked_file)
        else: # if self.__count_selected > 1:  # Why do i even > 1 very sus
            temp_i=0
            while True:
                if self.files[temp_i].is_selected: # first file is never selected because it is the .. one
                    self.__delete_file(self.files[temp_i])
                    del self.files[temp_i]
                    temp_i-=1
                    if self.__count_selected == 0:
                        break
                temp_i+=1
            self.resort() # replaced -> self.reload(keep_search_results=True)
            self.__index_of_clicked_file = temp_i
            self.__clicked_file          = self.files[temp_i]
            self.__pre_clicked_file      = None # hmm.. sus
            self.__pre_hov               = None           
            self.select(self.__clicked_file)