Review of "ESP (Extensions Search Provider)" version 50.0 (27)

Details Page Preview

Search, access settings and data and control installed extensions from Activities overview. Features: - (De)activate and delete extensions - Global and isolated search controlled by custom prefixes - Sorting options includes enabled first and order of enabling - Direct access to extension's directory, metadata file, dconf and website - Status information including compatibility, updates and errors

Extension Homepage
https://github.com/G-dH/extensions-search-provider

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental) warning 2 manual_review 2

Shexli found 4 issues that may need reviewer attention.

EGO-A-001 manual_review

extension code must not be minified or obfuscated

File appears obfuscated: 56% of identifiers are 1–2 characters (avg length 5.9).

Code must not be obfuscated

  • extension.js
    short identifier ratio: 56%, identifiers scored: 25

EGO-A-005 manual_review

extensions should not access the clipboard directly

Direct clipboard access via `St.Clipboard.get_default()` requires reviewer scrutiny.

Review Guidelines

  • listSearchResult.js:440
    St.Clipboard.get_default()
  • listSearchResult.js:441
    St.Clipboard.get_default()
  • listSearchResult.js:472
    St.Clipboard.get_default()
  • listSearchResult.js:473
    St.Clipboard.get_default()
  • listSearchResult.js:607
    St.Clipboard.get_default()

EGO-L-002 warning

objects created by extension should be destroyed in disable()

Objects assigned in `enable()` are missing matching `.destroy()` calls in `disable()` or its helper methods.

Destroy all objects

  • dashIcon.js:154
            this.toggleButton = new St.Button({
                style_class: 'show-apps',
                track_hover: true,
                can_focus: true,
                toggle_mode: false,
            })

EGO-L-005 warning

owned object references should be released in disable()

Owned references that are cleaned up in `disable()` should also be released with `null` or `undefined`.

Destroy all objects

  • dashIcon.js:154
            this.toggleButton = new St.Button({
                style_class: 'show-apps',
                track_hover: true,
                can_focus: true,
                toggle_mode: false,
            })

All Versions

Version Status
50.0 (27) Active
49.0 (26) Active
48.0 (25) Active
47.2 (24) Active
44.7 (23) Active
44.7 (22) Inactive
47.2 (21) Inactive
47.1 (20) Inactive
47.0 (19) Inactive
44.6 (18) Active
46.6 (17) Inactive
44.6 (16) Rejected
46.6 (15) Rejected
44.5 (14) Inactive
46.5 (13) Inactive
44.4 (12) Inactive
46.4 (11) Inactive
44.3 (10) Inactive
46.3 (9) Inactive
46.2 (8) Rejected
44.2 (7) Inactive
44.2 (6) Inactive
46.2 (5) Inactive
44.1 (4) Inactive
46.1 (3) Inactive
46.1 (2) Inactive
46.1 (1) Rejected

Previous Reviews on this Version

fmuellner active