My issues at drupal.org

Error running drush bam-destinations

Mon, 12/07/2010 - 8:29am

Hi There
Trying to restore a DB created with backup migrate.
When getting a list of destinations, with
drush bam-destinations
I run into this error.
Initialized Drupal site runireland.com at sites/runireland.com                                                             [notice]
Warning: The command callback function backup_migrate_drush_destinations has a deprecated name.  It must begin with        [notice]
drush_backup_migrate_.  Skipping hook functions.

Any ideas?

Google Chrome and Safari still download the image files in the background

Fri, 09/07/2010 - 2:24pm

The resources tab of the developer tool that comes with Chrome/Safari shows that all background images within the CSS file get downloaded, regardless of whether or not they are shown.
So the CSS sprites file overwrites rule is used to display the image, but the image file is still downloaded in the background.
I guess it's just being smart - the images might get used as some point, eg with a javsascript event.

This isn't an issue with Firefox or Internet Explorer.
I ran a network sniffer to confirm this.
It's only a minor issue

Fails on aggregated CSS files - Plan of attack.

Fri, 09/07/2010 - 2:13pm

In a follow-up to #849356: Does this really need to work on non-aggregated files?,
The module doesn't currently work when CSS aggregation is enabled.

I've had a look through the code, but I can't see the issue that's causing it.

On a broader sense, In terms of a plan of attack, I would suggest hooking into the CSS aggregation process, and manipulating the CSS aggregated file before it is saved at all.
This post
http://www.phpied.com/data-uris-mhtml-ie7-win7-vista-blues/
is the most comprehensive guide I could find in terms of how to manage all the different scenarios.

However, I don't like the browser sniffing employed in his example PHP class.
I think that the method used to satisfy IE on Vista would still work for all browsers [I still need to test this],
and that would be sufficient.
In this case, all background images are replaced by a single HTML file.
The original css is rewritten to point to this external css file.

Alternatively, we can do it 'properly' for normal browsers.
Rewrite the CSS file before saving the aggregated file.
And then add the 2 files in conditional stylesheets for IE7, IE6.
It does mean that they get the same information twice.

Its all a case of trade-off between satisfying different scenarios in any case.

Anyway, that's my 2c!

Alan

Coder Module Review

Fri, 09/07/2010 - 1:01pm

I ran the Coder module script through the module.
It picks up some syntax that doesn't comply with Drupal's coding standards.
Patch attached.

Does this really need to work on non-aggregated files?

Fri, 09/07/2010 - 11:57am

At the moment, this module works on non-aggregated CSS files.
I'm not sure that is really necessary.
It seem if you are concerned about reducing http requests [the aim of this module], then you'll have that enabled.

If that assumption is made, then coding could be a bit easier.
The work only need to happen when the aggregated css file is saved.

On the User interface, the setting to generate Data URIs could be disabled/hidden until the CSS aggregation box option was enabled.

User Interface

Fri, 09/07/2010 - 11:53am

Hi Dimitry
I'm continuing to test the module.
I have some thoughts on the interface.
If you agree with any of them, let me know, and I'll try to make some patches.

The overall principle I'm looking at is "Less is more".
Ideally a module should 'Just work'.
For example the jQuery Update modules simply needs to be enables in order to provide a new version of the jQuery Library.
No need for any configuration.
The pathauto module starts working straight away, but lets the user change the default settings.

So, with that in mind.

1. Settings page.
I don't know if there is a need for a separate settings page.
Ideally any configuration would be done directly on
admin/settings/performance
within the 'Bandwidth optimizations' section.

2. 32Kb per image limit
Is this setting really needed?
The reality is that IE7 and IE8 support is mandatory [from a practical point of view], so why would anyone uncheck that box?
The code could be written so as to make that available as an option if a future release [when usage of those browsers is lower], but not expose that setting to the user right now.
Is there was some extra work needed for IE6 only?
If so, then that could be an option, as many people [though not me just yet], choose not to support IE 6.

Regards
Alan

Thanks for the module!

Tue, 22/06/2010 - 3:45pm

