Problem/Motivation

Over in #1396892: drupal_get_library() may break JavaScript library path by clobbering it with the library version number we added a shorthand.js to test adding JavaScript files to pages using the shorthand form of drupal_add_js().
But hook_library_info() and drupal_add_js() are now redundant, and the test coverage it was added for is also gone.
So we should remove the empty js file.

Proposed resolution

Remove it

Remaining tasks

Review

User interface changes

None

API changes

None

CommentFileSizeAuthor
shorthand-plugin-not-needed.patch615 byteslarowlan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

LinL’s picture

LinL CreditAttribution: LinL commented
Status: Needs review » Reviewed & tested by the community

Couldn't find any other reference to shorthand.js, RTBC.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed 0fb70c7 and pushed to 8.x. Thanks!

  • alexpott committed 0fb70c7 on 8.x 
    Issue #2241165 by larowlan: Fixed Remove redundant shorthand.js file.

Status: Fixed » Closed (fixed)

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