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!
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | table_altrow-7.x-1.x-dev.tar_.gz | 7.11 KB | graytoby |
| #2 | table_altrow-7.x-1.x-dev.tar_.gz | 7.1 KB | graytoby |
| #1 | table_altrow-7.x-1.x-dev.tar_.gz | 7.45 KB | graytoby |
Comments
Comment #1
graytoby commentedI 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.
Comment #2
graytoby commentedI've cleaned the code a bit. Above warning still persist.
Comment #3
graytoby commentedI fixed the php warning. I'd say this version works , but it needs more testing.
Comment #4
deviantintegral commentedI 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.
Comment #5
graytoby commentedI just tested it, works fine.
Comment #6
deviantintegral commentedTagged and released - the package should be up in a few minutes.