Page MenuHomePhabricator

Fix audit commenting where there are only inlines
ClosedPublic

Authored by btrahan on Jul 30 2014, 5:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 12 2024, 12:10 PM
Unknown Object (File)
Feb 12 2024, 12:10 PM
Unknown Object (File)
Feb 5 2024, 7:43 PM
Unknown Object (File)
Nov 30 2023, 2:18 AM
Unknown Object (File)
Nov 26 2023, 1:26 PM
Unknown Object (File)
Nov 15 2023, 11:38 PM
Unknown Object (File)
Oct 30 2023, 9:35 PM
Unknown Object (File)
Oct 27 2023, 12:26 PM
Subscribers

Details

Summary

been some changes here and this code was broked. turns out we re-assign $action like two lines later and never used the initial value, so we can simply delete the offending line. Fixes T5745.

Test Plan

submitted inline comment pre-patch and fatal. re-submitted post patch and great success!

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan retitled this revision from to Fix audit commenting where there are only inlines.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.

We swapped to a different datasource for the "which things are native in PHP", it might be missing ZipArchive.

This revision is now accepted and ready to land.Jul 30 2014, 5:46 PM
btrahan updated this revision to Diff 24236.

Closed by commit rP98c2ad38e564 (authored by @btrahan).