Fix evidence out of range issue due to an oversight (#836)
This commit is contained in:
		
							parent
							
								
									1cfbfd51a7
								
							
						
					
					
						commit
						4124a43674
					
				@ -261,13 +261,14 @@ void Courtroom::set_evidence_list(QVector<evi_type> &p_evi_list)
 | 
				
			|||||||
  if (ui_evidence_overlay
 | 
					  if (ui_evidence_overlay
 | 
				
			||||||
          ->isVisible()) // Update the currently edited evidence for this user
 | 
					          ->isVisible()) // Update the currently edited evidence for this user
 | 
				
			||||||
  {
 | 
					  {
 | 
				
			||||||
 | 
					    int p_id = current_evidence - (max_evidence_on_page * current_evidence_page);
 | 
				
			||||||
    if (current_evidence >= local_evidence_list.size()) {
 | 
					    if (current_evidence >= local_evidence_list.size()) {
 | 
				
			||||||
      evidence_close();
 | 
					      evidence_close();
 | 
				
			||||||
      ui_evidence_name->setText("");
 | 
					      ui_evidence_name->setText("");
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
    else if (ui_evidence_ok->isHidden()) // We haven't clicked to edit it or anything
 | 
					    else if (ui_evidence_ok->isHidden()) // We haven't clicked to edit it or anything
 | 
				
			||||||
    {
 | 
					    {
 | 
				
			||||||
      on_evidence_double_clicked(current_evidence);
 | 
					      on_evidence_double_clicked(p_id);
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
    // Todo: make a function that compares two pieces of evidence for any
 | 
					    // Todo: make a function that compares two pieces of evidence for any
 | 
				
			||||||
    // differences
 | 
					    // differences
 | 
				
			||||||
@ -298,7 +299,7 @@ void Courtroom::set_evidence_list(QVector<evi_type> &p_evi_list)
 | 
				
			|||||||
        break;
 | 
					        break;
 | 
				
			||||||
      case QMessageBox::No:
 | 
					      case QMessageBox::No:
 | 
				
			||||||
        // "Discard changes and keep theirs"
 | 
					        // "Discard changes and keep theirs"
 | 
				
			||||||
        on_evidence_double_clicked(current_evidence);
 | 
					        on_evidence_double_clicked(p_id);
 | 
				
			||||||
        break;
 | 
					        break;
 | 
				
			||||||
      default:
 | 
					      default:
 | 
				
			||||||
        // should never be reached
 | 
					        // should never be reached
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
		Reference in New Issue
	
	Block a user