Project

General

Profile

Refactor #2845

Command's output definition shouldn't use formatting functions

Added by Tomáš Strachota almost 7 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

Description

Formatting functions should be replaced with field type. The function spoils field's information forever. Eg. once datetime is formatted it can't be printed in different format. More flexible approach would be to set field type (to DateTimeField ind this case) and move the formatting logic to output adapters. The adapter then decides how it shoud be formatted. Eg. Json adapter prints more structured info vs. Table adapter tends to shorten things.

History

#1 Updated by Martin Bacovsky almost 7 years ago

  • Project changed from Hammer CLI to Foreman
  • Category changed from Hammer core to 44

#2 Updated by Tomáš Strachota almost 7 years ago

  • Target version set to 1.3.0

#3 Updated by Tomáš Strachota almost 7 years ago

  • Status changed from New to Assigned

#4 Updated by Tomáš Strachota almost 7 years ago

  • Status changed from Assigned to Closed

#5 Updated by Dominic Cleal over 6 years ago

  • Project changed from Foreman to Hammer CLI
  • Category deleted (44)

Also available in: Atom PDF