AddClientAssets isn't loaded?

Toby

BinhQuang Could it be an issue of case-sensitivity? i.e. the namespace in your classes is connor0095\imgurupload but Composer is only autoloading for Connor0095\Imgurupload?

Davis

Toby This is my composer.json

{
    "name": "connor0095/flarum-imgurupload",
    "description": "Seamlessly upload and embed Imgur images",
    "type": "flarum-extension",
    "keywords": ["formatting"],
    "license": "MIT",
    "authors": [
        {
            "name": "Connor Davis",
            "email": "cdavis@produes.co"
        }
    ],
    "support": {
        "issues": "https://github.com/connor0095/flarum-imgur-upload/issues",
        "source": "https://github.com/connor0095/flarum-imgur-upload"
    },
    "require": {
        "flarum/core": "^0.1.0-beta.3"
    },
    "extra": {
        "flarum-extension": {
            "title": "Imgur Upload",
            "icon": {
                "image": "icon.jpg",
                    "backgroundColor": "#1C1C1C",
                    "backgroundSize": "50%",
                    "backgroundRepeat": "no-repeat",
                    "backgroundPosition": "center"
            } 
        }
    }
}

I have the name set with 'flarum-' beacause the quickstart guide told me to do so.

"name is the name of the Composer package in the format vendor/extension. You should prefix the extension part with flarum- to indicate that it’s a package specifically intended for use with Flarum."

Also, my version doesn't show up inside the admin panel.

Toby

(Oops! My first reply wasn't very useful because I confused @BinhQuang with the OP...)

Do you have franzliedke/studio set up, or a Composer path repository? In order for the root Composer installation to know about your extension's autoloading rules, you'll need to get one of them working.

Davis Also, my version doesn't show up inside the admin panel.

The versions displayed in the admin panel reflect the package version that has been installed via Composer/Packagist. No version is displayed for in-development extensions.

luceos

{
    "name": "connor0095/flarum-imgurupload",
    "description": "Seamlessly upload and embed Imgur images",
    "type": "flarum-extension",
    "keywords": ["formatting"],
    "license": "MIT",
    "authors": [
        {
            "name": "Connor Davis",
            "email": "cdavis@produes.co"
        }
    ],
    "autoload": {
        "psr4": [
            "connor0095\\imgurupload\\": "src/"
        ]
    },
    "support": {
        "issues": "https://github.com/connor0095/flarum-imgur-upload/issues",
        "source": "https://github.com/connor0095/flarum-imgur-upload"
    },
    "require": {
        "flarum/core": "^0.1.0-beta.3"
    },
    "extra": {
        "flarum-extension": {
            "title": "Imgur Upload",
            "icon": {
                "image": "icon.jpg",
                    "backgroundColor": "#1C1C1C",
                    "backgroundSize": "50%",
                    "backgroundRepeat": "no-repeat",
                    "backgroundPosition": "center"
            } 
        }
    }
}

the above should work if you used psr4 standards, this means the directory Listeners is in src/Listeners. After that run a

composer dumpautoload

Davis

luceos I changed it to this because it was throwing syntax errors with
composer dumpautoload
The dumpautoload goes through seamlessly but it still throws the error in the browser.

{
    "name": "connor0095/flarum-imgurupload",
    "description": "Seamlessly upload and embed Imgur images",
    "type": "flarum-extension",
    "keywords": ["formatting"],
    "license": "MIT",
    "authors": [
        {
            "name": "Connor Davis",
            "email": "cdavis@produes.co"
        }
    ],
    "support": {
        "issues": "https://github.com/connor0095/flarum-imgur-upload/issues",
        "source": "https://github.com/connor0095/flarum-imgur-upload"
    },
    "require": {
        "flarum/core": "^0.1.0-beta.3"
    },
    "autoload": {
        "psr-4": {
            "connor0095\\imgurupload\\": "src/"
        }
    },
    "extra": {
        "flarum-extension": {
            "title": "Imgur Upload",
            "icon": {
                "image": "icon.jpg",
                    "backgroundColor": "#1C1C1C",
                    "backgroundSize": "50%",
                    "backgroundRepeat": "no-repeat",
                    "backgroundPosition": "center"
            } 
        }
    }
}

luceos

Davis What's your path structure? Where is your Listener?

Davis

luceos This is everything, capitalization and all:

extensions/ -->
    connor0095-imgurupload/ -->
        js/ -->
            forum/ -->
                dist/ -->
                    extension.js
                node_modules/ -->
                    *omitted*
                src/ -->
                    main.js
                Gulpfile.js
                package.json
        src/ -->
            Listener/ -->
                AddClientAssets.php
        bootstrap.php
        composer.json
        icon.jpg
        LICENSE

luceos

Davis do you have a public repository? Also Extensions is not the default name for the extensions folder..

Davis

luceos Yeah. I just committed the changes.

https://github.com/connor0095/flarum-imgur-upload

luceos Also Extensions is not the default name for the extensions folder..

My bad, it isn't capitalized on my server.

luceos

Your namespace doesn't match the namespace you defined in composer.json:

connor0095\imgurupload vs Connor0095\Imgurupload. Then re-run composer dumpautoload.

Would have loved to sent in your first PR, but can't until tomorrow.

The Listener should have the namespace:

namespace Connor0095\Imgurupload\Listeners;

Davis

luceos It didn't work when they were matching. I tried changing them to capitols and it didn't fix it. I will fix that, but I'm sure that's not the issue.

I fixed the issue. Hopefully someone adds a tutorial of how to add extensions properly soon.

luceos

Davis we'll put it on the agenda. Documentation is about to be rewritten anyway.

thefirefox

I have the same problem, how did you solve this issue?

Davis

thefirefox

https://discuss.flarum.org/d/1608-extension-development-using-composer-repositories-path

thefirefox

Thanks for your fast reply.

Davis

thefirefox I have my email set up on my phone ?