Pull Request for Issue # .

Summary of Changes

This PR improves the code of contact view of com_contact component (mostly relate to handle breadcrumb). The code which we are having right now is a mess, hard to understand and difficult to maintain:

• We are using string comparison to check if the current active menu item is linked to com_contact
• Duplicate to calculate params for the contact item
• Bad code to handle breadcrumb

The new code is base on the current code which we are having in view article in com_content. It is much easier to understand compare to the original code.

Testing Instructions

1. Create two contact categories. Add 2 or 3 contacts to each category (You can use Save As Copy button to save time for creating multiple contacts)
2. Create menu items to the following menu item types of Contacts component:

• List All Categories in a Contact Category Tree
• List Contacts in a Category
• Single Contact

3. Apply patch
4. Access to above menu items, then try to access to contact, make sure breadcrumb is handled properly

• For contact accessed base on the first two menu item types, the breadcrumb displayed in the breadcrumb has this format:
  Home / Active Menu Item / [Sub Category] / Contact Name

• For contact accessed from Single Contact menu item type, the breadcrumb should have this format:
  Home / Active Menu Item

Actual result BEFORE applying this Pull Request

Works but code looks bad.

Expected result AFTER applying this Pull Request

Works and code is cleaner, easier to understand.