On Tue, 2009-08-11 at 15:42 -0500, Pradeep Kilambi wrote:
> So lookong through the code, I think I know what I problem is. The
> perl
> code when settup up the channel cloning process does'nt not account
> for
> checksum value. So what ends up happening is, after the perl pxt
> setup
> page it navigates to edit.do which makes assumptions based on channel
> creation and edits and ends up trying to set the value thats not in
> the
> form to being with. A workaround fix to get the cloning working for
> now
> is to do something like,
>
> ---
> a/java/code/src/com/redhat/rhn/frontend/action/channel/manage/EditChannelAct
> +++
> b/java/code/src/com/redhat/rhn/frontend/action/channel/manage/EditChannelAct
> @@ -568,7 +568,9 @@ public class EditChannelAction extends RhnAction
> implements
> form.set("maintainer_phone", c.getMaintainerPhone());
> form.set("maintainer_email", c.getMaintainerEmail());
> form.set("support_policy", c.getSupportPolicy());
> - form.set("checksum", c.getChecksum().getLabel());
> + if (!c.isCloned()) {
> + form.set("checksum", c.getChecksum().getLabel());
> + }
> if
> (c.isGloballySubscribable(ctx.getLoggedInUser().getOrg())) {
> form.set("per_user_subscriptions", "all");
> }
>
> eventually once the cloning logic in perl supports the checksum
> stuff,
> we can remove this additional check.
>
> I'll looking into adding this ability for the cloned channels. Please
> open a bug on this and I'll knock it out asap.
>
> Thanks,
> ~ PradPrad, I filed a bug as requested, grabbed the source and patched as suggested. This definitely fixed the problem short term. Thanks for helping me through this. -- Andy Speagle "THE Student" - UCATS Wichita State University
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Spacewalk-devel mailing list [email protected] https://www.redhat.com/mailman/listinfo/spacewalk-devel
