Page MenuHomePhabricator

Make getHeader, getBody and getFooter members public on PhutilConsoleTable
AbandonedPublic

Authored by hach-que on Jan 8 2015, 7:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 11:51 PM
Unknown Object (File)
Sat, Apr 20, 4:11 AM
Unknown Object (File)
Thu, Apr 11, 8:06 AM
Unknown Object (File)
Tue, Apr 2, 8:43 PM
Unknown Object (File)
Sat, Mar 30, 1:16 AM
Unknown Object (File)
Mar 5 2024, 6:51 PM
Unknown Object (File)
Jan 20 2024, 11:40 PM
Unknown Object (File)
Jan 19 2024, 12:36 PM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

This makes the getHeader, getBody and getFooter members public on PhutilConsoleTable instead of protected.

In Omni, we route all of the standard output from builtins through pipes, not the standard output normally obtained using echo or any of phutil's console functionality. As such, when using the PhutilConsoleTable class I need some way of getting the strings that make up the table instead of just calling draw.

Test Plan

Currently used within Omni to access those members.

Diff Detail

Repository
rPHU libphutil
Branch
console-table-public
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3674
Build 3684: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

hach-que retitled this revision from to Make getHeader, getBody and getFooter members public on PhutilConsoleTable.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
epriestley edited edge metadata.

I'll accept a render() or something which draw() calls internally. This change doesn't seem like a good fit for the use case (which does not require separate access to the header, body and footer).

This revision now requires changes to proceed.Aug 8 2015, 6:38 PM

I think this was probably obsoleted or otherwise affected by D14136.