Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

searchController::spider() crashes page

#283

When the site is spider'd (content migration, etc...) it'll crash because $available_controllers contains both controllers and models. I've partially fixed this by 1) fixing a newly introduced typo in the method, and 2) adding an isSearchable() method to expRecord, but still can't get through it completely without crashing.

Reported by dleffler · August 6th, 2011 @ 12:45 AM

State: resolved
Milestone: Release Candidate 2
Assigned to: dleffler dleffler

Activity

  1. expNinja
    expNinja
    • State changed from open to resolved

    (from [f7e3410ee80be734e96a6976fc4a5e0adcfc4ae2]) spider searches $available_controllers which now also contains models stored in the modules folders. Crashed because there was no isSearchable() method, so I added it to expRecord. Also the Twitter model was extended to use expRecord (not sure if this will cause other problems, if so, just add an isSearchable() method). Also a disconnect in textController. Is says it is Searchable, but set to return false when getting spider content (made an assumption that search operated like in 1.0). I commented out the addContentToSearch() method, so it works per expRecord. If this is not desired, uncomment it out and change isSearchable() to return false. Also corrected a recent typo in the exponent_modules_list() function. [#283 searchController::spider() crashes page state:resolved] https://github.com/exponentcms/exponent-cms/commit/f7e3410ee80be734...

    August 6th, 2011 @ 01:21 AM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.