J4 Issue ?
avatar infograf768
infograf768
27 Oct 2018

Steps to reproduce the issue

Install a clean 4.0 dev. ( npm install done)

Go to media manager
index.php?option=com_media

Expected result

Manager should display.
Same when using the Image xtd.

Actual result

screen shot 2018-10-27 at 07 23 49

ReferenceError: MediaManager is not defined

System information (as much as possible)

// Add javascripts
HTMLHelper::_('script', 'media/com_media/js/mediamanager.js');

// Add stylesheets
HTMLHelper::_('stylesheet', 'media/com_media/css/mediamanager.css');

should load these files. But they do not exist, even after a npm install.

avatar infograf768 infograf768 - open - 27 Oct 2018
avatar joomla-cms-bot joomla-cms-bot - change - 27 Oct 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 27 Oct 2018
avatar infograf768 infograf768 - change - 27 Oct 2018
Labels Added: J4 Issue
avatar infograf768 infograf768 - labeled - 27 Oct 2018
avatar Fedik
Fedik - comment - 27 Oct 2018

hm, I just tried with latest version of 4.0-dev, and all works fine for me
same for #22830

avatar infograf768
infograf768 - comment - 27 Oct 2018

Have you tried with nightly build or a 4.0 checkout?

avatar brianteeman
brianteeman - comment - 27 Oct 2018

Make sure there were no errors with the npm install. Sounds like the same problem described with the nightly yesterday.

avatar Fedik
Fedik - comment - 27 Oct 2018

I have tried with latest 4.0-dev branch

avatar infograf768
infograf768 - comment - 27 Oct 2018

Here I have errors for com_media

> node-sass@4.9.3 install /Applications/MAMP/htdocs/testnew/joomla40/administrator/components/com_media/node_modules/node-sass
> node scripts/install.js

Cached binary found at /Users/mac/.npm/node-sass/4.9.3/darwin-x64-57_binding.node

> node-sass@4.9.3 postinstall /Applications/MAMP/htdocs/testnew/joomla40/administrator/components/com_media/node_modules/node-sass
> node scripts/build.js

Binary found at /Applications/MAMP/htdocs/testnew/joomla40/administrator/components/com_media/node_modules/node-sass/vendor/darwin-x64-57/binding.node
Testing binary
Binary is fine
npm WARN ajv-keywords@3.2.0 requires a peer of ajv@^6.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN babel-loader@6.4.1 requires a peer of webpack@1 || 2 || ^2.1.0-beta || ^2.2.0-rc but none is installed. You must install peer dependencies yourself.
npm WARN sass-loader@5.0.1 requires a peer of webpack@^2.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN com_media@4.0.0 No repository field.

removed 23 packages, updated 18 packages and audited 8538 packages in 4.884s
found 0 vulnerabilities


> com_media@4.0.0 build /Applications/MAMP/htdocs/testnew/joomla40/administrator/components/com_media
> cross-env NODE_ENV=production webpack --progress --hide-modules

sh: /Applications/MAMP/htdocs/testnew/joomla40/administrator/components/com_media/node_modules/.bin/cross-env: Permission denied
npm ERR! code ELIFECYCLE
npm ERR! errno 126
npm ERR! com_media@4.0.0 build: `cross-env NODE_ENV=production webpack --progress --hide-modules`
npm ERR! Exit status 126
npm ERR! 
npm ERR! Failed at the com_media@4.0.0 build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

CSS for cassiopea is definitely broken: see nightly build where they are not present either.

avatar Fedik
Fedik - comment - 27 Oct 2018

hm, maybe try remove (or rename) manually:
/administrator/components/com_media/node_modules
and
/node_modules

and run "npm install" again , they will be recreated

avatar infograf768
infograf768 - comment - 27 Oct 2018

removing node_modules for com_media solved it for com_media. It is frustrating as it was a clean install. I should not have to delete the node_modules folder.

It does not solve cassiopea absent css
See #22824

Closing this PR and leaving the one for Cassiopea.

avatar infograf768 infograf768 - change - 27 Oct 2018
Status New Closed
Closed_Date 0000-00-00 00:00:00 2018-10-27 09:22:11
Closed_By infograf768
avatar infograf768 infograf768 - close - 27 Oct 2018

Add a Comment

Login with GitHub to post a comment