I'm forced to set "Slide default Music" path. It shouldn't

xn2001 - October 28, 2009 - 19:41
Project:album photos
Version:6.x-2.6-beta3
Component:User interface
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

I'm forced to set "Slide default Music" path. It shouldn't since I'm not using dfgallery module.

In photos/inc/photos.admin.inc, $form['slideshow']['photos_slide_music'] should be changed to:

  // "Slide default Music" will only be required if the dfgallery module is enabled.
  $form['slideshow']['photos_slide_music'] = array(
    '#type' => 'textfield',
    '#default_value' => variable_get('photos_slide_music', false),
    '#title' => t('Slide default Music'),
    '#required' => (module_exists('dfgallery') ? true: false),
    '#description' => t('e.g: http://mysite/music.mp3'),
    '#disabled' => (module_exists('dfgallery') ? false: true)
  );

#1

genfoch01 - October 28, 2009 - 21:54

why is it required at all?

#2

xn2001 - October 28, 2009 - 23:47

That is a good question.
I don't know. I'm not currently using dfgallery module.

#3

genfoch01 - October 29, 2009 - 01:17

I cant get it to work (photo not df gal)... it will let you create an album but when I click on view album all I get is a blank page.
sigh.

#4

xn2001 - October 31, 2009 - 04:28

In photos/inc/photos.admin.inc, $form['slideshow']['photos_slide'] should be changed to:

<?php
    $form
['slideshow']['photos_slide'] = array(
       
'#type' => 'radios',
       
'#title' => t('Slide show'),
       
'#default_value' => (module_exists('dfgallery') ? 1: 0), //variable_get('photos_slide', 1),
       
'#description' => (!module_exists('dfgallery') ? t('Please enable <a href="@url">dfgallery</a> module',array('@url' => url('http://drupal.org/project/dfgallery'))): ''),
       
'#options' => array(t('Disabled'), t('Enabled')),
   
'#disabled' => (module_exists('dfgallery') ? false: true)
    );
?>

By default, #default_value should be Disabled when dfgallery module is not enabled.

#5

genfoch01 - October 31, 2009 - 04:35

I have df gal enabled and when I click on slide show it works... but view album just gives a blank page. I'll turn off df gal, make the above code change and let you know the results.

Thank you so much for responding, none of my other posts have been replied to.

#6

genfoch01 - October 31, 2009 - 16:41

ok, I made the code change above and turned off df gal. as expected the slideshow link no longer works. I just got a blank page. but now I wonder since I am getting a blank page on view album is there some other dependency I am missing? something album photos needs to provide this function?

#7

xn2001 - October 31, 2009 - 18:23

@genfoch01
Please stop hijacking this issue. This is related to the settings of the "Slide show". I wasn't even addressing to you in my previous post(#4).

Now, for your issue, I'm not qualified to answer as I was testing out this module a few days ago. I saw that you created an issue #617446: view album gives blank page about your problem but your description is not specific (Define what is 'view album'. Steps by steps are the best way to go). No 1 will be able to help if you are not specific.

#8

genfoch01 - October 31, 2009 - 20:17

I was not trying to hijack anything. Since the slide show default music setting appears to be something you must set on photo album I thought they were kind of related. that is issues regarding photo. my apologies.

#9

zenrei - November 21, 2009 - 04:52

I tried the above 2 edits, but I'm still required to enter default music even when I disabled slideshow.

 
 

Drupal is a registered trademark of Dries Buytaert.