hey

can you port it to D7 ? :P

CommentFileSizeAuthor
#14 splash.zip11.74 KBtommychris
#9 splash-7.zip16.24 KBchrisroane

Comments

barwonhack’s picture

Subscribing.

Would be awesome to get this for D7.

:)

bryancasler’s picture

subscribe

elgandoz’s picture

subbing too. sympa. better with colorbox integration

toxiclung’s picture

+1

basicmagic.net’s picture

subscribe

astutonet’s picture

I'm interested in that

sw3b’s picture

Title: D7 port » Port Splash to D7

Rename for easy reading in dashboard and subscribing !

Fidelix’s picture

Subscribing...

chrisroane’s picture

StatusFileSize
new16.24 KB

I went through the D6 version of this module, and updated the code so that it works in D7. I've already tested this code on multiple sites, and we thought we would share it with the community.

I haven't really done this before, so I'm not sure on the exact process for this. I've submitted a patch for a different module, but I wasn't sure I needed to create a patch with a new version port?

Here is what I did with the code:

- The popup now uses the jQuery Colorbox module. You will want to install the drupal colorbox module for this to work right.

- The admin interface is exactly like it was before. Same options.

Let me know what you think and if I should submit the code differently.

djg_tram’s picture

Thanks, that saves some work, I was looking for the same... :-)

I'd have a suggestion, though. In D7, it's not really polite to overcrowd the admin menu on the top, especially with single menu items like Splash. So, you should move the settings pages to eg. admin/config/user-interface/splash and make sure you also have a configure entry in the .info file pointing to the same.

lvdekken’s picture

Hi, installed the D7 code as posted in #9. how can I access the userinterface. Does not show up in modules-page. thx

djg_tram’s picture

It goes to the black top admin menu. It shouldn't, hence my previous post... :-)

Gaelan’s picture

Colorbox shouldn't be a dependency as it is optional. The radio button should be only shown if the module is available.

tommychris’s picture

StatusFileSize
new11.74 KB

I have created a v2 with a little modification:
- Moved the settings page to admin/config/user-interface/splash
and
- fixed the permission function, D6 uses hook_perm(), D7 uses hook_permission()
and
- added a new permission: Enable Splash - My goal is to show splash screen only not authenticated users.

tike012’s picture

I can confirm TommyChris' version works pretty well in D7.

One error I've run into: after setting a path to display, if you change the setting to display something else (like HTML with or without the template) and open the page in IE7 you get an error:

Line: 4
Char: 3
Error: Object expected
Code: 0
URL: http://website-url.com/specified-path

It looks like the module is remembering the specified path and trying to use it somehow even though the setting has been changed to display something else. I'm not sure if this is specific to this D7 version or present in 6.x as well since I haven't tried it on D6.

tike012’s picture

I've figured out my issue. It was not due to an error in setting the path, but rather an error because I tried to use PHP code in the input form.

The directions do specify "text/HTML" but I presumed it would run whatever text was entered through Drupal's input filters (PHP code, in my case). I was wrong. Hope this helps someone.

tike012’s picture

Scratch that, I was wrong again. It's not PHP code that causes the error (it works fine), but rather the fact that in IE the scripts and CSS that are normally pulled in by Drupal don't load on that page. I had some jQuery in the page when it wasn't loading the jQuery library. I had to manually include the script--an imperfect solution because this means on other browsers it gets loaded twice. That goes for CSS too.

chrisroane’s picture

Thanks TommyChris for updating the code and fixing those issues. I had actually fixed these in the version we have been using, but I didn't have email notification on for this thread....so I wasn't notified that people were commenting on it. :(

I would love to help out with making this the official Drupal 7 version, if my help is needed. I'm still fairly new to the community, so I'm not sure how the process for this works.

chrisroane’s picture

For those interested, I have created a new drupal 7 project called Splashify. Right now it is only a sandbox project, and I'm in the process of making it a full project.

Here is the sandbox project: http://drupal.org/sandbox/chrisroane/1423456

Here is the issue that covers the process in becoming a full project: http://drupal.org/node/1425392

My goal here is to actively maintain this project, including through Drupal 8. I could use help in getting this pushed through the approval process.

drupalnesia’s picture

My clients need a very simple Splash, just enter the NID then Splash is working both for D6 and D7.
So, I create TinyBox (Simple Splash) module, check out here: http://drupal.org/project/tinybox

TinyBox not only support Splash Screen from NID but also Content Type and Views. Since TinyBox supports Views then sky is the limit :)