When HS is enabled on a vocabulary, and a content type uses that vocab to select the nodes taxonomy, the preview does not show when creating a node of that content type.

For example:
the vocab 'vehicles' is enables for a story. When I go to create a story (i do not select a category), I fill out title and body, and click preview. It reloads as though it is going to show a preview, but does not.

Comments

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Assigned: Unassigned » wim leers

That's a very interesting bug. I know where I'll have to look I think.

Good catch, thanks!

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

It seems the problem is the #after_build callback. This'll be very tricky to solve :(

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Title: 'Preview' wont show on content types with hs for catagories » Node preview functionality stops working
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Active » Fixed

Ok, it *was* a very small mistake after all! Phew :)

Fixed!

that0n3guy’s picture

that0n3guy commented
Status: Fixed » Active

It is still not working for me but maybe I'm updating wrong.

Here is what I did.
I removed the hs select directory from my modules directory.

I'm using drush with only "drush Package Manager CVS Support" enabled. I didn't enable wget because I didnt know if it would pull from the beta. I did /drush.php pm -v install hierarchical_select-5.x-3.x-dev.

It installed just fine. I did not disable HS in admin/build/modules. Do I need to?

Thanks

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Active » Fixed

Not correct. That's the dev tarball (which only gets updated every X hours, so it doesn't have the latest commits). You need CVS HEAD.

But just wait a couple more hours or wait for tomorrow: I'll release beta 2 later today.

that0n3guy’s picture

that0n3guy commented

Do you know if drush can pull from cvs head then? ( dont need the HS update right now, I just wanted to know )

Also, below is the output from my drush install.. it has HEAD written as the option (2nd line down). I assumed it was head because of that.

Downloading project hierarchical_select ...
Executing: cvs -z6  -d:pserver:anonymous:anonymous@cvs.drupal.org:/cvs/drupal-contrib checkout -d hierarchical_select -r HEAD contributions/modules/hierarchical_select
  cvs checkout: Updating hierarchical_select
  U hierarchical_select/API.txt
  U hierarchical_select/README.txt
  U hierarchical_select/UPGRADE.txt
  U hierarchical_select/hierarchical_select-rtl.css
  U hierarchical_select/hierarchical_select.admin.inc
  U hierarchical_select/hierarchical_select.css
  U hierarchical_select/hierarchical_select.info
  U hierarchical_select/hierarchical_select.install
  U hierarchical_select/hierarchical_select.js
  U hierarchical_select/hierarchical_select.module
  U hierarchical_select/hierarchical_select_cache.js
  U hierarchical_select/hierarchical_select_formtoarray.js
  cvs checkout: Updating hierarchical_select/includes
  U hierarchical_select/includes/common.inc
  U hierarchical_select/includes/common_config_form.css
  U hierarchical_select/includes/common_config_form.js
  U hierarchical_select/includes/views.js
  cvs checkout: Updating hierarchical_select/js
  cvs checkout: Updating hierarchical_select/js/jquery.ui
  U hierarchical_select/js/jquery.ui/effects.core.js
  U hierarchical_select/js/jquery.ui/effects.drop.js
  cvs checkout: Updating hierarchical_select/modules
  U hierarchical_select/modules/hs_content_taxonomy.css
  U hierarchical_select/modules/hs_content_taxonomy.info
  U hierarchical_select/modules/hs_content_taxonomy.module
  U hierarchical_select/modules/hs_content_taxonomy_views.info
  U hierarchical_select/modules/hs_content_taxonomy_views.module
  U hierarchical_select/modules/hs_menu.info
  U hierarchical_select/modules/hs_menu.module
  U hierarchical_select/modules/hs_subscriptions_taxonomy.info
  U hierarchical_select/modules/hs_subscriptions_taxonomy.module
  U hierarchical_select/modules/hs_taxonomy.info
  U hierarchical_select/modules/hs_taxonomy.js
  U hierarchical_select/modules/hs_taxonomy.module
  U hierarchical_select/modules/hs_taxonomy_views.info
  U hierarchical_select/modules/hs_taxonomy_views.module
  U hierarchical_select/modules/taxonomy.js
  cvs checkout: Updating hierarchical_select/po
  U hierarchical_select/po/he.po
  U hierarchical_select/po/hierarchical_select.pot
Checking out hierarchical_select was successful.
Project hierarchical_select successfully installed (version 5.x-3.x-dev).
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Fixed » Active

That *is* CVS HEAD indeed. It was this line that made me think it was the tarball:
Project hierarchical_select successfully installed (version 5.x-3.x-dev).
So that's just bad behaviour of Drush.

Could you please go into the hierarchical_select.module file and copy/paste the hierarchical_select_form_alter() function? That's the only function that had to be changed to fix this.

that0n3guy’s picture

that0n3guy commented
Status: Active » Fixed

I figured out my problem... I did something I didn't think about being bad.

I had made a backup of my HS directory (just in case something went bad) but had it in the modules folder. Drupal was looking at it for some reason rather than the real HS folder.

It works now that I moved the backup folder away from the 'modules' directory.

Stupid mistake, sorry I wasted your time.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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