Trouble passing indexPath.row with prepareForSegue with UISearchBar

uisearchbaruistoryboardsegueuitableview

I'm having a dickens of a time getting this whole Core Data, Storyboard, UISearchBar trio working together as it should. Having finally successfully built the table with Core Data, narrow the items with a Search Text, and modified prepareForSegue, there is still one hiccup…

When I click on any item in the table to go to the detail view, all is fine in the unfiltered table. PrepareForSegue is called and the detail appears perfectly.

When I search, my table is filtered (I'm going to filtered array option for now instead of a second NSFetchedResultsController, but not for lack of trying!).

When I click on an item in the filtered list, prepareForSegue is called and the detail view is pushed, however, it always pulls the detail from the first item in the list!

For example, if I searched for "c" and the list was narrowed to "Charlie" and "Cookie", when I select "Charlie" I see the detail view for "Charlie". When I select "Cookie", I, unfortunately, also see the detail view for "Charlie"

I'm making the assumption that the prepareForSegue code is the issue (maybe incorrectly?). Here is the code:

    SampleTVC *sampleDetailTVC = segue.destinationViewController;
    sampleDetailTVC.delegate = self;

    // Store selected Role in selectedRole property
    NSIndexPath *indexPath = [self.tableView indexPathForSelectedRow];
  //  self.selectedRole = [self.fetchedResultsController objectAtIndexPath:indexPath];
    if (savedSearchTerm){
        self.selectedRole = [self.searchResults objectAtIndex:indexPath.row];
    } else {
        self.selectedRole = [self.fetchedResultsController objectAtIndexPath:indexPath];
    }
    NSLog(@"Passing selected role (%@) to SampleTVC", self.selectedRole.name);
    sampleDetailTVC.role = self.selectedRole;

Any help would be appreciated!

Best Answer

Thanks to Phillip Mills, for the answer:

just simply had to add:

    indexPath = [self.searchDisplayController.searchResultsTableView indexPathForSelectedRow];

full sample:

    SampleTVC *sampleDetailTVC = segue.destinationViewController;
    sampleDetailTVC.delegate = self;
     // Store selected Role in selectedRole property
    NSIndexPath *indexPath = [self.tableView indexPathForSelectedRow];
    //  self.selectedRole = [self.fetchedResultsController objectAtIndexPath:indexPath];
    if (savedSearchTerm){
         indexPath = [self.searchDisplayController.searchResultsTableView indexPathForSelectedRow];
    self.selectedRole = [self.searchResults objectAtIndex:indexPath.row];
    } else {
        self.selectedRole = [self.fetchedResultsController objectAtIndexPath:indexPath];
     }
    NSLog(@"Passing selected role (%@) to SampleTVC", self.selectedRole.name);
    sampleDetailTVC.role = self.selectedRole;

Related Solutions

Ios – How to pass prepareForSegue: an object

Simply grab a reference to the target view controller in prepareForSegue: method and pass any objects you need to there. Here's an example...

- (void)prepareForSegue:(UIStoryboardSegue *)segue sender:(id)sender
{
    // Make sure your segue name in storyboard is the same as this line
    if ([[segue identifier] isEqualToString:@"YOUR_SEGUE_NAME_HERE"])
    {
        // Get reference to the destination view controller
        YourViewController *vc = [segue destinationViewController];

        // Pass any objects to the view controller here, like...
        [vc setMyObjectHere:object];
    }
}

REVISION: You can also use performSegueWithIdentifier:sender: method to activate the transition to a new view based on a selection or button press.

For instance, consider I had two view controllers. The first contains three buttons and the second needs to know which of those buttons has been pressed before the transition. You could wire the buttons up to an IBAction in your code which uses performSegueWithIdentifier: method, like this...

// When any of my buttons are pressed, push the next view
- (IBAction)buttonPressed:(id)sender
{
    [self performSegueWithIdentifier:@"MySegue" sender:sender];
}

// This will get called too before the view appears
- (void)prepareForSegue:(UIStoryboardSegue *)segue sender:(id)sender
{
    if ([[segue identifier] isEqualToString:@"MySegue"]) {

        // Get destination view
        SecondView *vc = [segue destinationViewController];

        // Get button tag number (or do whatever you need to do here, based on your object
        NSInteger tagIndex = [(UIButton *)sender tag];

        // Pass the information to your destination view
        [vc setSelectedButton:tagIndex];
    }
}

EDIT: The demo application I originally attached is now six years old, so I've removed it to avoid any confusion.

Ios – indexPathForSelectedRow returning nil

A possible cause is that you've written something in one of the UITableViewDelegate methods -

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath{
}

It's a common practice to deselect the row which has just been selected.

Update:

In Swift 4.2, xcode 10:

Just comment the line

tableView.deselectRow(at: indexPath, animated: true)

in the function

func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath)

Best Answer

Related Solutions

Ios – How to pass prepareForSegue: an object

Ios – indexPathForSelectedRow returning nil

Related Topic