Will there be a Drupal 7 version of this module? It saves me so much time and I'm starting to do more Drupal 7 sites. Thanks!

Comments

graytoby’s picture

StatusFileSize
new7.45 KB

I hacked a quick version. So basically it works but it needs a lot of love. Filter works properly but it spits a lot of errors, see below.
Warning: preg_match() expects parameter 2 to be string, array given in table_altrow_process() (line 33 of /var/www/html/sites/all/modules/table_altrow/table_altrow.module).
Code is from 6.x branch and needs a lot of cleanup, perhaps even complete rewrite, so if someone has more time, feel free to do so.

graytoby’s picture

StatusFileSize
new7.1 KB

I've cleaned the code a bit. Above warning still persist.

graytoby’s picture

Status: Active » Needs review
StatusFileSize
new7.11 KB

I fixed the php warning. I'd say this version works , but it needs more testing.

deviantintegral’s picture

Version: 6.x-1.1 » 7.x-1.x-dev

I just pushed the following commits to a 7.x-1.x branch. If it works fine for you, I'll create a stable release.

3037309 #1156058 by spongecat: Initial port to Drupal 7.
238418b #1156058 by spongecat: Further cleanup for the Drupal 7 port.
a0ebbba #1156058 by spongecat: Fix preg_replace() warning.
1106abc #1156058: Remove old CVS $Id$ keywords.
1b731ef #1156058: Remove version number from info file as it is added by the packaging system.
6831094 #1156058: Code style fixes for indentation and phpdoc.
e90506c #1156058: Update the README to be consistent with the changes in Drupal 7.
dba6173 #1156058: Change the filter title to match the text style of other Drupal 7 filters.
0fccfee #1156058: Move the filter code from the prepare step to the process step.
f743eae #1156058: Define the filters array to prevent PHP notices.
def8660 #1156058: Rename filter callbacks to not conflict with any hook definitions.
5015e15 #1156058: Restore str_replace_count() as str_replace() doesn't have an offset parameter.
aa85488 #1156058: Modify str_replace_count() to follow Drupal coding standards.

graytoby’s picture

I just tested it, works fine.

deviantintegral’s picture

Status: Needs review » Fixed

Tagged and released - the package should be up in a few minutes.

Status: Fixed » Closed (fixed)

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