Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

event admin_news_update is not triggered after inline updating #4183

Closed
Moc opened this issue Jun 1, 2020 · 0 comments
Closed

event admin_news_update is not triggered after inline updating #4183

Moc opened this issue Jun 1, 2020 · 0 comments
Assignees
@CaMer0n
Labels
core: Admin-UI status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. type: bug A problem that should not be happening
Milestone
e107 2.3.0

Comments

@Moc
Copy link
Member

Moc commented Jun 1, 2020 •
edited
Loading

Bug Description

event admin_news_update is not triggered after inline updating.

  • The event is triggered when editing the news item by clicking the edit pencil and changing the fields, and then hitting the update button.
  • It does not work when INLINE editing the news title.

I have not tested if this is limited to news area only or if this happens for all inline editing using the Admin UI

How to Reproduce

Steps to reproduce the behavior:

  1. Inline edit news item (e.g. changing news title)
  2. No event is triggered.

Expected Behavior

event admin_news_update should be triggered when inline editing a news post.

Server Information

PHP Operating System

PHP 7.4
@Moc Moc added type: bug A problem that should not be happening status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. core: Admin-UI labels Jun 1, 2020
@Moc Moc added this to the Future milestone Jun 1, 2020
@CaMer0n CaMer0n self-assigned this Jun 1, 2020
@CaMer0n CaMer0n closed this as completed in 08260e8 Jun 1, 2020
@Moc Moc modified the milestones: Future, e107 2.3.0 Jun 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CaMer0n CaMer0n

Labels
core: Admin-UI status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. type: bug A problem that should not be happening
Projects
None yet
Milestone
e107 2.3.0
Development

No branches or pull requests
2 participants
@Moc @CaMer0n