Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-4463

PHP generated code match PSR-2

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 0.11.0
    • 0.12.0
    • PHP - Compiler
    • None

    Description

      When use thrift compiler to generate PHP code, there are many issues doesn't match PSR-2

      • indent by 2 spaces, PSR-2 require 4 spaces.
      • Opening brace should be on a new line (for function)
      • The static declaration must come after the visibility declaration
      • etc.

       

      So, we can make generated code more readable, match PSR-2(almost).

       

       

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Robert Lu Robert Lu
            Robert Lu Robert Lu
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment