Issue with Form Helper array('type' => 'file') - CakePHP

The old Google Groups will be going away soon, but your browser is incompatible with the new version.

« Groups Home

CakePHP

Home

Discussions

+ new post

About this group

Join this group

Issue with Form Helper array('type' => 'file')

Options

9 messages - Collapse all

Kro

Feb 23 2012, 5:28 am

From: Kro <yenohamonin...@gmail.com>

Date: Thu, 23 Feb 2012 02:28:03 -0800 (PST)

Local: Thurs, Feb 23 2012 5:28 am

Subject: Issue with Form Helper array('type' => 'file')

Print | Individual message | Show original | Report this message | Find messages by this author

jeremyharris

Feb 23 2012, 10:34 am

From: jeremyharris <funeralm...@gmail.com>

Date: Thu, 23 Feb 2012 07:34:13 -0800 (PST)

Local: Thurs, Feb 23 2012 10:34 am

Subject: Re: Issue with Form Helper array('type' => 'file')

Print | Individual message | Show original | Report this message | Find messages by this author

Kro

Feb 23 2012, 6:40 pm

From: Kro <yenohamonin...@gmail.com>

Date: Thu, 23 Feb 2012 15:40:11 -0800 (PST)

Local: Thurs, Feb 23 2012 6:40 pm

Subject: Re: Issue with Form Helper array('type' => 'file')

Print | Individual message | Show original | Report this message | Find messages by this author

Thanks for taking the time to reply @jeremyharris. The record is still
getting written but the 'Rate' data is missing. The following is an
example of a saved record. As you can see the 'Rate' data is missing.

Array
(
[Rate] => Array
(
[id] => 92
[id_ref] =>
[title] =>
[short_description] =>
[description] =>
[price_quote_as] =>
[adult_supplement] => 0.00
[child_supplement] => 0.00
[supplement] => 0.00
[minimum_stay] => 0
[maximum_stay] => 0
[imagefile] =>
[enabled] => 0
[special_offer] => 0
[list_no] => 0
[availability] =>
[valid_from] => 0000-00-00 00:00:00
[valid_to] => 0000-00-00 00:00:00
[is_corporate] => 0
[created] => 0000-00-00 00:00:00
[updated] => 0000-00-00 00:00:00
[deleted] => 0
[dirname] =>
[basename] =>
)

[Room] => Array
(
[0] => Array
(
[id] => 15
[title] => Deluxe Double Room
[description] => Situated in the new wing of....
[min_adults] => 2
[max_adults] => 2
[max_children] => 0
[no_of_rooms] => 0
[enabled] => 1
[list_no] => 2
[created] => 2011-09-04 21:37:08
[updated] => 2011-09-05 17:00:55
[deleted] => 0
[dirname] => img
[basename] => deluxe_bedroom_2.jpg
[RatesRoom] => Array
(
[id] => 1942
[rate_id] => 92
[room_id] => 15
[dirname] =>
[basename] =>
)

)

[CorporateAccount] => Array
(
)

)

On Feb 23, 3:34 pm, jeremyharris <funeralm...@gmail.com> wrote:

- Show quoted text -

jeremyharris

View profile

More options Feb 23 2012, 9:19 pm

From: jeremyharris <funeralm...@gmail.com>

Date: Thu, 23 Feb 2012 18:19:40 -0800 (PST)

Local: Thurs, Feb 23 2012 9:19 pm

Subject: Re: Issue with Form Helper array('type' => 'file')

Print | Individual message | Show original | Report this message | Find messages by this author

What behaviors are you using besides Transfer? Coupler tends to mess with
the data because it expects *just* the fields in the Media.Attachment
model. Try removing Coupler if it's attached (since you're not using it).

Also, unit tests are your friend. It would be much easier to try things and
debug using unit tests rather than not.

Kro

Feb 24 2012, 4:02 am

From: Kro <yenohamonin...@gmail.com>

Date: Fri, 24 Feb 2012 01:02:05 -0800 (PST)

Local: Fri, Feb 24 2012 4:02 am

Subject: Re: Issue with Form Helper array('type' => 'file')

Print | Individual message | Show original | Report this message | Find messages by this author

jeremyharris

Feb 24 2012, 12:48 pm

From: jeremyharris <funeralm...@gmail.com>

Date: Fri, 24 Feb 2012 09:48:32 -0800 (PST)

Local: Fri, Feb 24 2012 12:48 pm

Subject: Re: Issue with Form Helper array('type' => 'file')

Print | Individual message | Show original | Report this message | Find messages by this author

Kro

Feb 25 2012, 1:48 pm

From: Kro <yenohamonin...@gmail.com>

Date: Sat, 25 Feb 2012 10:48:59 -0800 (PST)

Local: Sat, Feb 25 2012 1:48 pm

Subject: Re: Issue with Form Helper array('type' => 'file')

Print | Individual message | Show original | Report this message | Find messages by this author

Thank you so much, it's working now! You were spot on, it was the
Couplers beforesave that was unsetting the data. I have stopped it
doing that now and I can't see why that would cause any other issues.

The change I have made is as follows:
function beforeSave(&$Model) {
if (!$Model->exists()) {
if (!isset($Model->data[$Model->alias]['file'])) {
removed unset ---> //unset($Model->data[$Model-

>alias]);

return true;
}
} else {
if (isset($Model->data[$Model->alias]['delete'])
&& $Model->data[$Model->alias]['delete'] !== '0') {
$Model->delete();
unset($Model->data[$Model->alias]);
return true;
}
}

Thanks again...

On Feb 24, 5:48 pm, jeremyharris <funeralm...@gmail.com> wrote:

- Show quoted text -

jeremyharris

Feb 25 2012, 3:17 pm