Hi there,
Just starting to use Sass, and found this module.
The Latest in the Drupal-6--1 branch is working well for me.

I've developed a theme without using the module,
so I'm just wondering what advantages the module might have over 'standard' development.
[Standard for me means one terminal for editing, one for watching ].

Thanks and Regards
Alan

Realex Redirect Integration: Problem connecting back to the merchant's web site.

Tue, 22/06/2010 - 12:09pm

Hi, Im hoping someone can shed some light on why this might be happening

This error has just started showing up after I finalise test card transactions.
"Your transaction has been successful but there was a problem connecting back to the merchant's web site. Please contact the merchant and advise them that you received this error message. Thank you."

Everything had been working fine up until two days ago. Processing the test card, sending back information about the order number, login details, etc...

I'm using Drupal 6.16, Ubercart 6.x-2.2 and UC_Realex 6.x-2.0-beta3
I have made no alterations to ubercart, drupal or this module for that mater that would/should of affected the processing.

I have been in contact with realex support and they send me back this log message and saying its something wrong with uc module.

Page logs just display the error msg : An error has occurred during payment. Please contact us to ensure your order has submitted.

Find below how the response is being handled on your page .

$VAR1 = bless( {
''_protocol'' => ''HTTP/1.1'',
''_content'' => ''An error has occurred during payment. Please contact us to ensure your order has submitted.'',
''_rc'' => ''206'',
''_headers'' => bless( {
''connection'' => ''close'',
''client-response-num'' => 1,
''cache-control'' => [
''store, no-cache, must-revalidate'',

read more

How to remove page suggestion classes from $classes

Thu, 10/06/2010 - 10:21pm

I want rip out the suggestion classes for one of my themes - at first I thought this looked strait forward and could do something like this in themename_process_html()

$classes = str_replace(theme_get_suggestions(arg(), 'page', '-'), '', $classes);

This removes the page-arg(0) bit but nothing after it, see the screen-shot as to what is left behind.

Could someone point out what I'm doing wrong here, since this is using the same function that is being used to build these classes.

The screen shot is taking when I am on node/1, so the classes added by Drupal are page-node, page-node-, page-node-1

D7 ThemeAPI is incomplete

Thu, 03/06/2010 - 12:57pm

I created this issue to focus on some small changes to ThemeAPI that will make
D7 theming much more flexible and consistent. This issue is related to #812796
and #364470. I know I'm late. But, as for D7 user I think that without small
improvement, D7 will look like incomplete.

My goal is to create D7 theme using only css and template.php files. For D6
themes I always changed following things:

- submitted message (both for nodes and comments)
- comment title (with number of comment included)

In current D7 ThemeAPI I cannot change this variables without using tpl files.
Comment title is hardcoded, submitted message is hardcoded. I think, creating
translation for submitted message to change how it looks, is not the way it
should be used. I cannot achieve my no-tpl-files goal with hardcoded variables.

What I propose is to:
- make comment title as variable (like I did in patch in #812796, for
comment-wrapper.tpl.php) - now I have to use comment-wrapper.tpl.php to add,
for example, number of comments
- do not hardcode submitted message (if not an old $submitted variable (see
#364470), then, maybe something like theme('links'), which is also quite
difficult for new Drupal themers, maybe something like a theme setting).

Any other variable can be adjusted in template.php, thus I think leaving those
I mentioned will make D7 ThemeAPI incomplete.

Installation instructions

Fri, 28/05/2010 - 1:36pm

Hi there
Very interesting module.
In the installation instructions, you mention "delayed payment".

Set your payment method to authorization only, uc_viral only works with delayed collection only previously authorized payments

I don't see anywhere in Ubercart where this can be configured.
Any tips?
Does it only work with certain payment methods?

Regards
Alan

Referral and Response URLs for Realex

Thu, 20/05/2010 - 10:16am

Hi All,

Can anybody please help me to find out Referral and Response URLs for Realex.
I think Response url should be {sitename}/cart/uc_realex/complete

But my client is asking me about reffering url also.
Please suggest me this url ??????????

Create a recommended release

Tue, 18/05/2010 - 9:01am

Please make a dev release to support downloading with drush.
From the commandline prompt:
There is no recommended release for project galleryformatter on      [error]
Drupal 6.x. Ask the maintainer to create a release in order to be
compatible with drush and the drupal.org security broadcast system. A
development snapshot release is sufficient.

Adverts and Revisions

Thu, 13/05/2010 - 11:23am

In the Drupal 6 taxonomy system, terms are attached to nodes per revision. This means that if a node is removed from a taxonomy term, and a new revision is created, an entry will persist in the term_node table for the old revision. (This is a change from Drupal 5, which only maintained a "current" term state per node.)

The part of the ad module that fetches ads based on the categories does not appear to be aware of this. Ads are still appearing in the categories they used to be in. In cases where the old category is displayed on the same page as the new category, and the old category appears first, this can also stop them appearing in the categories their active revision is actually in.

I've had a quick shufty through the code and I think this is broadly the responsibility of the function adserve_cache_id() (in adcache.inc) - for example:

<?php

    case 'tids':
      $result = db_query("SELECT a.aid FROM {ads} a INNER JOIN {term_node} n ON a.aid = n.nid WHERE a.adstatus = 'active' AND n.tid IN(%s)", $id);
?>

(There are a few more entries with other cases, some of which also mention term_node. Presumably the changes outlined below would need to be applied to the other cases as well.)

In order for this query to respect the revision system, this needs to be modified to incorporate a reference to the node table. Something like this:

<?php

read more

Impossible to install contributed modules through website interface when the site's temp directory is outside the drupal root

Wed, 05/05/2010 - 8:25am

1) First I tried to install view with the project link. Getting errors ==> screenshot 1...
2) Afterwards I tried to download and to upload it again to my website, giving me errors ==> screenshot 2 (known issue that need to get fixed!)
3) Next try: unpack the archive and rar it: rar is not supprted
4) Last try: unpack the archive and zip it: gives errors ==> screesnhot 3

This is kinda critical, I see this as a mother issue for all those "small" problems.
(please correct the component of this issue)

specs: Win 7 | latest head | chrome

Module shouldn't assume Euro.

Fri, 30/04/2010 - 8:47am

Test for default currency instead.

CVS login failing

Thu, 29/04/2010 - 4:21pm

I can't get login via CVS.
I haven't used it for ages- so I could be screwing up myself
alan@ubuntu-dell:~/projects/drupal-devel/sites/contributions/modules$ cvs login
Logging in to :pserver:alanburke@cvs.drupal.org:2401/cvs/drupal-contrib
CVS password:
cvs login: authorization failed: server cvs.drupal.org rejected access to /cvs/drupal-contrib for user alanburke
I've rest my password on the account page, and waited a bit.
Still no joy.
Any ideas what I might be doing wrong?

Regards
Alan

Database Merging issue for existing site

Thu, 29/04/2010 - 7:18am

Hello all,

I need a support from you all,
I m facing a very serious problem from last night.
Actually my site is working on live environment from last 1 year. And i have a lots of changes in my database at local server.
Presently a lots of user registration and content creation is performing at my live site.
Now i want to upload my local database to live site, and also want to merge live database ( For users and content).

Can Anyone help me to migrate database with my local database.

Integration with RealAuth service for remote Realex integration

Wed, 28/04/2010 - 10:14am

This module would be far better if it supported the RealAuth protocol for remote processing of credit/debit cards ;)

Clean up color module settings page

Tue, 27/04/2010 - 10:49pm

I think this page needs more work.

There are quite a few differences between what this page looks like when Bartik is the admin theme vs. when Seven is the admin theme. See the attached screenshot for a comparison.

I'm also really not a fan of the #preview h1, etc code this preview is having us add. I would prefer to see it all in the preview.css file. I understand it's like that for people that may end up changing these things, but I don't think the preview needs to be flexible in this way at all. For example, I think using pixels sizes for fonts here is totally fine, given the special circumstances of having to deal with Seven's massive reset and other CSS files. It's also likely going to be a problem with other admin themes people decide to install down the line.