Strider-CD / strider

Open Source Continuous Integration & Deployment Server
http://strider-cd.github.io/
4.59k stars 432 forks source link

"Cannot read property 'childNodes' of undefined" in project settings page #456

Closed terion-name closed 10 years ago

terion-name commented 10 years ago

When entering project settings page:

TypeError: Cannot read property 'childNodes' of undefined
    at compositeLinkFn (http://deploy.example.com:3000/javascripts/lib/angular.js:5764:36)
    at compositeLinkFn (http://deploy.example.com:3000/javascripts/lib/angular.js:5764:13)
    at nodeLinkFn (http://deploy.example.com:3000/javascripts/lib/angular.js:6344:24)
    at compositeLinkFn (http://deploy.example.com:3000/javascripts/lib/angular.js:5761:15)
    at compositeLinkFn (http://deploy.example.com:3000/javascripts/lib/angular.js:5764:13)
    at publicLinkFn (http://deploy.example.com:3000/javascripts/lib/angular.js:5666:30)
    at boundTranscludeFn (http://deploy.example.com:3000/javascripts/lib/angular.js:5780:21)
    at Object.controllersBoundTransclude [as transclude] (http://deploy.example.com:3000/javascripts/lib/angular.js:6371:18)
    at http://deploy.example.com:3000/javascripts/lib/angular.js:20355:32
    at Array.forEach (native) 

Even in fresh project without plugins added.

Mostly doesn't affect anything but maybe this causes some plugins failure (https://github.com/Strider-CD/strider-ssh-deploy/issues/4)

kfatehi commented 10 years ago

I think you have a plugin installed that has regressed... I recognize this error. Please see https://github.com/Strider-CD/strider/issues/437

kfatehi commented 10 years ago

@niallo perhaps terion installed over npm or docker or one of these other ones which did not include our patch removing webhook and sauce plugins from the default list? @terion-name how did you install strider?

kfatehi commented 10 years ago

Also @terion-name please ls node_modules and tell us which plugins exist there

terion-name commented 10 years ago
npm install -g strider

plugins: gitlab, custom scripts, ssh deploy

terion-name commented 10 years ago

ls -1 node_modules/ ansiparse async bcrypt connect connect-flash connect-mongo connect-slashes consolidate cookie cors eventemitter2 everypaas express express-forgot-password express-namespace gravatar httpcheck jade lodash loggly mailer moment mongoose nodemailer nomnom passport passport-local pw rc request socket.io ssh-keypair step strider-bitbucket strider-build-badge strider-custom strider-email-notifier strider-env strider-extension-loader strider-git strider-github strider-gitlab strider-heroku strider-mailer strider-node strider-python strider-ruby strider-sauce strider-simple-runner strider-ssh-deploy strider-webhooks stylus superagent swig underscore validator winston

kfatehi commented 10 years ago

Remove strider-sauce and strider-webhooks and that error will go away and the config pages will start working again.

On Jul 11, 2014, at 4:10 AM, Terion notifications@github.com wrote:

ls -1 node_modules/ ansiparse async bcrypt connect connect-flash connect-mongo connect-slashes consolidate cookie cors eventemitter2 everypaas express express-forgot-password express-namespace gravatar httpcheck jade lodash loggly mailer moment mongoose nodemailer nomnom passport passport-local pw rc request socket.io ssh-keypair step strider-bitbucket strider-build-badge strider-custom strider-email-notifier strider-env strider-extension-loader strider-git strider-github strider-gitlab strider-heroku strider-mailer strider-node strider-python strider-ruby strider-sauce strider-simple-runner strider-ssh-deploy strider-webhooks stylus superagent swig underscore validator winston

— Reply to this email directly or view it on GitHub.

niallo commented 10 years ago

I guess it's time to cut a new release and push to npm :)

On Fri, Jul 11, 2014 at 9:25 AM, Keyvan Fatehi notifications@github.com wrote:

Remove strider-sauce and strider-webhooks and that error will go away and the config pages will start working again.

On Jul 11, 2014, at 4:10 AM, Terion notifications@github.com wrote:

ls -1 node_modules/ ansiparse async bcrypt connect connect-flash connect-mongo connect-slashes consolidate cookie cors eventemitter2 everypaas express express-forgot-password express-namespace gravatar httpcheck jade lodash loggly mailer moment mongoose nodemailer nomnom passport passport-local pw rc request socket.io ssh-keypair step strider-bitbucket strider-build-badge strider-custom strider-email-notifier strider-env strider-extension-loader strider-git strider-github strider-gitlab strider-heroku strider-mailer strider-node strider-python strider-ruby strider-sauce strider-simple-runner strider-ssh-deploy strider-webhooks stylus superagent swig underscore validator winston

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/Strider-CD/strider/issues/456#issuecomment-48751887.

Niall O'Higgins W: http://niallohiggins.com E: n@niallo.me T: @niallohiggins

terion-name commented 10 years ago

@keyvanfatehi yeap, that helped. Thanx)

knownasilya commented 10 years ago

This should be fixed in master.