From: jeremyharris <funeralm...@gmail.com>

Date: Sat, 25 Feb 2012 12:17:39 -0800 (PST)

Local: Sat, Feb 25 2012 3:17 pm

Subject: Re: Issue with Form Helper array('type' => 'file')

Print | Individual message | Show original | Report this message | Find messages by this author

Awesome, glad you figured it out!

- Show quoted text -

On Saturday, February 25, 2012 10:48:59 AM UTC-8, Kro wrote:

> Thank you so much, it's working now! You were spot on, it was the
> Couplers beforesave that was unsetting the data. I have stopped it
> doing that now and I can't see why that would cause any other issues.

> The change I have made is as follows:
> function beforeSave(&$Model) {
> if (!$Model->exists()) {
> if (!isset($Model->data[$Model->alias]['file'])) {
> removed unset --->
> //unset($Model->data[$Model-
> >alias]);
> return true;
> }
> } else {
> if (isset($Model->data[$Model->alias]['delete'])
> && $Model->data[$Model->alias]['delete'] !== '0')
> {
> $Model->delete();
> unset($Model->data[$Model->alias]);
> return true;
> }
> }

> Thanks again...

> On Feb 24, 5:48 pm, jeremyharris <funeralm...@gmail.com> wrote:
> > Okay I think I misspoke. You do need the coupler. (Sorry lack of docs
> and
> > it's been a while since I had to configure.)

> > I'd stick some debug statements through all of the behaviors' beforeSave
> > calls to try and figure out what's happening. I'm suspicious that the
> rate
> > data is being removed in Coupler's beforeSave due to some issue with the
> > 'file' key not being there or not being validated. This will hopefully
> give
> > you a better understanding of what's going on.

> > On Friday, February 24, 2012 1:02:05 AM UTC-8, Kro wrote:

> > > Thanks for your reply. Okay, sounds like it could be related to the
> > > Coupler then. I tried removing the Coupler and the 'Rate' data is then
> > > saving correctly but the image is missing. I am using 'dirname' and
> > > 'basename' fields in the 'Rate' table rather than using a separate
> > > Attachment table, does this mean I don't need the Coupler element?

> > > On Feb 24, 2:19 am, jeremyharris <funeralm...@gmail.com> wrote:
> > > > What behaviors are you using besides Transfer? Coupler tends to mess
> > > with
> > > > the data because it expects *just* the fields in the
> Media.Attachment
> > > > model. Try removing Coupler if it's attached (since you're not using
> > > it).

> > > > Also, unit tests are your friend. It would be much easier to try
> things
> > > and
> > > > debug using unit tests rather than not.

Kro

Feb 25 2012, 10:01 pm

From: Kro <yenohamonin...@gmail.com>

Date: Sat, 25 Feb 2012 19:01:56 -0800 (PST)

Local: Sat, Feb 25 2012 10:01 pm

Subject: Re: Issue with Form Helper array('type' => 'file')

Print | Individual message | Show original | Report this message | Find messages by this author

Couldn't have done it without you...cheers...

On Feb 25, 8:17 pm, jeremyharris <funeralm...@gmail.com> wrote:

- Show quoted text -

> Awesome, glad you figured it out!

> On Saturday, February 25, 2012 10:48:59 AM UTC-8, Kro wrote:

> > Thank you so much, it's working now! You were spot on, it was the
> > Couplers beforesave that was unsetting the data. I have stopped it
> > doing that now and I can't see why that would cause any other issues.

> > The change I have made is as follows:
> > function beforeSave(&$Model) {
> > if (!$Model->exists()) {
> > if (!isset($Model->data[$Model->alias]['file'])) {
> > removed unset --->
> > //unset($Model->data[$Model-
> > >alias]);
> > return true;
> > }
> > } else {
> > if (isset($Model->data[$Model->alias]['delete'])
> > && $Model->data[$Model->alias]['delete'] !== '0')
> > {
> > $Model->delete();
> > unset($Model->data[$Model->alias]);
> > return true;
> > }
> > }

> > Thanks again...

> > On Feb 24, 5:48 pm, jeremyharris <funeralm...@gmail.com> wrote:
> > > Okay I think I misspoke. You do need the coupler. (Sorry lack of docs
> > and
> > > it's been a while since I had to configure.)

> > > I'd stick some debug statements through all of the behaviors' beforeSave
> > > calls to try and figure out what's happening. I'm suspicious that the
> > rate
> > > data is being removed in Coupler's beforeSave due to some issue with the
> > > 'file' key not being there or not being validated. This will hopefully
> > give
> > > you a better understanding of what's going on.

> > > On Friday, February 24, 2012 1:02:05 AM UTC-8, Kro wrote:

> > > > Thanks for your reply. Okay, sounds like it could be related to the
> > > > Coupler then. I tried removing the Coupler and the 'Rate' data is then
> > > > saving correctly but the image is missing. I am using 'dirname' and
> > > > 'basename' fields in the 'Rate' table rather than using a separate
> > > > Attachment table, does this mean I don't need the Coupler element?

> > > > On Feb 24, 2:19 am, jeremyharris <funeralm...@gmail.com> wrote:
> > > > > What behaviors are you using besides Transfer? Coupler tends to mess
> > > > with
> > > > > the data because it expects *just* the fields in the
> > Media.Attachment
> > > > > model. Try removing Coupler if it's attached (since you're not using
> > > > it).

> > > > > Also, unit tests are your friend. It would be much easier to try
> > things
> > > > and
> > > > > debug using unit tests rather than not.

End of messages

« Back to Discussions

« Newer topic

Older topic »

Create a group - Google Groups - Google Home - Terms of Service - Privacy Policy

Account Options