Return-Path: X-Original-To: apmail-poi-user-archive@www.apache.org Delivered-To: apmail-poi-user-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id BA8E7450A for ; Sun, 12 Jun 2011 06:24:32 +0000 (UTC) Received: (qmail 57247 invoked by uid 500); 12 Jun 2011 06:24:32 -0000 Delivered-To: apmail-poi-user-archive@poi.apache.org Received: (qmail 56931 invoked by uid 500); 12 Jun 2011 06:24:25 -0000 Mailing-List: contact user-help@poi.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "POI Users List" Delivered-To: mailing list user@poi.apache.org Received: (qmail 56912 invoked by uid 99); 12 Jun 2011 06:24:22 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 12 Jun 2011 06:24:22 +0000 X-ASF-Spam-Status: No, hits=-0.0 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (athena.apache.org: local policy) Received: from [209.85.210.50] (HELO mail-pz0-f50.google.com) (209.85.210.50) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 12 Jun 2011 06:24:16 +0000 Received: by pzk2 with SMTP id 2so1762917pzk.9 for ; Sat, 11 Jun 2011 23:23:55 -0700 (PDT) MIME-Version: 1.0 Received: by 10.143.61.20 with SMTP id o20mr600292wfk.367.1307859834999; Sat, 11 Jun 2011 23:23:54 -0700 (PDT) Received: by 10.142.44.10 with HTTP; Sat, 11 Jun 2011 23:23:54 -0700 (PDT) In-Reply-To: References: Date: Sun, 12 Jun 2011 10:23:54 +0400 Message-ID: Subject: Re: "Not implemented" exception for SUMIF function in xlsx file From: Yegor Kozlov To: POI Users List Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On Sun, Jun 12, 2011 at 8:28 AM, Chris McCann wrote= : > Thanks, Nick. =A0Your response made me dig deeply into all of the functio= ns > used in dependent cells. =A0I found that we were using the EOMONTH functi= on to > calculate the first/last day of a month. =A0That function is part of the > Analysis Toolpak, and I assume that's why it's not implemented. =A0Is the= re a > list of implemented functions somewhere in the docs? > Mappings for the AnalysisToolpack are defined here: http://svn.apache.org/repos/asf/poi/trunk/src/java/org/apache/poi/ss/formul= a/atp/AnalysisToolPak.java At the moment POI implements only five ATP functions: ISEVEN ISODD MROUND RANDBETWEEN YEARFRAC It should be easy to add support for EOMONTH, just follow the pattern: create a sub-class of FreeRefFunction and register it in AnalysisToolPak.java. Patches are welcome. Let me know if you need guidance. > Changing our EOMONTH for a different function that is supported made > everything work, so that problem is solved. > > Not being familiar with the code base in terms of how unimplemented > functions are handled it seems to me it should be possible to at least > surface the name of the requested, unimplemented function in the exceptio= n > message. =A0That would be tremendously helpful for debugging. > I'm going to look into the code and see if it is possible to propagate function name into the exception message. Yegor --------------------------------------------------------------------- To unsubscribe, e-mail: user-unsubscribe@poi.apache.org For additional commands, e-mail: user-help@poi.apache.org