Skip to content

Correct the location edit back-link label - #2949

Open
RabbiIslamRony wants to merge 1 commit into
sovware:developmentfrom
RabbiIslamRony:fix/961-location-back-link-label
Open

Correct the location edit back-link label#2949
RabbiIslamRony wants to merge 1 commit into
sovware:developmentfrom
RabbiIslamRony:fix/961-location-back-link-label

Conversation

@RabbiIslamRony

Copy link
Copy Markdown
Member

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Security fix
  • Improvement
  • New Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Text changes
  • Other... Please describe:

Description

How to reproduce the issue or how to test the changes

  1. Go to Directorist > Locations and open an existing location from the taxonomy list.
  2. Click Update without changing the location.
  3. Confirm the success notice reads Location updated. Go to Locations rather than using the Categories fallback.
  4. Click Go to Locations and confirm it returns to the Directorist location taxonomy list for the listing post type.
  5. Open and update a Directorist category and confirm its existing category-specific label remains unchanged.

Verification completed:

  • The exact LocalWP list → edit → update path displayed Go to Locations.
  • The rendered link returned to edit-tags.php?taxonomy=at_biz_dir-location&post_type=at_biz_dir.
  • The category taxonomy continued to expose its existing Go to Categories label.
  • PHP syntax, targeted PHPCS, and git diff --check passed.

Any linked issues

Fixes #961

Checklist

@RabbiIslamRony

Copy link
Copy Markdown
Member Author

The reported PHPCS failure occurs in the inherited pull_request_target workflow before source checkout; the PHPCS step never runs. PR #2945 replaces that unsafe workflow with the fork-safe pull_request workflow. The changed PHP file passes the repository rules locally with zero errors.

@Armanul46 Armanul46 added this to the v8.9.3 milestone Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants