FAPI Term is misleading. There is no FAPI in Drupal, so this name sound confusing. We must rename this module: form_api_example, so as to match with common name used in other places of Drupal Ecosystem.
Will submit the patch by this weekend after testing.

CommentFileSizeAuthor
#5 2804361_5.patch34.48 KBMile23
#2 rename-form-api-2804361-02.patch103.05 KBwebankit
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webankit created an issue. See original summary.

webankit’s picture

webankit CreditAttribution: webankit as a volunteer commented
FileSize
rename-form-api-2804361-02.patch103.05 KB

Renamed the fapi directory and all files inside the directory.

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 as a volunteer commented

Good idea, since FAPI is also a bit euphemistic.

There are some outstanding issues with patches, so let's do this after those.

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 as a volunteer commented
Assigned: webankit » Unassigned
Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 as a volunteer commented
Status: Active » Needs review
FileSize
2804361_5.patch34.48 KB

Updated after all the work done on the fapi_example.

We'll need to change the component name, too.

  • Mile23 committed 157b87b on 8.x-1.x authored by webankit 
    Issue #2804361 by webankit, Mile23: fapi_example must be renamed to...
Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 as a volunteer commented
Component: FAPI Example » Form API Example
Status: Needs review » Fixed

Committed. Also changed the FAPI Example component to Form API Example.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.